Switch from janus_http_request_duration to http.client.request.duration
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 70
- Forks
- 16
- Avg merge
- 4h 36m
- Merged PRs (30d)
- 30
Description
We currently define our own histogram metric for latency and status of HTTP client requests, janus_http_request_duration. The name is a bit unfortunate, as it doesn't include "client" in it. We could adopt the OpenTelemetry HTTP semantic conventions and use http.client.request.duration. Note that we are already following SemCov for HTTP server metrics.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Search the repository for janus_http_request_duration and inspect the code and tests that define or record this HTTP client histogram. The work is done when the metric uses http.client.request.duration consistently and its existing latency and status behavior remains covered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- observability
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100