Projet

Général

Profil

Bug #22322

stats : inclure un identifiant de worker dans les infos à prometheus

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a 8 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
06 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

cf #22270

Lu ailleurs (https://hynek.me/talks/prometheus/), mais je ne sais pas si c'est pertinent pour nous : « Another approach is to expose your metrics per process. In uWSGI you can use uwsgi.worker_id() as a label »

Oui c'est une bonne approche, de manière apparentée inclure le pid dans les paramètres de la métrique (pour que ça soit ok aussi avec gunicorn). (et côté graphe ça accumulera les valeurs).

faire le worker_id si on est dans uwsgi sinon getpid.


Fichiers

Historique

#1

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Assigné à mis à Emmanuel Cazenave
#2

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

Testé avec runserver et uwsgi (merci Thomas).

On calcule le worker_id au moment l'appel au middleware et pas au moment de l'import parce que sinon ça se comporte mal avec uwsgi (uwsgi.worker_id renvoie 0 == le process n'est pas un worker)

A voir à l'usage si prometheus agrège tout ça correctement (en particulier si lorsqu'il poll un https://monitored.publik.service/__metrics__ et que c'est un worker qui n'a rien fait qui lui répond, ça remet pas tout à 0)

#3

Mis à jour par Frédéric Péters il y a 8 mois

  • Statut changé de Nouveau à Fermé
  • Planning mis à Non

Ça se termine plutôt en virant les métriques prometheus, #79709

Formats disponibles : Atom PDF