Skip to content

chore(config): migrate Renovate config - #6832

Open
tanguille-cluster[bot] wants to merge 2 commits into
mainfrom
renovate/migrate-config
Open

tanguille-cluster[bot] wants to merge 2 commits into
mainfrom
renovate/migrate-config

Conversation

@tanguille-cluster

Copy link
Copy Markdown
Contributor

The Renovate config in this repository needs migrating. Typically this is because one or more configuration options you are using have been renamed.

You don't need to merge this PR right away, because Renovate will continue to migrate these fields internally each time it runs. But later some of these fields may be fully deprecated and the migrations removed. So it's a good idea to merge this migration PR soon.

PLEASE NOTE: JSON5 config file migrated! All comments & trailing commas were removed.

🔕 Ignore: Close this PR and you won't be reminded about config migration again, but one day your current config may no longer be valid.

❓ Got questions? Does something look wrong to you? Please don't hesitate to request help here.


This PR has been generated by Mend Renovate CLI.

@tanguille-cluster tanguille-cluster Bot mentioned this pull request Nov 12, 2026
5 tasks
@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 95004380-e47f-4789-b977-af231023b016

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@tanguille-cluster

tanguille-cluster Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor Author

Note

No rendered changes.

konflate 0.6.6 · rendered e70f2b1 · full diff →

@tanguille-cluster

tanguille-cluster Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor Author

Re-runViewKritika Review

Not approved: the approval could not be posted

Findings

Summary

Migrates Renovate's dashboard footer into the nested dependencyDashboard object

  • Renovate now reads the footer from dependencyDashboard.footer; the preset's enabled: true still merges in, so the dashboard and its manual-job checkbox stay as they were.
  • Every other setting keeps its previous value, so the next scheduled run of the RenovateJob (every 30 min) behaves the same.
  • The file is plain JSON now but keeps the .json5 name, so the oxfmt pre-commit job still formats it.
Review context · 6 sources · 0/3 skills · 3/6 commands · 1 overlapping pull request

Reviews (1) · Last reviewed commit: "style: format" · omniroute

Comment thread .renovaterc.json5
"github>home-operations/renovate-presets#11.0.0",
"github>home-operations/renovate-presets//apps/cnpg.json5#11.0.0",
"github>home-operations/renovate-presets//apps/grafanaDashboards.json5#11.0.0",
"github>home-operations/renovate-presets//apps/searxng.json5#11.0.0"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

P2 Restore the rationale comments the migration stripped · maintainability

The migration rewrote the file without comments, which is the documented side effect of the JSON5 rewrite, but the comments were the only record of several constraints a maintainer acts on:

  • platformAutomerge: false — "main has no required checks, so GitHub native auto-merge would merge before CI finishes; with this off Renovate merges itself only once every check is green (needs Checks: read)". Flipping that flag without the note merges PRs before checks finish.
  • Why the github> presets are all tag-pinned, and why the talosFactory preset is deliberately omitted (nodes pin ghcr.io/tanguille/installer/*, tuppr gates on the siderolabs git tag).
  • The kernel.org feed check (curl -s https://www.kernel.org/releases.json | jq '.releases[]|select(.moniker=="stable")') and the warning that a feed shape change yields zero releases so bumps stop silently — that command exists nowhere else in the repository.
  • The removal condition for the inline-OCI custom manager: "Drop once Renovate's flux manager extracts chart.kind: OCIRepository (renovatebot/renovate#47438)". Without it the workaround becomes permanent.
  • Why timestamp-optional, internalChecksFilter: flexible and the logLevelRemap entry are set, and that csi.<sidecar> keys miss the flux manager's values heuristic.

Nothing else in the repository restates these; the file's comments are the whole record.

Suggested fix

Re-add the removed comment lines in a follow-up commit on this branch, e.g.:

  • Before the github> presets: // Pin every github> preset to the same tag; an unpinned one tracks main. and, after the last one, // talosFactory preset omitted: nodes pin ghcr.io/tanguille/installer/* and tuppr gates on the siderolabs git tag.
  • Above platformAutomerge: false: the two-line main has no required checks … note.
  • Above "linux-kernel": the moniker filter note plus the curl … | jq check and the "bumps stop SILENTLY" warning.
  • Above the "Inline OCI chart sources in HelmReleases" manager's description: // Drop once Renovate's flux manager extracts chart.kind: OCIRepository (renovatebot/renovate#47438).
  • The remaining rationale comments from the base revision of the file (helm-values ResourceSet note, rebaseWhen, automergeType, timestamp-optional, internalChecksFilter, logLevelRemap, and the per-manager/per-rule notes).

Recover them with git show <base>:.renovaterc.json5` and re-apply as comments above the options they explain.

Prompt for a coding agent
In .renovaterc.json5 at the head of this PR, the Renovate config migration deleted every comment. Restore them with `git show HEAD~1:.renovaterc.json5` as the source, re-adding each `//` comment directly above (or, for the extends/unpinned note, inside) the option it explained, preserving the surrounding JSON5 formatting and 2-space indentation. Do not change any option value. At minimum restore: the `// Pin every github> preset to the same tag; an unpinned one tracks main.` comment before the `github>home-operations/renovate-presets#11.0.0` entry and the `// talosFactory preset omitted: ...` comment after the last preset entry in `extends`; the two-line `// main has no required checks, so GitHub native auto-merge would merge before CI finishes;` / `// with this off Renovate merges itself only once every check is green (needs Checks: read).` comment above `platformAutomerge: false`; the `// Moniker filter survives series bumps; a git mirror would offer -rc tags.` and `// A feed shape change yields zero releases and bumps stop SILENTLY; check: curl -s https://www.kernel.org/releases.json | jq '.releases[]|select(.moniker=="stable")'` comments above `"linux-kernel": {`; and `// Drop once Renovate's flux manager extracts chart.kind: OCIRepository (renovatebot/renovate#47438)` above the `description: "Inline OCI chart sources in HelmReleases"` line. Also restore the notes on `rebaseWhen`, `automergeType`, `minimumReleaseAgeBehaviour`, `internalChecksFilter`, `logLevelRemap`, the `helm-values` ResourceSet override, and the per-manager and per-package-rule rationale comments from the base revision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants