You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "dheerajturaga (via GitHub)" <gi...@apache.org> on 2026/03/09 15:07:13 UTC

[PR] Fix _execution_api_server_url() reading edge.api_url when execution_api_server_url is already set [airflow]

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix _execution_api_server_url() reading edge.api_url when execution_api_server_url is already set [airflow]

Posted by "dheerajturaga (via GitHub)" <gi...@apache.org>.
dheerajturaga merged PR #63192:
URL: https://github.com/apache/airflow/pull/63192


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org