kubernetes/minikube

move viper calls from packages into cmd to improve testability/resusablitily

Open

#21.670 aperta il 30 set 2025

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)Go (5222 fork)batch import
help wantedkind/cleanuppriority/important-soon

Metriche repository

Star
 (31.799 star)
Metriche merge PR
 (Merge medio 12g 19h) (43 PR mergiate in 30 g)

Descrizione

packages inside minikube should not be calling viper to get a config, the viper (getting confg flags) should only be called form Cmd packages

this will help better testing and also avoid race conditions in Setting or Getting Flags in Parallel testing and Parallel usage of minikube and make minikube code more reusable and more testable

Originally posted by @medyagh in https://github.com/kubernetes/minikube/pull/21635#discussion_r2392461773

Guida contributor

Direzione di ricerca
Identifica tutte le funzioni nei pacchetti non cmd che chiamano viper.Get* o viper.Set* e spostale nei pacchetti cmd, passando la configurazione come parametro.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGoViper
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70