henry3260 commented on PR #64523:
URL: https://github.com/apache/airflow/pull/64523#issuecomment-6060691004

   Added. The prefix is `/execution` rather than `/api/execution` — 
`API_ROOT_PATH` defaults to `/`, so that mount point is the whole path.
   
   Adding the prefix alone wasn't enough: `--apps execution` never reaches 
`init_middlewares`, so that deployment shape had no metrics middleware at all. 
Registration moved to `init_api_metrics(app)`, called for every `--apps`
   selection — the same shape #72026 used for the access log. The route tag now 
prepends `root_path`, since a mounted sub-app's routes are relative to its 
mount point.
   
   Kept to a single instance on the outer app: it already sees execution 
requests, so a second one inside the sub-app would double-count and would sit 
inside `GZipMiddleware`, excluding compression time from execution durations 
only.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to