Skip to content

General Renovate config improvements - #391

Merged
cert-manager-prow[bot] merged 3 commits into
cert-manager:mainfrom
erikgb:renovate-config-improve
Sep 2, 2025
Merged

General Renovate config improvements#391
cert-manager-prow[bot] merged 3 commits into
cert-manager:mainfrom
erikgb:renovate-config-improve

Conversation

@erikgb

@erikgb erikgb commented Sep 2, 2025

Copy link
Copy Markdown
Member

This Pr is trying to improve the Renovate configuration even more. It consists of a separate commit for each improvement:

  1. Remove redundant match all packages. The matchers inside packageRules only filter matching upgrades. As long as we have at least one match filter, Renovate is happy.
  2. Move postUpgradeTasks into packageRules to avoid running heavy workflow jobs when not required to, e.g., to upgrade GHA.
  3. DependencyDashboardApproval for major/digest upgrades in general. I hope this will work a little better than fix(deps): update cloud go deps (major) - abandoned cert-manager#8014 (example). The goal here is to be able to request one major upgrade PR at a time, which will make it easier to work with the upgrade, as major upgrades typically require manual work.

@cert-manager-prow cert-manager-prow Bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 2, 2025
@ThatsMrTalbot

Copy link
Copy Markdown
Contributor

/lgtm

@cert-manager-prow cert-manager-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Sep 2, 2025
@erikgb

erikgb commented Sep 2, 2025

Copy link
Copy Markdown
Member Author

/approve

@cert-manager-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: erikgb

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cert-manager-prow cert-manager-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 2, 2025
@cert-manager-prow
cert-manager-prow Bot merged commit b635d0b into cert-manager:main Sep 2, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants