prometheus / prometheus/client_python

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

オープン
#901 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Python
スター
4.4k
フォーク
876
平均マージ
8日 4時間
マージ済み PR(30日)
1

説明

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

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

prometheus_client/openmetrics/exposition.py の CONTENT_TYPE_LATEST から始め、issue にリンクされている OpenMetrics 仕様とその header を比較します。0.0.1 から 1.0.0 への変更に header の更新以上の対応が必要かどうかを判断します。client が最新の必須 format をサポートし、参照されている互換性上の懸念と矛盾しなければ完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
observability
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
52/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。