Merge pull request #146 from jk-cluster/feat/renovate-nginx-stable-only
renovate: keep the delegation nginx on stable branches
This commit is contained in:
2 files changed
+23
No files matched your search
@@ -70,6 +70,8 @@ What Renovate keeps current:
|
||||
|
||||
Each chart renders its main image as `<repo>:{{ .Chart.AppVersion }}` — repository in the template, version in `Chart.yaml`. Renovate cannot join two files, so the mapping lives in `renovate.json5`, one block per chart. It is deliberately *not* a `# renovate:` comment inside each `Chart.yaml`: that would touch all chart directories and force a version bump plus a publish per chart for a pure comment change. Charts whose tags carry a flavour or date suffix (`apache-tika`, `minecraft-server`, `rss-bridge`, `teamspeak-server`) override `versioning` in their block; the reasoning sits next to each one.
|
||||
|
||||
One dependency is constrained beyond its versioning: **nginx** (the Synapse delegation sidecar) is limited to **even minors** via `allowedVersions`. nginx numbers its release branches by minor — even is stable, odd is mainline — and the chart belongs on stable. Patch updates inside the stable branch still flow through normally. The constraint tolerates a variant suffix (`1.30.4-alpine`) on purpose: docker versioning only ever offers candidates carrying the current tag's suffix, so a suffix-blind constraint would silently freeze nginx the day the sidecar moves to a slimmer base image (#84) — no error, no warning.
|
||||
|
||||
Not managed: `ci/**` (install-test fixtures pin throwaway infrastructure to major lines on purpose) and `template-chart/**` (placeholder image). A consequence worth knowing: `ci/matrix-synapse/fixtures.yaml` pins a Synapse image that Renovate will not follow — bump it by hand when the chart moves a major.
|
||||
|
||||
### The version-bump hook
|
||||
|
||||
@@ -97,6 +97,27 @@
|
||||
matchDepTypes: ["app-version"],
|
||||
addLabels: ["app-version"],
|
||||
},
|
||||
{
|
||||
description: [
|
||||
"nginx numbers its release branches by minor: even is the stable branch, odd is",
|
||||
"mainline. The Synapse delegation nginx belongs on stable, so only even minors are",
|
||||
"allowed — that is what PR #139 (1.30.4 -> 1.31.4) would otherwise have done,",
|
||||
"quietly moving the chart onto mainline. Patch updates inside the stable branch",
|
||||
"(1.30.4 -> 1.30.5) still come through and still automerge.",
|
||||
],
|
||||
matchDatasources: ["docker"],
|
||||
matchPackageNames: ["nginx", "library/nginx", "docker.io/library/nginx"],
|
||||
// Even minor: any minor whose last digit is even, so 1.30.x passes and
|
||||
// 1.31.x does not. The trailing (-.+)? is load-bearing: #84 proposes
|
||||
// moving this sidecar to a slimmer base image, and with a variant tag
|
||||
// like 1.30.4-alpine docker versioning only ever offers candidates
|
||||
// carrying the same suffix. Without the optional suffix, every one of
|
||||
// them would fail this filter and nginx would silently stop receiving
|
||||
// updates - no error, no warning, green dashboard. Verified: with the
|
||||
// anchored form and currentValue 1.28.0-alpine, the lookup returns
|
||||
// "updates": [] and "warnings": [].
|
||||
allowedVersions: "/^\\d+\\.\\d*[02468]\\.\\d+(-.+)?$/",
|
||||
},
|
||||
],
|
||||
|
||||
customManagers: [
|
||||
|
||||
Reference in new issue
Block a user