-
Notifications
You must be signed in to change notification settings - Fork 856
Add Overrides API component and rename old overrides to overrides-configs #6975
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 4 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
aad1fee
rename overrides to overridesConfig
bogdan-st 60d09fb
initial overrides api draft
bogdan-st 9e8166d
Merge branch 'master' into overrides_api
bogdan-st f97b319
fix docs and integration tests
bogdan-st 608f1ab
remove from all target
bogdan-st 15f9f3c
remove consul and unneeded flags from test
bogdan-st a2a7665
missed some
bogdan-st 5e21b22
suggestions
bogdan-st 61632a1
Merge branch 'master' into overrides_api
bogdan-st dff25e3
sugestions from dsabsay
bogdan-st 8e50e9a
lint
bogdan-st 06195e0
lint2
bogdan-st 8ba408b
modernize?
bogdan-st 9a1bb28
Merge upstream/master into overrides_api
bogdan-st 4d9e7c7
fix import
bogdan-st 83b1b7d
fix sum
bogdan-st 255d648
Merge branch 'master' into overrides_api
friedrichg d2e8c3a
Missing from rebase
friedrichg 1640ed0
Modernize again
friedrichg 1b33cea
fix merging
friedrichg 5dd9cd7
Modernize it
friedrichg cd5e8d4
Close bucket when stopping to prevent leakage
friedrichg 3ceffe9
Include initial guide on overrides API
friedrichg fa29aa7
Make sure limits include maximum
friedrichg 9257ce5
Very unlikely, let's remove it
friedrichg 9cc96fd
Merge pull request #1 from cortexproject/pr/6975
bogdan-st 9336c34
Fix more things
friedrichg b070ba2
Make sure readers are closed
friedrichg 8277244
Merge branch 'master' into pr/6975
friedrichg 03a9e9d
Fix tests
friedrichg c431a56
Move tests to its own testing thread
friedrichg 10b4787
After more local testing
friedrichg ab1b285
Merge pull request #2 from cortexproject/pr/6975
bogdan-st File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.