GoogleContainerTools/skaffold

Allow default dev cleanup mode to be set in skaffold.yaml

Ouverte

#4 268 ouverte le 27 mai 2020

Voir sur GitHubGuide contributeur
 (7 commentaires) (1 réaction) (0 personne assignée)Go (1 416 forks)batch import
area/configfixithelp wantedkind/feature-requestpriority/p3

Métriques du dépôt

Stars
 (12 822 étoiles)
Métriques de merge PR
 (Merge moyen 3j 6h) (16 PRs mergées en 30 j)

Description

Currently skaffold dev defaults to cleanup=true.

There are occasions when I'd like to modify this behavior for all users of a specific project profile. The simplest way to do that would be a cleanup setting in skaffold.yaml.

The CLI flag would still override the setting in skaffold.yaml.

After discussion in office hours, we determined this feature might be useful but could be confusing for some use cases. Open to discussion as always :-)

Original Question: https://groups.google.com/forum/#!topic/skaffold-users/rqzKmFI-qaM

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la commande `dev` et comment elle gère le drapeau `cleanup`. Ensuite, ajoutez un champ `cleanup` à la configuration du pipeline Skaffold (probablement dans `pkg/skaffold/config/config.go` ou similaire) et intégrez le à la logique du drapeau de sorte que la valeur YAML soit utilisée à moins que le drapeau ne soit explicitement défini.
Stack technique
go
Domaine
cli
Type d'issue
Fonctionnalité
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.
1-3 heures
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.
75