prometheus / prometheus/client_python
Multiprocess mode support for 'remove'
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 4.4k
- Forks
- 876
- Merge moyen
- 8 j 4 h
- PR mergées (30 j)
- 1
Description
In multiprocess mode calling 'remove' on a metric/label doesn't actually remove it from the metrics output.
Also, because of the disconnect between the Metric classes's remove and MultiProcessValue, the following snippet will use more memory every iteration as the MultiProcessValue.values list keeps adding copies.
import os
os.environ['PROMETHEUS_MULTIPROC_DIR'] = '/tmp'
from prometheus_client import Counter
c = Counter('a', 'a', ['a'])
while True:
c.labels('hi').inc()
c.remove('hi')
Perhaps 'remove' should be disabled in multiprocess mode, or at least something added to the multiprocess documentation?
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par reproduire la boucle fournie en mode multiprocess, puis examinez le comportement de remove des classes Metric ainsi que MultiProcessValue et la documentation de multiprocess. Déterminez comment la suppression devrait affecter les métriques émises et la liste values qui s’agrandit ; le travail est considéré comme terminé lorsque le comportement choisi est implémenté et couvert par un test de régression, ou lorsque la limitation est clairement documentée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- observability-sre
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100