Skip to content

Conversation

@vpaturet
Copy link
Contributor

@vpaturet vpaturet commented Dec 11, 2025

Summary

Create a new OTP feature that exports per-client response time metrics.
This is used to produce SLA/SLO dashboards grouped by API users.
The metrics uses a header in the HTTP request to identify the client.

This is a sandbox feature.

Issue

Close #7149

Unit tests

Added unit tests

Documentation

Updated documentation

@vpaturet vpaturet added the Entur Test This is currently being tested at Entur label Dec 11, 2025
@vpaturet vpaturet force-pushed the http-response-time-metric branch 3 times, most recently from 37d3f37 to 788776c Compare December 11, 2025 16:18
@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

❌ Patch coverage is 84.48276% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.12%. Comparing base (d0862d5) to head (3168bf2).
⚠️ Report is 36 commits behind head on dev-2.x.

Files with missing lines Patch % Lines
...ipplanner/standalone/server/OTPWebApplication.java 0.00% 9 Missing ⚠️
...onsetimemetrics/HttpResponseTimeMetricsFilter.java 86.44% 5 Missing and 3 partials ⚠️
...r/standalone/config/routerconfig/ServerConfig.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #7133      +/-   ##
=============================================
+ Coverage      72.09%   72.12%   +0.03%     
- Complexity     20684    20737      +53     
=============================================
  Files           2244     2249       +5     
  Lines          83892    84094     +202     
  Branches        8348     8364      +16     
=============================================
+ Hits           60481    60656     +175     
- Misses         20482    20501      +19     
- Partials        2929     2937       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vpaturet vpaturet removed the Entur Test This is currently being tested at Entur label Dec 11, 2025
@vpaturet vpaturet force-pushed the http-response-time-metric branch from 788776c to 0f3e506 Compare December 15, 2025 14:37
@vpaturet vpaturet added the Entur Test This is currently being tested at Entur label Dec 15, 2025
@vpaturet vpaturet force-pushed the http-response-time-metric branch 4 times, most recently from dc225ab to c385258 Compare December 18, 2025 12:37
@vpaturet vpaturet changed the title POC - Add HTTP response time metric Add HTTP response time metric Dec 18, 2025
@vpaturet vpaturet force-pushed the http-response-time-metric branch from 056c3cd to a00efa7 Compare December 18, 2025 18:27
@vpaturet vpaturet force-pushed the http-response-time-metric branch from 6d0cb28 to 3168bf2 Compare December 19, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Entur Test This is currently being tested at Entur

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add API response time monitoring per API client

1 participant