tektoncd/pipeline

Disallow unknown keys in feature-flags configmap

Aperta

#6938 aperta il 18 lug 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Go (1943 fork)auto 404
Hacktoberfesthelp wanted

Metriche repository

Star
 (9013 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Currently, we don't perform any validation of the feature flags configmap. https://github.com/tektoncd/pipeline/pull/6935 adds validation for keys representing existing feature flags, but it's still possible to misspell a feature flag or add a flag that's no longer supported, thinking it has an effect (example: https://github.com/tektoncd/pipeline/pull/6937). We should validate that the configmap doesn't contain any unknown keys. We will probably need to fix https://github.com/tektoncd/pipeline/issues/4542 first, otherwise upgrades will cause problems if we remove feature flags.

Guida contributor

Direzione di ricerca
Studia la validazione esistente per i flag di funzionalità nella PR #6935, quindi implementa un controllo per rifiutare chiavi sconosciute nella configmap. Fai attenzione alla dipendenza dall'issue #4542 per gestire correttamente gli aggiornamenti.
Tech stack
go
Dominio
backenddevops
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Bloccata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GoKubernetes
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60