-
Notifications
You must be signed in to change notification settings - Fork 810
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
Support otlp metadata ingestion #6617
Support otlp metadata ingestion #6617
Conversation
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.
Let's also update /~https://github.com/cortexproject/cortex/blob/master/integration/otlp_test.go ?
It looks like query fuzz test becomes super flaky after the upgrade. Issue seems the same. value 0 and NaN. Something worth checking.
|
a910ebc
to
7cfffb8
Compare
@yeya24 |
Signed-off-by: SungJin1212 <tjdwls1201@gmail.com>
7cfffb8
to
9bd9db7
Compare
Merging this PR first as the failure is unrelated |
Looks like the result mismatch comes from prometheus/prometheus#14941 Since it is a legit behavior, I propose that for any fuzzy tests that we compare with old release of Cortex (v1.18.x, v1.19.x), let's don't enable |
This PR adds ingestion OTLP metadata.
Which issue(s) this PR fixes:
Fixes #6247
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]