prometheus / prometheus/client_python
Multiprocess mode support for 'remove'
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 4.4k
- Forks
- 876
- Merge medio
- 8 d 4 h
- PR fusionados (30 d)
- 1
Descripción
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?
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza reproduciendo el bucle proporcionado en modo multiproceso y, después, inspecciona el comportamiento de remove de las clases Metric junto con MultiProcessValue y la documentación de multiprocess. Determina cómo debería afectar la eliminación a las métricas emitidas y a la lista de values, que sigue creciendo; se considera terminado cuando el comportamiento elegido está implementado y cubierto por una prueba de regresión, o cuando la limitación está claramente documentada.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- observability-sre
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100