kubernetes/minikube

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

Open

Aperta il 30 set 2025

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

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

Tech stack
go
Dominio
backend
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
active
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
GitGoViper
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
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.