Skip to content

feat: add api http metrics#4162

Merged
SuperQ merged 1 commit intoprometheus:mainfrom
siavashs:feat-api-metrics
Oct 20, 2025
Merged

feat: add api http metrics#4162
SuperQ merged 1 commit intoprometheus:mainfrom
siavashs:feat-api-metrics

Conversation

@siavashs
Copy link
Contributor

@siavashs siavashs commented Dec 16, 2024

  • measure api http request duration

@siavashs siavashs requested a review from SuperQ October 18, 2025 19:51
@siavashs
Copy link
Contributor Author

CircleCI is flaky as always, i think we should drop it and just use Github actions.

@SuperQ
Copy link
Member

SuperQ commented Oct 19, 2025

We've been slowly migrating things to GitHub Actions. While I'm not a huge fan, it's the most supported free pipeline system right now.

I have a backlog TODO to refactor the promci actions into separate repos for different steps. We started it as a monorepo, which really doesn't work for actions.

@siavashs siavashs force-pushed the feat-api-metrics branch 2 times, most recently from c704b54 to 6e059ae Compare October 19, 2025 15:47
@siavashs siavashs requested a review from SuperQ October 19, 2025 15:47
@siavashs
Copy link
Contributor Author

siavashs commented Oct 19, 2025

Lint issues are unrelated!
We run lint in a dedicated job and also in the CI job.

@SuperQ
Copy link
Member

SuperQ commented Oct 20, 2025

Fix for test failure: #4613

- measure api http request duration

Signed-off-by: Siavash Safi <siavash@cloudflare.com>
@siavashs
Copy link
Contributor Author

thanks, rebased!

@SuperQ SuperQ merged commit 97bbb5a into prometheus:main Oct 20, 2025
11 checks passed
@siavashs siavashs deleted the feat-api-metrics branch October 20, 2025 14:30
@SoloJacobs SoloJacobs mentioned this pull request Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants