kubernetes/minikube

rename "none" driver to "native"

Ouverte

#7 772 ouverte le 18 avr. 2020

Voir sur GitHubGuide contributeur
 (19 commentaires) (1 réaction) (0 personne assignée)Go (5 222 forks)batch import
co/none-driverhelp wantedkind/featurelifecycle/frozenpriority/backlog

Métriques du dépôt

Stars
 (31 799 étoiles)
Métriques de merge PR
 (Merge moyen 12j 19h) (43 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle du pilote 'none', probablement dans pkg/minikube/drivers/none. Notez le commentaire de f1sherman concernant l'approche de test pilotée par table dans pkg/minikube/cluster/cluster test.go. Vérifiez également les tests d'intégration référencés (test/integration/). Le problème propose deux phases : d'abord, renommer les références existantes à 'BareMetal' dans les tests, puis ajouter un alias pour que 'none' et 'native' (ou 'baremetal') soient tous deux acceptés. Les mainteneurs semblent ouverts à cela mais ont besoin d'un plan concret. Recherchez s'il existe des fichiers de configuration ou des constantes qui font référence au nom du pilote.
Stack technique
go
Domaine
cli
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70