prometheus / prometheus/client_python
Remove child metric
オープン
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 4.4k
- フォーク
- 876
- 平均マージ
- 8日 4時間
- マージ済み PR(30日)
- 1
説明
Hello.
Tried to remove some childs, but got KeyError.
My custom metric (gauge) have some lables and worked fine. But when i needed to delete some childs, i'm stuck:
global statusOK
statusOK = "OK"
global statusNOK
statusNOK = "Fail"
service_login_status_prom = prom.Gauge('service_login_status', 'Check access token availability', ['status', 'dc_name'])
result = str('0')
service_login_status_prom.labels(statusOK, dc_name).set(result)
service_login_status_prom.remove(statusNOK, dc_name)
Error is -
File "/usr/local/lib/python3.8/site-packages/prometheus_client/metrics.py", line 187, in remove
del self._metrics[labelvalues]
KeyError: ('Fail', '11u25')
When label is only one, error quite different -
File "/usr/local/lib/python3.8/site-packages/prometheus_client/metrics.py", line 187, in remove
del self._metrics[labelvalues]
KeyError: ('Fail', )
Comma at the end of labelvalues confused me
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
prometheus_client/metrics.py の 187 行目付近にある remove の実装から始め、issue にある 2 つのラベル数の例を再現してください。既存のメトリクステストで、child の削除と不足しているラベルタプルを確認してください。child を削除する際の期待される動作がテストでカバーされ、issue の結果に反映されていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- observability
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100