GoogleContainerTools/skaffold

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

Offen

#4.268 geöffnet am 27.05.2020

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Go (1.416 Forks)batch import
area/configfixithelp wantedkind/feature-requestpriority/p3

Repository-Metriken

Stars
 (12.822 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 6h) (16 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung des `dev` Befehls und wie er mit dem `cleanup` Flag umgeht. Fügen Sie dann ein `cleanup` Feld zur Skaffold Pipeline Konfiguration hinzu (wahrscheinlich in `pkg/skaffold/config/config.go` oder ähnlich) und integrieren Sie es mit der Flag Logik, sodass der YAML Wert verwendet wird, es sei denn, das Flag wird explizit gesetzt.
Tech Stack
go
Domain
cli
Issue Type
Funktion
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
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75