-
Notifications
You must be signed in to change notification settings - Fork 535
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
Upgrade to latest mimir-prometheus with Prometheus 3.0 changes #9844
base: main
Are you sure you want to change the base?
Conversation
cb6d823
to
a1b4f48
Compare
a1b4f48
to
6a3bfe1
Compare
To address one small set of test failures: #9854 |
71feed6
to
69c0bd2
Compare
Signed-off-by: Arve Knudsen <[email protected]>
* Fix failing tests due to left-open range change * Set lookback delta for `TestSplitQueryByInterval` test cases, and fix expectations to match left-open range change * Update querier tests to match new behaviour
Signed-off-by: Arve Knudsen <[email protected]>
d457cc1
to
a4ae1eb
Compare
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.
I've reviewed the changes to pkg/streamingpromql
and those lgtm. I have not reviewed in detail the rest of the changes, but they also look good at a glance.
Signed-off-by: Arve Knudsen <[email protected]>
05a17d8
to
1d0ea03
Compare
Signed-off-by: Arve Knudsen <[email protected]>
What this PR does
Upgrade to latest mimir-prometheus@main, which syncs in Prometheus 3.0 changes.
NB: I add a
go-kit/log
toslog
adapter, since Prometheus now uses the latter.Which issue(s) this PR fixes or relates to
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]
.about-versioning.md
updated with experimental features.