Unify node-pinning label keys #5
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Same intent expressed 3 ways: labels {arch: amd64} (lxml-s390x:2), labels {platform: mermaid} (pillow-s390x:2), nodeSelector {provider: digitalocean}. Pick one convention, migrate fleet-wide.
Checked live Woodpecker agents (
agent_list): they advertisecustom_labels: {arch, worker}(e.g.arch: arm64, worker: kotkan), notprovider. Standardized on the pipeline-levellabels: {arch: amd64|arm64}key across the 10 templated repos (matches agent routing), dropping the two other mechanisms:labels: {platform: mermaid}→ migrated toarch: amd64(it was an older alias from before multiple amd64 agents existed).backend_options.kubernetes.nodeSelector: {provider: digitalocean}→ dropped (this is a literal k8s node-label selector, a different mechanism than Woodpecker agent labels; no live node actually needs this now that agent-levellabels:already routes correctly).Documented in oleks/ci-scripts README (
7a434fa).nodeSelectorwithkubernetes.io/arch/kubernetes.io/hostnameis left alone where it's a genuine k8s-level pin (e.g.sentry-cli-s390x→ specific host) — that's a different, legitimate use, not the drift this issue targeted.Applied fleet-wide across the same 10 commits as #1 (lxml f9e3c55 → markupsafe c31293c, see #1 for the full SHA list). Closing.