Bug #22322
stats : inclure un identifiant de worker dans les infos à prometheus
0%
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
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
- Fichier 0001-include-worker-id-in-stats-22322.patch 0001-include-worker-id-in-stats-22322.patch ajouté
- Patch proposed changé de Non à Oui
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)
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