Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

cluster-autoscaler: no test pins the min-size boundary in the patched SetSize

Aperta Adatta ai principianti
#4,787 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go
Ambito
testing

Direzione di ricerca

Parti da packages/apps/kubernetes/images/cluster-autoscaler/fix-downscale.diff, che applica una patch a SetSize in cloudprovider/clusterapi di cluster-autoscaler-1.32.0 upstream. Leggi il TestNodeGroupSetSizeRefusesDecreaseBelowMinSize già incluso nella patch, poi aggiungi il caso limite: 4 repliche, min 3, SetSize(3) deve restituire nessun errore. Significa che i test del pacchetto falliscono se la condizione viene cambiata in <= e passano con l'< attuale, operazione che la build dell'immese esegue automaticamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

triage/needs-triage

The min-size guard in our cluster-autoscaler patch has no test for a decrease that lands exactly on the min size. packages/apps/kubernetes/images/cluster-autoscaler/fix-downscale.diff makes SetSize in the clusterapi provider refuse nreplicas < r.minSize when that is also below the current replicas (#4768). If the comparison is changed to <=, which would refuse a scale-down to exactly the min, the whole cloudprovider/clusterapi package still passes. That holds with the patch and without it, so the gap comes from the upstream tests at cluster-autoscaler-1.32.0.

The fix is one more case in TestNodeGroupSetSizeRefusesDecreaseBelowMinSize inside the patch: start at 4 replicas with min 3, call SetSize(3), and expect no error. The image build runs the tests of the packages the patch touches, so the case would be checked on every build.

Related, but a separate change: #4730 is about the max-size side of the same patch.

Lingua principale
Go
Stelle
2.2k
Fork
209
Merge medio
3g 12h
PR unite (30g)
230

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di cozystack/cozystack

Tutte le issue di cozystack/cozystack

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.