prometheus / prometheus/client_python
OpenMetrics version header out-of-date / Support latest OpenMetrics format
まだ誰も着手していません。
- 主要言語
- 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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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