Project

General

Profile

Bug #22322

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

Added by Frédéric Péters about 2 years ago. Updated about 2 years ago.

Status:
Nouveau
Priority:
Normal
Category:
-
Target version:
-
Start date:
06 Mar 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-include-worker-id-in-stats-22322.patch View (6.11 KB) Emmanuel Cazenave, 07 Mar 2018 04:04 PM

History

#1 Updated by Emmanuel Cazenave about 2 years ago

  • Assignee set to Emmanuel Cazenave

#2 Updated by Emmanuel Cazenave about 2 years ago

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)

Also available in: Atom PDF