Use prometheus v2.9.2, common v0.4.0 & tsdb v0.8.0#1133
Merged
bwplotka merged 3 commits intothanos-io:masterfrom May 31, 2019
Merged
Use prometheus v2.9.2, common v0.4.0 & tsdb v0.8.0#1133bwplotka merged 3 commits intothanos-io:masterfrom
bwplotka merged 3 commits intothanos-io:masterfrom
Conversation
povilasv
reviewed
May 11, 2019
Member
povilasv
left a comment
povilasv
left a comment
There was a problem hiding this comment.
Minor nit, other than that 💯
Member
|
This needs a rebase. I have no strong opinion on the tsdb_errors vs tsdberrors thing, while tsdb may be doing something wrong here I generally prefer to stay as close as possible to goimports as named imports usually just end up confusing people. In that sense i think i would prefer not renaming and sticking to the upstream and try to continue that consistently. (And fix it in tsdb) In any case once rebased this is good to go. |
sylr
force-pushed
the
prometheus-2.9.2
branch
4 times, most recently
from
81886b5 to
dfa6499
Compare
Member
|
@povilasv is it okay with you that we leave it like |
Member
|
@GiedriusS yes @brancz makes a good point |
bwplotka
reviewed
May 14, 2019
Member
bwplotka
left a comment
bwplotka
left a comment
There was a problem hiding this comment.
LGTM, but let's fix this package prefix. Also this is not the most important part - the important part is to grab this and deploy on clusters ): It's always critical change.
Can we walk through the changelog and grab what changes and map to what we use? I was always doing it for last Prom/TSDB upgrades
https://github.com/improbable-eng/thanos/blob/master/CHANGELOG.md
For example as we had in previous one:
Changes that affects Thanos:
query:
[ENHANCEMENT] In histogram_quantile merge buckets with equivalent le values. #5158.
[ENHANCEMENT] Show list of offending labels in the error message in many-to-many scenarios. #5189
[BUGFIX] Fix panic when aggregator param is not a literal. #5290
ruler:
[ENHANCEMENT] Reduce time that Alertmanagers are in flux when reloaded. #5126
[BUGFIX] prometheus_rule_group_last_evaluation_timestamp_seconds is now a unix timestamp. #5186
[BUGFIX] prometheus_rule_group_last_duration_seconds now reports seconds instead of nanoseconds. Fixes our issue #1027
[BUGFIX] Fix sorting of rule groups. #5260
store: [ENHANCEMENT] Fast path for EmptyPostings cases in Merge, Intersect and Without.
tooling: [FEATURE] New dump command to tsdb tool to dump all samples.
compactor:
[ENHANCEMENT] When closing the db any running compaction will be cancelled so it doesn't block.
[CHANGE] Renamed flag --sync-delay to --consistency-delay #1053
Member
|
Thanks for the work @sylr ! Also please rebase, this failing preview might be worrying (: |
Contributor
Author
|
All checks are green now. |
Contributor
Author
|
It's conflicting again, will you merge it if I rebase ? |
Member
|
I think @bwplotka was primarily waiting on something doing:
If we can validate this, I'm happy with moving forward. |
Member
I believe this comment by @bwplotka is still not addressed. It would be nice to have these things in the CHANGELOG.md 😄 Could you do that, @sylr? Really awesome job that you've done here 👍 |
Member
|
Yea, the changelog is quite manual job ): |
Member
|
Seems like a rebase is not difficult as well @sylr |
added 2 commits
May 31, 2019 13:37
- prometheus/prometheus@v2.9.2+incompatible - prometheus/common@v0.4.0 - prometheus/tsdb@v0.8.0 Signed-off-by: Sylvain Rabot <s.rabot@lectra.com>
Signed-off-by: Sylvain Rabot <s.rabot@lectra.com>
sylr
changed the title
Member
|
However we need to still have a changelog for this @sylr ): This adds a lot, if you can grab what Prometheus and TSDB Changelog covers from our previous versions to the thing we update in this PR offline we can work out what is interesting/impacting Thanos users. What do you think? |
Contributor
Author
|
TSDB changelog: Prometheus changelog: |
povilasv
approved these changes
May 31, 2019
Signed-off-by: Sylvain Rabot <s.rabot@lectra.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
No description provided.