prometheus / prometheus/client_python

Add support for passing in label kwargs to remove() function

Ouverte
#820 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

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

Hello,

Would it be possible/would it get maintainer support to add the ability to pass in kwargs to the remove() function:
https://github.com/prometheus/client_python/blob/5a5261dd45d65914b5e3d8225b94d6e0578882f3/prometheus_client/metrics.py#L200

I think it should be pretty simple to do, I think the logic would remain largely the same as the above labels() function from an implementation perspective, but rather than setting a value it just calls del on it.

Thanks!

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez dans prometheus_client/metrics.py, autour de la fonction remove() mentionnée, et comparez-la à la fonction labels(). Ajoutez la prise en charge de la suppression via label kwargs tout en préservant le comportement existant, puis exécutez les tests pertinents du projet et confirmez que les deux formes de suppression fonctionnent correctement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
observability
Type d'issue
Fonctionnalité
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.