Uh oh!
There was an error while loading. Please reload this page.
Cap fastapi <0.137 and fix execution API health empty-path route - #68578
Conversation
Revanth14
commented
Jun 16, 2026
Thanks for the review and approval @shahar1 |
Uh oh!
There was an error while loading. Please reload this page.
potiuk
commented
Jun 22, 2026
I believe Cadwyn 7.1.0 has now support for 0.137.1+ -> I think we should bring it back @Revanth14 ? |
Revanth14
commented
Jun 22, 2026
Thanks for the heads-up @potiuk. I checked Cadwyn 7.1.0 and it now declares I’ll open a follow-up PR to bump Cadwyn to |
Revanth14
commented
Jun 22, 2026
Opened the follow-up PR #68826 @potiuk@shahar1 : bumped Cadwyn to |
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("")underprefix="/health", which now raisesPrefix and path cannot be both empty, andCadwyn's versioned router generation is not yet compatible with FastAPI 0.137.0.
fastapi<0.137.0, withuv.locksynced, to restore startup while waiting for Cadwyn compatibility./health,/health/ping) so they remain valid once the cap is lifted. External paths are unchanged.Was generative AI tooling used to co-author this PR?