Skip to content

chore: Add renovate configuration - #4568

Merged
marcalff merged 4 commits into
open-telemetry:mainfrom
thompson-tomo:patch-1
Sep 17, 2026
Merged

marcalff merged 4 commits into
open-telemetry:mainfrom
thompson-tomo:patch-1

Conversation

@thompson-tomo

@thompson-tomo thompson-tomo commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Fixes # (issue)

Changes

Adds in a renovate config to mirror setup in contrib. Only difference in setups is enablement of docker updates ie devContainers.

the goal will be to eventually automate cmake fetchContent updates etc.

@thompson-tomo
thompson-tomo requested a review from a team as a code owner September 14, 2026 09:30
@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.51%. Comparing base (a3f652f) to head (9e759e2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4568   +/-   ##
=======================================
  Coverage   86.51%   86.51%           
=======================================
  Files         525      525           
  Lines       20464    20464           
=======================================
  Hits        17702    17702           
  Misses       2762     2762           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lalitb lalitb 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.

Thanks for adding this. Are we planning to replace Dependabot with Renovate, or use Renovate only for dependencies Dependabot doesn't cover? It would be good to agree on that first, so we avoid duplicate update PRs and keep our existing seven-day cooldown.

@thompson-tomo

thompson-tomo commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

The intention is to mirror what all languages do (except rust) which is use renovate where possible & dependabot for security updates.

Is The cooldown needed given it was only added a few weeks ago by zizmor & zizmor doesn't suggest adding it to renovate?

See prior discussion open-telemetry/opentelemetry-cpp-contrib#640

@lalitb

lalitb commented Sep 16, 2026

Copy link
Copy Markdown
Member

The intention is to mirror what all languages do (except rust) which is use renovate where possible & dependabot for security updates.

Is The cooldown needed given it was only added a few weeks ago by zizmor & zizmor doesn't suggest adding it to renovate?

See prior discussion open-telemetry/opentelemetry-cpp-contrib#640

Thanks, that makes sense. I'm happy with that split. Could we disable Dependabot's regular updates where Renovate takes over? We can discuss the cooldown separately.

@thompson-tomo

thompson-tomo commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Yes, i usually like to do that once the renovate setup is confirmed end to end ie pr's raised to give confidence nothing is being missed.

Especially with the dependabot cooldown there is no issue running them side by side.

@lalitb

lalitb commented Sep 16, 2026

Copy link
Copy Markdown
Member

Yes, i usually like to do that once the renovate setup is confirmed end to end ie pr's raised to give confidence nothing is being missed.

Especially with the dependabot cooldown there is no issue running them side by side.

Sounds good, thanks for clarifying. We can keep both temporarily while we confirm Renovate is working, then disable Dependabot's regular updates while keeping security updates enabled.

@thompson-tomo

thompson-tomo commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

Correct & i will help monitor the rollout of it etc. Note i also have the pr for the build tools open-telemetry/cpp-build-tools#36

@marcalff marcalff 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.

LGTM

@marcalff
marcalff merged commit 0d4f673 into open-telemetry:main Sep 17, 2026
76 of 77 checks passed
@thompson-tomo
thompson-tomo deleted the patch-1 branch September 18, 2026 02:43
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.

3 participants