Skip to content

chore: Make renovate behave more like dependabot#2364

Merged
wpessers merged 5 commits into
open-telemetry:mainfrom
thompson-tomo:renovate_fixes
Jun 18, 2026
Merged

chore: Make renovate behave more like dependabot#2364
wpessers merged 5 commits into
open-telemetry:mainfrom
thompson-tomo:renovate_fixes

Conversation

@thompson-tomo

@thompson-tomo thompson-tomo commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

This has been tested to resolve the following issues:

  • incorrect grouping of otel dependencies - resolved by changing order & match pattern
  • Not updating tests directory - adjustment to the ignore paths

Partially test to resolve

  • incorrect pr titles - review of logs showed title changing due to adding scope to label rule but not all pr combinations tested

this also seperates major updates from other groups & adds approval to avoid accidentally merging major bump as well as enable to delay them till after minor etc. We can certainly change this rule.

@thompson-tomo thompson-tomo requested a review from a team as a code owner June 13, 2026 04:37
Comment thread .github/renovate.json5 Outdated
},
{
groupName: 'collector-other',
updateTypes: [

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When I look for this config, I only find matchUpdateTypes here: https://docs.renovatebot.com/configuration-options/#packagerulesmatchupdatetypes

The intended change here is that we will have to manually approve renovate to open the pr for major version bumps, correct? That's not how it is today with Dependabot, but I'm down to try that. Will have to see how it goes for some weeks and then we can decide whether that config works for us or not!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for spotting that, been fixed.

Correct, let's see how approvals for major go given there shouldn't be many.

@wpessers wpessers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@thompson-tomo found some remaining ones, after this LGTM!

Comment thread .github/renovate.json5 Outdated
Comment thread .github/renovate.json5 Outdated
Comment thread .github/renovate.json5 Outdated
Comment thread .github/renovate.json5 Outdated
Co-authored-by: Warre Pessers <45464718+wpessers@users.noreply.github.com>
@thompson-tomo

Copy link
Copy Markdown
Contributor Author

Thanks @wpessers those suggestions have been committed.

@wpessers wpessers merged commit b83fb7e into open-telemetry:main Jun 18, 2026
11 checks passed
@wpessers wpessers added enhancement New feature or request dependencies Pull requests that update a dependency file labels Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants