Revanth14 opened a new pull request, #68578:
URL: https://github.com/apache/airflow/pull/68578
Closes: #68562
FastAPI 0.137.0's lazy router inclusion breaks api-server and dag-processor
startup in two ways: the execution API health route used `@router.get("")`
under `prefix="/health"`, which now raises `Prefix and path cannot be both
empty`, and
Cadwyn's versioned router generation is not yet compatible with FastAPI
0.137.0.
- Cap `fastapi<0.137.0`, with `uv.lock` synced, to restore startup while
waiting for Cadwyn compatibility.
- Change the execution API health routes to explicit paths (`/health`,
`/health/ping`) so they remain valid once the cap is lifted. External paths are
unchanged.
- Add a regression test to prevent reintroducing an empty health route path.
---
##### Was generative AI tooling used to co-author this PR?
<!--
If generative AI tooling has been used in the process of authoring this PR,
please
change below checkbox to `[X]` followed by the name of the tool, uncomment
the "Generated-by".
-->
- [X] Yes (Opus 4.8)
<!--
Generated-by: [Tool Name] following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
-->
--
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]
