himanshuchawla0807-gif commented on code in PR #74912:
URL: https://github.com/apache/airflow/pull/74912#discussion_r4306972821
##########
chart/files/pod-template-file.kubernetes-helm-yaml:
##########
@@ -122,7 +122,7 @@ spec:
- name: KRB5CCNAME
value: {{ include "kerberos_ccache_path" . | quote }}
{{- end }}
- {{- include "standard_airflow_environment" . | indent 6}}
+ {{- include "standard_airflow_environment" (merge (dict
"ExcludeDatabase" true) .) | indent 6 }}
Review Comment:
You are right: the template-only correction in 06a653f is insufficient at
runtime and should not be applied alone to older images.
I separated the CLI prerequisite into #75000, with no chart changes. It
skips the Kerberos command’s mandatory migration and log-template preflight.
Two regressions cover standard and one-time dispatch when both database helpers
would fail. An isolated CLI with an empty SQLite database and migration checks
enabled reaches stub `kinit`; this does not establish real KDC or cluster
operation.
The refreshed-base Always/CLI database suites pass 2,613 tests, and the
non-database suite passes 659 with one expected failure. Regular/manual static
checks and core documentation build also pass.
Released 3.2.2 images remain unchanged, so the Helm change still needs a
verified compatibility strategy for those images. Optional CLI audit logging
can still attempt database access.
---
Drafted-by: OpenAI Codex (no human review before posting)
--
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]
