-
Notifications
You must be signed in to change notification settings - Fork 812
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
Update Thanos to v0.19.1-0.20210923155558-c15594a03c45. #4505
Conversation
Signed-off-by: Peter Štibraný <[email protected]>
Signed-off-by: Peter Štibraný <[email protected]>
…using lint check. Signed-off-by: Peter Štibraný <[email protected]>
@@ -80,3 +80,5 @@ replace github.com/bradfitz/gomemcache => github.com/themihai/gomemcache v0.0.0- | |||
|
|||
// TODO review the change introduced by https://github.com/grpc/grpc-go/pull/4416 before upgrading to 1.39.0 | |||
replace google.golang.org/grpc => google.golang.org/grpc v1.38.0 | |||
|
|||
replace github.com/thanos-io/thanos v0.22.0 => github.com/thanos-io/thanos v0.19.1-0.20210923155558-c15594a03c45 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, whenever you pin a version, add a comment stating why, and under what conditions we will remove the pin.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pin is used to make sure we use version from master. Otherwise go modules want to use v0.22, which is latest tag (update: now it’s actually v0.23, which is more recent), but is not reachable from master, so we would be stuck with version from July.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah put it in the file so I can see it and not have to trace back through PRs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here are the changes I identified in Thanos code:
I think the first one would deserve a mention in the changelog, if it works for users. |
Signed-off-by: Bryan Boreham <[email protected]>
* Add 1.11.0-rc.0 marker to CHANGELOG * Add changelog entries for #4505 Signed-off-by: Bryan Boreham <[email protected]>
…t#4505) * Update Thanos to v0.19.1-0.20210923155558-c15594a03c45. Signed-off-by: Peter Štibraný <[email protected]> * Added forgotten vendored files Signed-off-by: Peter Štibraný <[email protected]> * Use original version of file with CRLF line separators, to avoid confusing lint check. Signed-off-by: Peter Štibraný <[email protected]> Signed-off-by: Manish Kumar Gupta <[email protected]>
…texproject#4507) * Add 1.11.0-rc.0 marker to CHANGELOG * Add changelog entries for cortexproject#4505 Signed-off-by: Bryan Boreham <[email protected]>
…t#4505) * Update Thanos to v0.19.1-0.20210923155558-c15594a03c45. Signed-off-by: Peter Štibraný <[email protected]> * Added forgotten vendored files Signed-off-by: Peter Štibraný <[email protected]> * Use original version of file with CRLF line separators, to avoid confusing lint check. Signed-off-by: Peter Štibraný <[email protected]> Signed-off-by: Alvin Lin <[email protected]>
…texproject#4507) * Add 1.11.0-rc.0 marker to CHANGELOG * Add changelog entries for cortexproject#4505 Signed-off-by: Bryan Boreham <[email protected]> Signed-off-by: Alvin Lin <[email protected]>
What this PR does: This PR updates Thanos to v0.19.1-0.20210923155558-c15594a03c45.
This PR does not integrate new Thanos' compactor feature for skipping blocks with out of order chunks.
This PR also updates Prometheus, but doesn't integrate new snapshot on shutdown feature yet.
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]