anzinmhd opened a new pull request, #74912:
URL: https://github.com/apache/airflow/pull/74912

   <!-- SPDX-License-Identifier: Apache-2.0 -->
   
   Exclude the chart-generated `AIRFLOW__DATABASE__SQL_ALCHEMY_CONN` and 
`AIRFLOW_CONN_AIRFLOW_DB` variables from KubernetesExecutor task containers, 
allowing tasks to start without a metadata database Secret in their namespace.
   
   The shared environment helper accepts an internal `ExcludeDatabase` flag, 
enabled for the task container. Other callers retain their existing behavior.
   
   Closes #74850. Reproduction details are recorded in the issue.
   
   ### Verification
   
   - **Airflow 3.2.2, separate worker namespace:** reproduced 
`CreateContainerConfigError` without the metadata Secret before the fix. A 
fresh task succeeded with fake database credentials. After the fix, a fresh 
task succeeded with the Secret absent, and neither database variable appeared 
in the task container.
   - **Rendered configuration:** confirmed the task container omits both 
variables while the scheduler retains them. The original relevant templates 
matched upstream `main` at `b9aa889577`; runtime verification used Airflow 
3.2.2.
   - **Regression coverage:** both the chart-default and Airflow 3.2.2 cases 
passed. Confirmed the regression test fails without the fix.
   - **Tests and checks:** the complete pod-template test file passed; common 
chart tests passed (**39 tests**). Ruff, applicable prek checks on the three 
modified files, and `git diff --check` passed.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Codex (GPT-6)
   
   
   


-- 
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]

Reply via email to