helm/helm

helm lint misses bad chart names

Aperta

#10.537 aperta il 10 gen 2022

Vedi su GitHubGuida contributor
 (21 commenti) (0 reazioni) (0 assegnatari)Go (7602 fork)batch import
featuregood first issuekeep open

Metriche repository

Star
 (29.815 stelle)
Metriche merge PR
 (Merge medio 40g 17h) (48 PR mergiate in 30 g)

Descrizione

helm lint only catches if a chart name is missing. You can see that in the following snippet...

https://github.com/helm/helm/blob/292368a64d6c9794b4573afa5a8dc6e940a21045/pkg/lint/rules/chartfile.go#L106-L111

The chartutil package has the ability to validate names in some forms. We could provide better hints (like no capital letters) via lint.

Output of helm version: 3.7

Output of kubectl version: N/A

Cloud Provider/Platform (AKS, GKE, Minikube etc.): N/A

Guida contributor

Direzione di ricerca
Ispeziona la convalida esistente del nome del grafico in pkg/lint/rules/chartfile.go e la convalida del nome in chartutil, quindi aggiungi una regola lint per avvisare su nomi di grafici non validi (es. lettere maiuscole).
Tech stack
go
Dominio
backendcli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitGo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60