Skip to content

🔧 QD-15046 Switch dependency updates from Dependabot to Renovate-only - #979

Draft
Anna Zhukova (bindreams) wants to merge 1 commit into
mainfrom
azhukova/QD-15046
Draft

🔧 QD-15046 Switch dependency updates from Dependabot to Renovate-only#979
Anna Zhukova (bindreams) wants to merge 1 commit into
mainfrom
azhukova/QD-15046

Conversation

@bindreams

Copy link
Copy Markdown
Collaborator

What

Make Renovate the sole dependency-update tool. Removes Dependabot and rewrites the Renovate config in JSON5, taking over Dependabot's scope and adding the requested behaviors.

Ticket: QD-15046

Changes

  • Delete .github/dependabot.yml.
  • Replace .github/renovate.json.github/renovate.json5 (modernized deprecated keys: config:baseconfig:recommended, regexManagerscustomManagers, fileMatchmanagerFilePatterns, datasourcesmatchDatasources, packagePatternsmatchPackageNames, pathsmatchFileNames).
  • Add .github/workflows/renovate-validate.yml — runs renovate-config-validator --strict on config changes (there was no CI validation before).
  • Update a stale "dependabot" comment in internal/tooling/scripts/pom.xml.

Behaviors

Requirement How
Auto-merge everything automerge + platformAutomerge (semi-auto: still gated by main's 1 required review + Test checks)
Group minor / individual major per-ecosystem minor+patch+digest groups; majors stay individual
Ticket prefix from Dependabot :arrow_up: QD-12983 (gomod/actions/maven), bare QD-8148 (docker/custom)
7-day supply-chain cooldown minimumReleaseAge: "7 days" + internalChecksFilter: "strict"; security updates bypass it
DHI registry auth hostRules for dhi.io (hostType: docker) using dashboard secrets DHI_USERNAME/DHI_PASSWORD
Renovate owns security osvVulnerabilityAlerts + vulnerabilityAlerts; Dependabot security updates are already disabled at repo level and alerts stay on

Decisions made autonomously — please confirm

  1. Dropped Dependabot's gomod *docker* exclusion. It existed to avoid overlap with Renovate, which already updates those modules (e.g. the recent security PR QD-8148 Update module github.com/docker/cli to v29 [SECURITY] #972). In a Renovate-only world they should be covered. Docker Go SDK bumps will now group + auto-merge on green CI. Re-add a packageRule if you'd rather keep them manual.
  2. Scoped Renovate to match Dependabot via ignorePaths (**/testdata/**, .teamcity/**) so it never starts bumping test fixtures (poms, Dockerfiles, compose) or the TeamCity DSL pom — verified excluded.
  3. Included .devcontainer/Dockerfile (real file, uses dhi.io) — a small superset of Dependabot's hand-picked dirs.
  4. Kept the dormant repology/docker-datasource rules from the old config (no matching deps today). Say the word to drop them.
  5. Major updates auto-merge too (on green) per "tests are the arbiter".

Coordination

dhi.io hostRules are also added by the in-flight QD-14979 (47e82a1e, same secret names) on the old renovate.json. This PR puts the identical rules in renovate.json5 and deletes renovate.json, so whichever merges second needs a trivial rebase. QD-14979 additionally pins the dhi.io/golang digest — that part is untouched here.

Verification

  • renovate-config-validator --strict passes (repo + global modes).
  • Local renovate --platform=local --dry-run=extract confirms: custom managers match all 34 Dockerfiles + the JBR pin; maven scoped to only internal/tooling/scripts/pom.xml; no testdata/.teamcity fixtures extracted; DHI secret interpolation succeeds with secrets present.
  • Note: the validator version is pinned (renovate@43.227.0) for reproducible CI; bump manually (or add a self-update annotation) as desired.

@github-actions

Copy link
Copy Markdown

Qodana for Go

114 new problems were found

Inspection name Severity Problems
Check dependency licenses 🔴 Failure 5
Vulnerable declared dependency 🔴 Failure 1
Check GO source code coverage 🔶 Warning 51
Unhandled error 🔶 Warning 13
Potential resource leak 🔶 Warning 6
Check dependency licenses 🔶 Warning 4
Deprecated element 🔶 Warning 4
Imported package name as a name identifier 🔶 Warning 1
Reserved word used as name 🔶 Warning 1
Vulnerable declared dependency 🔶 Warning 1
Unsorted imports ◽️ Notice 13
Name starts with a package name ◽️ Notice 8
Empty slice declared using a literal ◽️ Notice 3
Comment of exported element starts with the incorrect name ◽️ Notice 1
Redundant type conversion ◽️ Notice 1
Vulnerable declared dependency ◽️ Notice 1
@@ Code coverage @@
+ 65% total lines covered
10664 lines analyzed, 6959 lines covered
# Calculated according to the filters of your coverage tool

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

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.

1 participant