Skip to content

cI: Add more renovate rules based on having done a scan. - #641

Open
thompson-tomo wants to merge 2 commits into
open-telemetry:mainfrom
thompson-tomo:Larger_renovate_usage
Open

cI: Add more renovate rules based on having done a scan.#641
thompson-tomo wants to merge 2 commits into
open-telemetry:mainfrom
thompson-tomo:Larger_renovate_usage

Conversation

@thompson-tomo

@thompson-tomo thompson-tomo commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This analyses the renovate scan to implement rules to help mantain the dependencies going forward.

A balance between grouping and individual pr's will be raised to minimise updates blocked due to failing ci.

Each rule has a description of what it does.

This is a Continuation of #640 which enabled the scanning.

@thompson-tomo
thompson-tomo requested a review from a team as a code owner August 3, 2026 03:46
@lalitb

lalitb commented Aug 3, 2026

Copy link
Copy Markdown
Member

Thanks for working on this. I think Renovate is useful for this repo, but this change enables too much at once.

Can we keep dashboard approval as the default and enable each dependency type separately after reviewing what Renovate finds?

I also suggest keeping digest updates separate. Combining Docker, GitHub Actions, Bazel, and library updates in one PR could make CI failures difficult to understand and may block unrelated updates.

A safer approach would be:

  • Keep automatic updates for GitHub Actions.
  • Keep other updates behind dashboard approval for now.
  • Add separate rules for Docker and library dependencies.
  • Group only closely related dependencies.

This will give us the benefits of Renovate without creating large mixed update PRs.

@thompson-tomo

Copy link
Copy Markdown
Contributor Author

Can we keep dashboard approval as the default and enable each dependency type separately after reviewing what Renovate finds?

Thats exactly what i did, renovate didn't find as much as i was hoping for but thats due to using apt packages, git repo's.

I also suggest keeping digest updates separate. Combining Docker, GitHub Actions, Bazel, and library updates in one PR could make CI failures difficult to understand and may block unrelated updates.

Only patches/digest are combined everything else is seperate. If a patch breaks ci, then something is wrong as that shouldn't be the case.

Keep other updates behind dashboard approval for now.

Have just put more behind approval.

Add separate rules for Docker and library dependencies.

Already the case except for patches.

Group only closely related dependencies.

Most updates are already seperate, key thing i like to avoid is having a too noisy setup especially when review bandwidth is low. Shortly after the config pr is merged the issue will show you its update plan.

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