-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Revert "fix(deps): update module github.com/prometheus/common to v0.63.0 (#12656)" #12673
Conversation
…3.0 (open-telemetry#12656)" This reverts commit c11fa87.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12673 +/- ##
=======================================
Coverage 91.63% 91.63%
=======================================
Files 479 479
Lines 26423 26423
=======================================
Hits 24214 24214
Misses 1749 1749
Partials 460 460 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@jade-guiton-dd could you fix the merge conflict? |
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.
cc @ArthurSens
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.
Yes, please revert! Sorry I didn't see the original PR on core. I've mentioned in contrib that we can't move forward with this upgrade yet. open-telemetry/opentelemetry-collector-contrib#38745 (comment)
I wonder why CI caught the problem on contrib, but not on core 🤔
Description
Contrib tests are currently failing on main with the following build error:
I believe this is because of the update in #12656: there is a breaking change in
github.com/prometheus/common
0.63.0 that is incompatible with the version ofgithub.com/prometheus
used in some contrib modules. While the latter package has been updated on main, the updated version hasn't been released yet, so I this the only solution is to revert this PR until then.