tektoncd/pipeline

Disallow unknown keys in feature-flags configmap

Aberta

#6.938 aberto em 18 de jul. de 2023

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)Go (1.943 forks)auto 404
Hacktoberfesthelp wanted

Métricas do repositório

Stars
 (9.013 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue a validação existente para flags de funcionalidade no PR #6935 e, em seguida, implemente uma verificação para rejeitar chaves desconhecidas no configmap. Esteja ciente da dependência da issue #4542 para lidar com atualizações de forma elegante.
Pilha de tecnologia
go
Domain
backenddevops
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GoKubernetes
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60