Skip to content

Exclude OpenTelemetry updates from the grouped weekly Renovate PR - #8727

Closed
trask wants to merge 1 commit into
mainfrom
trask-renovate-exclude-otel
Closed

Exclude OpenTelemetry updates from the grouped weekly Renovate PR#8727
trask wants to merge 1 commit into
mainfrom
trask-renovate-exclude-otel

Conversation

@trask

@trask trask commented Aug 17, 2026

Copy link
Copy Markdown
Member

OpenTelemetry dependency updates now always get their own Renovate PR instead of being swallowed into the grouped weekly all patch and minor versions PR. These updates are used for dogfooding and for cascaded releases across the OpenTelemetry Java repos, so batching them until Tuesday delays the release chain.

The affected artifacts are io.opentelemetry.contrib:opentelemetry-aws-xray-propagator, io.opentelemetry.proto:opentelemetry-proto, and io.opentelemetry.semconv:opentelemetry-semconv-incubating.

The exclusion is written as !io.opentelemetry** without a colon, so it covers every OpenTelemetry group id rather than only the bare io.opentelemetry group. !open-telemetry/** covers dependencies sourced from GitHub.

OpenTelemetry-published Docker images such as otel/opentelemetry-collector-contrib are deliberately still grouped, because they are test infrastructure rather than part of a cascaded release.

GitHub Actions and Dockerfile updates are unaffected. The later weekly update rule matches by manager and still overrides the grouping for them, including open-telemetry/shared-workflows action bumps.

The comment on the rule also said "group all patch updates" even though the rule has covered minor updates too; that is corrected here.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Renovate grouping configuration so OpenTelemetry-related dependency updates don’t get bundled into the weekly “all patch and minor versions” group PR, allowing dogfooding and cascaded releases to land promptly.

Changes:

  • Clarifies the weekly grouping rule comment to reflect that it applies to both patch and minor updates.
  • Adds exclusions intended to keep OpenTelemetry Maven artifacts and GitHub-sourced OpenTelemetry dependencies out of the weekly grouped PR.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/renovate.json5
Comment on lines +27 to +30
matchPackageNames: [
'!io.opentelemetry**',
'!open-telemetry/**',
],
@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.27%. Comparing base (f4bb945) to head (1e70f6d).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8727   +/-   ##
=========================================
  Coverage     91.26%   91.27%           
- Complexity    10461    10462    +1     
=========================================
  Files          1006     1006           
  Lines         28244    28244           
  Branches       3564     3564           
=========================================
+ Hits          25778    25779    +1     
  Misses         1674     1674           
+ Partials        792      791    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@trask

trask commented Aug 17, 2026

Copy link
Copy Markdown
Member Author

Reopened from a fork branch as #8728, which has the same commit and the same diff. Closing this one.

@trask trask closed this Aug 17, 2026
@trask
trask deleted the trask-renovate-exclude-otel branch August 17, 2026 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants