kubernetes/minikube

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

Open

#21.670 aberto em 30 de set. de 2025

Ver no GitHub
 (4 comments) (0 reactions) (1 assignee)Go (31.799 stars) (5.222 forks)batch import
help wantedkind/cleanuppriority/important-soon

Description

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

Guia do colaborador

Pilha de tecnologia
go
Domain
backenddeveloper experience
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
active
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Go programmingViper configuration libraryMinikube project structure
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
Examine the typical Viper configuration usage in Minikube's internal packages. Review the refactor example in PR #21635 and the discussion at https://github.com/kubernetes/minikube/pull/21635#discussion r2392461773. Identify all packages that directly call viper and plan to move those calls to the cmd layer. Ensure that configuration is passed as parameters to avoid global state.