Skip to content

Move execution_api_server_url config to the core section - #46969

Merged
jedcunningham merged 1 commit into
apache:mainfrom
astronomer:document_execution_api_server_url
Feb 21, 2025
Merged

jedcunningham merged 1 commit into
apache:mainfrom
astronomer:document_execution_api_server_url

Conversation

@jedcunningham

Copy link
Copy Markdown
Member

This adds it to config.yml so itll be documented and moves the default into config (if we have a fallback default, we might as well show it in docs too, and every use will have the default already).

This adds it to config.yml so itll be documented and moves the default into
config (if we have a fallback default, we might as well show it in docs
too, and every use will have the default already).
@jedcunningham jedcunningham added the full tests needed We need to run full set of tests for this PR to merge label Feb 21, 2025
@boring-cyborg boring-cyborg Bot added area:Executors-core LocalExecutor & SequentialExecutor area:helm-chart Airflow Helm Chart area:task-sdk provider:celery provider:edge Edge Executor / Worker (AIP-69) / edge3 labels Feb 21, 2025
Comment thread task_sdk/src/airflow/sdk/execution_time/execute_workload.py
Comment thread airflow/executors/local_executor.py

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@jedcunningham
jedcunningham merged commit 2d267dd into apache:main Feb 21, 2025
@jedcunningham
jedcunningham deleted the document_execution_api_server_url branch February 21, 2025 18:59
aaron-y-chen pushed a commit to aaron-y-chen/airflow that referenced this pull request Apr 4, 2025
This adds it to config.yml so itll be documented and moves the default into
config (if we have a fallback default, we might as well show it in docs
too, and every use will have the default already).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Executors-core LocalExecutor & SequentialExecutor area:helm-chart Airflow Helm Chart area:task-sdk full tests needed We need to run full set of tests for this PR to merge provider:celery provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants