Skip to content

[v3-4-test] Allow FastAPI 0.137+ by requiring cadwyn 8.1 (#75689) - #75715

Merged
vatsrahul1001 merged 1 commit into
v3-4-testfrom
backport-b70a439-v3-4-test
Oct 27, 2026
Merged

vatsrahul1001 merged 1 commit into
v3-4-testfrom
backport-b70a439-v3-4-test

Conversation

@vatsrahul1001

@vatsrahul1001 vatsrahul1001 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Manual backport of #75689 to v3-4-test.

The auto-backport conflicted because #75689 was built on two features that are not on v3-4-test (both 3.5.0-scoped and deliberately kept off the branch). This is a pruned subset that drops the parts depending on them and keeps the FastAPI/cadwyn bump plus the lazy-router + native-OpenTelemetry-disable adaptations.

Dropped (absent from v3-4-test):

Kept / adapted:

  • fastapi>=0.142 (resolves 0.161) + cadwyn>=8.1 in airflow-core and task-sdk (+ uv.lock).
  • NATIVE_TELEMETRY_DISABLED applied to all FastAPI apps (core, execution, log server).
  • Lazy-router adaptation (iter_route_contexts / iter_api_routes) in execution_api/app.py and the remaining execution-API tests, including test_token_scope_boundaries.py::TestTokenScopeBoundaries and the new test_fastapi_native_telemetry_records_no_spans.
  • taskinstance.py Discriminator(...) change.

Verified locally: no conflict markers, py_compile clean, and the remaining source references only symbols present on v3-4-test (require_auth, _REQUEST_SCOPE_TOKEN_KEY, InProcessExecutionAPI) plus fastapi 0.161's iter_route_contexts. Full execution-API test validation runs here in CI under fastapi 0.161.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.4.0 milestone Oct 27, 2026
Manual backport: pruned subset of #75689 excluding the #64523 (REST metrics) and #74370 (dag-processor scoped Execution API) pieces that are not on v3-4-test. Keeps the fastapi>=0.142/cadwyn>=8.1 bump + native-OTel-disable + lazy-router adaptations.

(cherry picked from commit b70a439)
@vatsrahul1001
vatsrahul1001 force-pushed the backport-b70a439-v3-4-test branch from 999fa53 to 1ee8e80 Compare October 27, 2026 05:55
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ 2 job(s) of attempt 1 were stopped because AWS reclaimed their EC2 Spot instances. These are not test failures.

RunsOn re-runs them automatically when the run finishes; no action is needed.

@vatsrahul1001
vatsrahul1001 merged commit 0ff47dc into v3-4-test Oct 27, 2026
159 checks passed
@vatsrahul1001
vatsrahul1001 deleted the backport-b70a439-v3-4-test branch October 27, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:dev-tools area:providers area:task-sdk provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants