kubernetes-sigs/cluster-api

BeforeClusterUpgrade is not called with cp unavailable

Ouverte

#8 942 ouverte le 30 juin 2023

Voir sur GitHubGuide contributeur
 (14 commentaires) (1 réaction) (0 personne assignée)Go (1 532 forks)auto 404
help wantedkind/bugpriority/backlogtriage/accepted

Métriques du dépôt

Stars
 (4 267 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

What steps did you take and what happened?

Using ClusterClass we found an unexpected behavior when we try to upgrade the cluster and intercept the BeforeClusterUpgrade hook using RuntimeSDK. If a ControlPlane isn't in an healthy state the hook doesn't get propagated to the runtime extension, I think this is logically wrong and in contrast with the documentation.

What did you expect to happen?

To receive the BeforeClusterUpgrade request even if the ControlPlanes are unhealthy and manage the problem in the phase of upgrading.

Cluster API version

1.4.2+

Kubernetes version

1.27.2

Anything else you would like to add?

No response

Label(s) to be applied

/kind bug One or more /area label. See https://github.com/kubernetes-sigs/cluster-api/labels?q=area for the list of labels.

Guide contributeur

Direction de recherche
Examinez la logique du contrôleur de mise à niveau dans Cluster API qui décide s'il faut invoquer les hooks de cycle de vie. Recherchez les vérifications de l'état de santé du plan de contrôle avant d'appeler BeforeClusterUpgrade. Le problème pourrait se trouver dans la boucle de réconciliation du contrôleur de mise à niveau, où il vérifie l'état du plan de contrôle avant de procéder aux hooks.
Stack technique
go
Domaine
backendinfrastructure
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoKubernetesCluster API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30