kubernetes-client/python

leaderelection do not stop leading properly

Open

#2 075 ouverte le 12 juin 2023

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Python (3 323 forks)batch import
help wantedkind/bug

Métriques du dépôt

Stars
 (6 225 stars)
Métriques de merge PR
 (Merge moyen 10j 3h) (2 PRs mergées en 30 j)

Description

What happened (please include outputs or screenshots): After onstarted_leading function throw exception and finish, the process still leading.

What you expected to happen: The process stop leading immediatly.

How to reproduce it (as minimally and precisely as possible): Just follow the leaderelection/example.py.

Anything else we need to know?: Go client use ctx.Done() to find out the OnStartedLeading finish, and then release lock: https://github.com/kubernetes/client-go/blob/1309f64d6648411b4a36a2f7fa84dd8df31884b6/tools/leaderelection/leaderelection.go#L265-L291

Environment:

  • Kubernetes version (kubectl version): v1.20.15
  • OS (e.g., MacOS 10.13.6): 10.15.7
  • Python version (python --version): 3.8.10
  • Python client version (pip list | grep kubernetes): 21.7.0

Guide contributeur

Direction de recherche
Examinez comment la bibliothèque d'élection de leader déclenche OnStartedLeading et comment détecter quand elle se termine ; envisagez d'utiliser un future ou un callback similaire à l'annulation de contexte de Go.
Stack technique
python
Domaine
backenddistributed systems
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
GitPythonKubernetes
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60