kubernetes/minikube

rename "none" driver to "native"

Open

#7772 aperta il 18 apr 2020

Vedi su GitHub
 (19 commenti) (1 reazione) (0 assegnatari)Go (5222 fork)batch import
co/none-driverhelp wantedkind/featurelifecycle/frozenpriority/backlog

Metriche repository

Star
 (31.799 star)
Metriche merge PR
 (Merge medio 18g 3h) (62 PR mergiate in 30 g)

Descrizione

none is a terrible name. and doesnt look good at least in integration tests.I suggest we add an alias to none driver.

we could do it in two phases:

change all the current integration tests names to BareMetal in second phase, if we add an alias, so both none or barmetal be accepted.

Guida contributor

Direzione di ricerca
Rivedi l'implementazione corrente del driver 'none', probabilmente in pkg/minikube/drivers/none. Nota il commento di f1sherman sull'approccio dei test basati su tabelle in pkg/minikube/cluster/cluster test.go. Inoltre, controlla i test di integrazione menzionati (test/integration/). Il problema propone due fasi: prima, rinomina i riferimenti esistenti a 'BareMetal' nei test, poi aggiungi un alias in modo che siano accettati sia 'none' che 'native' (o 'baremetal'). I manutentori sembrano aperti a questo ma hanno bisogno di un piano concreto. Indaga se ci sono file di configurazione o costanti che fanno riferimento al nome del driver.
Tech stack
go
Dominio
cli
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70