grafana/pyroscope

[remoteWrite] add more info (like url) when showing errors

Open

#1,961 opened on 2023年5月25日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)C (9,041 stars) (528 forks)batch import
good first issue

説明

time="2023-05-25T15:55:10.103354" level=error msg="error happened while ingesting data" file=" remotewrite/queue.go:96" error="response not ok: status code: '404'. body: '404 page not found\n'" key="foo.cpu{}" remote_target=bar

This message would be more helpful if it had the url which is trying to be hit.

コントリビューターガイド

技術スタック
go
領域
backend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
1
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
なし
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
90
調査方針
The error message is constructed in `remotewrite/queue.go` around line 96. Look for the line that logs the error and modify it to include the URL being requested. Check the HTTP client configuration in the same package to access the request URL. Ensure the URL is available in that scope. No open PRs or conflicting work.