kubernetes-sigs/cluster-api

ClusterClass rebase should check variable compatibility

Fechada

#8.154 aberto em 22 de fev. de 2023

Ver no GitHubGuia do colaborador
 (5 comentários) (0 reação) (0 responsável)Go (1.532 forks)auto 404
area/clusterclasshelp wantedkind/featurepriority/important-longtermtriage/accepted

Métricas do repositório

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

Description

PR #8153 removes the variable compatibility checks for Clusters using a ClusterClass when that class is being updated in place. This change is a consequence of external variables.

During ClusterClass rebase - moving a Cluster from one ClusterClass to another - similar variable checks could be done allowing users to safely update from one set of variables to another while changing ClusterClass. Currently validation of those variables is not done during ClusterClass rebase.

This validation should be added to give rebase webhook based validation of variable changes.

/area topology /kind feature

Guia do colaborador

Direção de pesquisa
Revise as verificações de compatibilidade de variáveis em internal/topology/check/compatibility.go, especialmente a função na linha 200. Entenda a diferença entre atualização de ClusterClass e rebase. Projete uma validação para rebase que verifique a compatibilidade das variáveis.
Pilha de tecnologia
go
Domain
backendapi
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.
2
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55