feat(upgrade): warn when node body install.image differs from resolved target image
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 54/100
Direzione di ricerca
Start by tracing the talm upgrade -f flow where the target image is resolved from values.yaml and the existing Using image from values.yaml message is printed. Find how node bodies are read and how post-upgrade write-back is performed; add coverage for matching and differing machine.install.image values. Done when a best-effort warning names both refs and explains the body will be patched after a successful upgrade, without blocking the upgrade.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Since the upgrade target image resolves from values.yaml::image (#176), the node body's machine.install.image is no longer consulted. When the two diverge — typical in projects initialized with an older preset where values.yaml was never bumped but node files were edited or re-templated — talm upgrade -f nodes/<name>.yaml prints only Using image from values.yaml: <ref> and proceeds. There is no signal that the file's image was ignored, nor that the file will be rewritten by the post-upgrade write-back to match the applied image.
This surprised a real operator: their values.yaml carried an old preset default (talos:v1.10.5) while the node file pointed at a current image; the printed line was the only hint that the wrong knob was about to win.
Proposal
When -f is given and --image is unset, after resolving the target from values.yaml, parse machine.install.image from each -f node body (best-effort, plain YAML read, no templating). If any body's image differs from the resolved target, print a warning naming both refs and stating that the body will be patched to the target after a successful upgrade. A warning, not a blocker — drift here is expected mid-upgrade.
- Lingua principale
- Go
- Stelle
- 493
- Fork
- 23
- Merge medio
- 4g 8h
- PR unite (30g)
- 4
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di cozystack/talm
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
TestCommittedTextFilesIgnoresUntrackedArtefacts commits into the real repo when GIT_DIR is setAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
-
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 70/100
-
area/apply area/commands area/upgrade kind/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Tutte le issue di cozystack/talm
Issue simili
-
[doc-healer] DDUw improvement: retry integrity-filtered issue reads with direct single-item lookupsApertaautomation documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
bug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
containerd/containerd#14476 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
Infisical/infisical#8987 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
scan framework and scan control accept empty and whitespace-only policy names in Args validationForse già presa @WKr1shna l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno