Skip to content
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

Adjust Renovate Rules #464

Merged
merged 3 commits into from
May 8, 2024
Merged

Adjust Renovate Rules #464

merged 3 commits into from
May 8, 2024

Conversation

mallardduck
Copy link
Member

@mallardduck mallardduck commented May 7, 2024

This PR adds a few Rancher modules as global ignores across all branches. This will hopefully prevent the renovate PRs from causing as many breaking changes. Often when those modules are updated they need code generation to run too which is not currently part of Renovate tasks.

This PR attempts to address the way that the 3.x and 4.x branch must keep K8S support to match their rancher versions. Essentially when the auto bump affects these packages (which are not versioned) the result is bumping peers (k8s.io items) that these branches of BRO cannot increase without causing breaking changes.

Example Open PRs Causing issues:
#461
#462

And here's an example of what it takes to get these types of PRs mergable:
#458
#459

Copy link
Contributor

@alexandreLamarre alexandreLamarre left a comment

Choose a reason for hiding this comment

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

Looks like a better way to do this would be to use https://docs.renovatebot.com/configuration-options/#ignoredeps.

I think potentially we would want to ignore bumps to those in any branch of backup-restore-operator, since it likely requires manual work or aggressive updates that force support out of a given k8s range

An example from another big open source repo is:

https://github.com/open-telemetry/opentelemetry-collector/blob/c6b70a7ec794d1a296f82863032def0848f870e0/renovate.json#L69C1-L71C7

@mallardduck
Copy link
Member Author

@alexandreLamarre - thanks for the input here, that does seem like it will make more sense! Updated PR to use that.

@mallardduck mallardduck merged commit 04aa3e5 into release/v5.0 May 8, 2024
3 checks passed
@mallardduck mallardduck deleted the renovate-adjustments branch May 13, 2024 14:30
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