Adding Udaan as Adopter of Thanos #5676
Closed
Conversation
This file contains 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
2 tasks
Signed-off-by: Prakul Jain <[email protected]>
Signed-off-by: Prakul Jain <[email protected]>
* added_devcontainer Signed-off-by: Akshit42-hue <[email protected]> * correct indentation Signed-off-by: Axit Patel <[email protected]> Signed-off-by: Axit Patel <[email protected]> * correct_Identation Signed-off-by: Akshit42-hue <[email protected]> * enter_empty_line Signed-off-by: Akshit42-hue <[email protected]> Signed-off-by: Akshit42-hue <[email protected]> Signed-off-by: Axit Patel <[email protected]> Signed-off-by: Prakul Jain <[email protected]>
* Cache vertical shards in query frontend The vertical sharding middleware is currently executed after the caching middleware. Because of this, individual vertical shards are not getting cached when the succeed. Caching is only done when the entire requests including all shards complete successfully. This commit moves the vertical sharding middleware before the caching middleware. It also modifies caching keys to contain the total shards and the shard number so that each vertical shard gets an independent caching key. Signed-off-by: Filip Petkovski <[email protected]> * Adjust cache key tests Signed-off-by: Filip Petkovski <[email protected]> * Remove source of flakiness by using sync.Cond Signed-off-by: Filip Petkovski <[email protected]> Signed-off-by: Filip Petkovski <[email protected]> Signed-off-by: Prakul Jain <[email protected]>
…-io#5666) * Bump protoc to 3.20.1 from 3.4.0 Signed-off-by: Pedro Tanaka <[email protected]> * Adding changelog Signed-off-by: Pedro Tanaka <[email protected]> * Improving scripts Signed-off-by: Pedro Tanaka <[email protected]> * Remove entry from changelog Signed-off-by: Pedro Tanaka <[email protected]> * Passing protoc version from Makefile to genproto script Signed-off-by: Pedro Tanaka <[email protected]> Signed-off-by: Pedro Tanaka <[email protected]> Co-authored-by: Giedrius Statkevičius <[email protected]> Signed-off-by: Prakul Jain <[email protected]>
… tenant (thanos-io#5565) * Allow per-tenant limits to be configured via file Signed-off-by: Douglas Camata <[email protected]> * Refactor Receive's limiting logic Signed-off-by: Douglas Camata <[email protected]> * Fix some methods that were in plural Signed-off-by: Douglas Camata <[email protected]> * Improve metric description Signed-off-by: Douglas Camata <[email protected]> * Add a TODO for later Signed-off-by: Douglas Camata <[email protected]> * Do some cleanup after moving limits to config file Signed-off-by: Douglas Camata <[email protected]> * Isolate rest of limiting logic from the handler Signed-off-by: Douglas Camata <[email protected]> * Small refactor to the request limiter Signed-off-by: Douglas Camata <[email protected]> * Rename MergeWith -> OverlayWith Signed-off-by: Douglas Camata <[email protected]> * Update changelog Signed-off-by: Douglas Camata <[email protected]> * Update documentation Signed-off-by: Douglas Camata <[email protected]> * Add missing copyright notice to few files Signed-off-by: Douglas Camata <[email protected]> * Fix test after change in config file tenants Signed-off-by: Douglas Camata <[email protected]> * Retrigger CI because of bundled-Cortex failing test Signed-off-by: Douglas Camata <[email protected]> * Expose default limits as metrics Signed-off-by: Douglas Camata <[email protected]> * Retrigger CI Signed-off-by: Douglas Camata <[email protected]> * Replace comment with a TODOs Signed-off-by: Douglas Camata <[email protected]> * Fix changelog after bad merge Signed-off-by: Douglas Camata <[email protected]> Signed-off-by: Douglas Camata <[email protected]> Signed-off-by: GitHub <[email protected]> Signed-off-by: Prakul Jain <[email protected]>
|
Had few issues while signing previous commits. Opened #5678 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Udaan is using Thanos in production. We are a legally registered company in India.
Signed-off-by: Prakul Jain [email protected]
Changes
Verification
Does not require testing. Followed these steps.