Skip to content

Consolidate in-process Execution API hosting - #73327

Merged
ephraimbuddy merged 2 commits into
apache:mainfrom
astronomer:consolidate-execution-api-hosting
Oct 9, 2026
Merged

ephraimbuddy merged 2 commits into
apache:mainfrom
astronomer:consolidate-execution-api-hosting

Conversation

@ephraimbuddy

Copy link
Copy Markdown
Contributor

Dag parsing, trigger execution, local Dag tests and schema generation need consistent Execution API setup. Keeping in-process authentication and transport lifecycle together prevents setup drift while preserving isolation from network-served apps.

Manager construction should not load the API server dependency stack. Existing Task SDK consumers and schema generation against older checkouts also need the established app.py import path to remain valid.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Codex gpt6, claude Opus 5 following the guidelines

Comment thread airflow-core/src/airflow/dag_processing/manager.py
Comment thread airflow-core/src/airflow/jobs/triggerer_job_runner.py
Comment thread airflow-core/tests/unit/api_fastapi/execution_api/test_in_process.py Outdated
Dag parsing, trigger execution, local Dag tests and schema generation
need consistent Execution API setup. Keeping in-process authentication
and transport lifecycle together prevents setup drift while preserving
isolation from network-served apps.

Manager construction should not load the API server dependency stack.
Existing Task SDK consumers and schema generation against older
checkouts also need the established app.py import path to remain valid.
@ephraimbuddy
ephraimbuddy force-pushed the consolidate-execution-api-hosting branch from 8830f1c to cc8c268 Compare October 9, 2026 10:48
@ephraimbuddy
ephraimbuddy merged commit dc38f84 into apache:main Oct 9, 2026
85 checks passed
@ephraimbuddy
ephraimbuddy deleted the consolidate-execution-api-hosting branch October 9, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants