prometheus / prometheus/client_python

`namespace` (and `subsystem`) for `*MetricFamily` classes

Offen
#1,048 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
4.4k
Forks
876
Ø Merge
8 T. 4 Std.
Gemergte PRs (30 T.)
1

Beschreibung

Hey.

The MetricWrapperBase derived classes (Counter, Gauge, etc) all have namespace and subsystem in their constructors and automatically combine that into a final name.

Wouln't it make sense to have at least namespace also for the *MetricFamily classes`

Cheers,
Chris.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie damit, die *MetricFamily-Klassen zu lokalisieren und ihre Konstruktoren mit MetricWrapperBase und dessen Counter- und Gauge-Ableitungen zu vergleichen. Klären Sie, ob nur namespace oder sowohl namespace als auch subsystem erforderlich sind, definieren Sie anschließend das erwartete Verhalten des kombinierten Namens und überprüfen Sie es mit gezielten Tests.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
observability-sre
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.