prometheus / prometheus/client_python

OpenMetrics version header out-of-date / Support latest OpenMetrics format

Offen
#901 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

Hi,
Looking at the Content-Type header

CONTENT_TYPE_LATEST = 'application/openmetrics-text; version=0.0.1; charset=utf-8'

The version=0.0.1 should be bumped. Looking at the spec:

The content type MUST be: application/openmetrics-text; version=1.0.0; charset=utf-8

I'm afraid I haven't been able to find any clear doc detailing changes between 0.0.1 and 1.0.0 so I don't know if it's as simple as just updating that line.

FWIW I've seen other places encountering the same trouble with the MUST restriction: prometheus/prometheus#9430, prometheus/client_java#702

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

Beginne in prometheus_client/openmetrics/exposition.py bei CONTENT_TYPE_LATEST und vergleiche dessen Header mit der OpenMetrics-Spezifikation, die im Issue verlinkt ist. Ermittle, ob die Änderung von 0.0.1 zu 1.0.0 mehr als eine Aktualisierung des Headers erfordert; als erledigt gilt die Aufgabe, wenn der Client das neueste erforderliche Format unterstützt, ohne den genannten Kompatibilitätsbedenken zu widersprechen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
observability
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

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