fix: scheduler, triggerer, worker airflow components have inconsistent log volume

[rich7420]

related: #56015

How I Fixed It in this PRr

I added the missing logs volume mount to the wait-for-airflow-migrations initContainers in these files:

• chart/templates/scheduler/scheduler-deployment.yaml
• chart/templates/triggerer/triggerer-deployment.yaml
• chart/templates/workers/worker-deployment.yaml

Test

Test 1: Reproduce the Original Problem

Grab this values file to mimic the issue:

# test-original-issue.yaml
scheduler:
  extraVolumeMounts:
  - mountPath: /usr/local/airflow/logs
    name: airflow-logs
  - mountPath: /tmp
    name: tmp
  extraVolumes:
  - emptyDir: {}
    name: airflow-logs
  - emptyDir: {}
    name: tmp

triggerer:
  extraVolumeMounts:
  - mountPath: /usr/local/airflow/logs
    name: airflow-logs
  - mountPath: /tmp
    name: tmp
  extraVolumes:
  - emptyDir: {}
    name: airflow-logs
  - emptyDir: {}
    name: tmp

workers:
  extraVolumeMounts:
  - mountPath: /usr/local/airflow/logs
    name: airflow-logs
  - mountPath: /tmp
    name: tmp
  extraVolumes:
  - emptyDir: {}
    name: airflow-logs
  - emptyDir: {}
    name: tmp

securityContexts:
  containers:
    readOnlyRootFilesystem: true

Run this to check:

helm template test-release chart/ -f test-original-issue.yaml | grep -A 25 "wait-for-airflow-migrations"

What you should see is all initContainers now have both the standard logs volume and any extra mounts—no more mismatches.

Test 2: With Persistence Enabled

Try this values file for persistent logs:

# test-persistence.yaml
logs:
  persistence:
    enabled: true
    subPath: "airflow-logs"

Check it out:

helm template test-release chart/ -f test-persistence.yaml | grep -A 25 "wait-for-airflow-migrations"

What you should see is InitContainers include the logs volume with the right subPath config.

Test 3: Just the Basics

No extra bells and whistles—test the default setup:

# test-minimal.yaml
# Empty file, basically defaults only

Run:

helm template test-release chart/ -f test-minimal.yaml | grep -A 25 "wait-for-airflow-migrations"

Then you should see Every initContainer has the standard logs volume mount, loud and clear.

Any Breaking Changes?

Nope, none at all.

---

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

[jason810496]

Nice! Thanks for the PR.
It would be nice to update the helm-tests as well. Thanks!

[rich7420]

@jason810496 Thanks for your suggestion. I've added helm-tests .

[Miretpl]

Hi @rich7420, some tests are failing. Could you take a look?

[rich7420]

@Miretpl thanks for the reminding! I'll try to fix it up as soon as possible.

[Miretpl]

Looks good to me. Thanks for the fix!

[jason810496]

Thanks for the update!
It would be nice to test with breeze k8s command, thanks!

https://github.com/apache/airflow/blob/cc16dd60ac78ffff9e2b6c49a3afac677adcd92c/contributing-docs/testing/k8s_tests.rst

[rich7420]

@jason810496 I've tried test with breeze k8s.
results below:
=========================== short test summary info ============================
SKIPPED [1] tests/kubernetes_tests/test_kubernetes_pod_operator.py:1495: AIP-72: Secret Masking yet to be implemented
SKIPPED [1] tests/kubernetes_tests/test_other_executors.py:29: Does not run on KubernetesExecutor
SKIPPED [1] tests/kubernetes_tests/test_other_executors.py:52: Does not run on KubernetesExecutor
============ 56 passed, 3 skipped, 63 warnings in 676.90s (0:11:16) ============

[rich7420]

Thanks @jscheffl , @jason810496 , @Miretpl !