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 (5.222 forks)batch import
help wantedkind/cleanuppriority/important-soon

Métricas do repositório

Stars
 (31.799 stars)
Métricas de merge de PR
 (Mesclagem média 12d 19h) (43 fundiu PRs em 30d)

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

Direção de pesquisa
Identifique todas as funções em pacotes não cmd que chamam viper.Get* ou viper.Set* e mova as para pacotes cmd, passando a configuração como parâmetro.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitGoViper
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70