Multiprocess mode support for 'remove'
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- observability-sre
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.
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?
- Langage dominant
- Python
- Étoiles
- 4.4k
- Forks
- 876
- Merge moyen
- 8 j 4 h
- PR mergées (30 j)
- 1
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.
Autres issues de prometheus/client_python
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
prometheus/client_python#1177 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
prometheus/client_python#1210 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 58/100
prometheus/client_python#1199 · 1 réaction ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
prometheus/client_python#1176 ·
-
WSL and MultiProcessCollector Ouverte
Difficulté 1/5 1-3 heures Accessibilité débutants 52/100
prometheus/client_python#1126 · 2 commentaires ·
Toutes les issues de prometheus/client_python
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
stephrobert/dsoxlab#238 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
sublimehq/package_control#1780 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
nwg-piotr/nwg-displays#145 ·