dheerajturaga opened a new pull request, #63192:
URL: https://github.com/apache/airflow/pull/63192
Previously, _execution_api_server_url() unconditionally read edge.api_url
even when core.execution_api_server_url was explicitly configured, causing
an unnecessary AirflowConfigException if only the latter was set. This also
caused test_supervise_launch and test_supervise_launch_fail to fail since
the test environment does not load provider config.
Reorder config reads so core.execution_api_server_url is checked first and
edge.api_url is only read as a fallback when needed. Mock
_execution_api_server_url in test_supervise_launch and
test_supervise_launch_fail
to isolate them from config requirements. Add a test case covering
core.execution_api_server_url set without edge.api_url.
Issue stemming from #56457
---
##### 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".
-->
- [ ] Yes (please specify the tool below)
ClaudeCode
--
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]
