-
Notifications
You must be signed in to change notification settings - Fork 45
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
editoast: update opentelemetry dependencies #7569
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #7569 +/- ##
============================================
- Coverage 28.65% 28.64% -0.01%
Complexity 2058 2058
============================================
Files 1247 1247
Lines 154280 154280
Branches 3031 3031
============================================
- Hits 44203 44191 -12
- Misses 108275 108287 +12
Partials 1802 1802
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e176be5
to
8b90c1c
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.
LGTM (tested)
By default, Dependabot opens a new PR for each dependency. However, it doesn't work well for OpenTelemetry ecosystem which usually require a manual operation (see #7102, #7569, #8713). This modification in how Dependabot operates should help a bit. See [documentation](https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups) Signed-off-by: Jean SIMARD <woshilapin@tuziwo.info>
By default, Dependabot opens a new PR for each dependency. However, it doesn't work well for OpenTelemetry ecosystem which usually require a manual operation (see #7102, #7569, #8713). This modification in how Dependabot operates should help a bit. See [documentation](https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups) Signed-off-by: Jean SIMARD <woshilapin@tuziwo.info>
By default, Dependabot opens a new PR for each dependency. However, it doesn't work well for OpenTelemetry ecosystem which usually require a manual operation (see #7102, #7569, #8713). This modification in how Dependabot operates should help a bit. See [documentation](https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups) Signed-off-by: Jean SIMARD <woshilapin@tuziwo.info>
Should close a bunch of opened PR by @dependabot about
opentelemetry
. They all need to be bump together so Dependabot is never going to be completely helpful on this:⚠️ New version ofopentelemetry-datadog
is not yet released to supportopentelemetry:0.23
, this is the last one to be released before we can merge this PR. It's probably not happening before [Bug]: OpenTelemetry datadog exporter build is failing open-telemetry/opentelemetry-rust-contrib#67 is fixed. The PR Prepare for release (most) contrib crates. open-telemetry/opentelemetry-rust-contrib#81 seems to have fix the tests, maybe the release is not far away.opentelemetry-datadog
has been released.