kubernetes/minikube

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

Offen

#21.670 geöffnet am 30.09.2025

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Go (5.222 Forks)batch import
help wantedkind/cleanuppriority/important-soon

Repository-Metriken

Stars
 (31.799 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 12T 19h) (43 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Identifiziere alle Funktionen in Nicht cmd Paketen, die viper.Get* oder viper.Set* aufrufen, und verschiebe sie in cmd Pakete, wobei die Konfiguration als Parameter übergeben wird.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitGoViper
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70