This is an automated email from the ASF dual-hosted git repository.
Miretpl pushed a commit to branch chart/v1-2x-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/chart/v1-2x-test by this push:
new 6739d2863af [chart/v1-2x-test] Add Worker Set override tests for
Kerberos sidecar startup probe (#72017) (#72067)
6739d2863af is described below
commit 6739d2863aff81289f3a1b0f724b8800b87bce45
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Aug 25 20:48:03 2026 +0200
[chart/v1-2x-test] Add Worker Set override tests for Kerberos sidecar
startup probe (#72017) (#72067)
The Kerberos sidecar startup probe landed in #71221 without coverage for how
a Worker Set overrides it, so a regression in the per-set merge could ship
unnoticed. Reported in review of #71682.
(cherry picked from commit f27ddea356188ce7edcd9d9799e0269a7be62111)
Co-authored-by: Aaron Chen <[email protected]>
---
.../helm_tests/airflow_core/test_worker_sets.py | 62 ++++++++++++++++++++++
1 file changed, 62 insertions(+)
diff --git a/helm-tests/tests/helm_tests/airflow_core/test_worker_sets.py
b/helm-tests/tests/helm_tests/airflow_core/test_worker_sets.py
index ce38d98b2b7..8753ef388db 100644
--- a/helm-tests/tests/helm_tests/airflow_core/test_worker_sets.py
+++ b/helm-tests/tests/helm_tests/airflow_core/test_worker_sets.py
@@ -2140,6 +2140,68 @@ class TestWorkerSets:
jmespath.search("spec.template.spec.containers[?name=='worker-kerberos'] |
[0]", docs[0]) is None
)
+ @pytest.mark.parametrize(
+ ("startup_probe", "worker_set_startup_probe", "expected"),
+ [
+ (
+ {"enabled": False},
+ {"enabled": True},
+ {
+ "exec": {"command": ["klist", "-s"]},
+ "timeoutSeconds": 5,
+ "initialDelaySeconds": 0,
+ "periodSeconds": 10,
+ "failureThreshold": 6,
+ },
+ ),
+ ({"enabled": True}, {"enabled": False}, None),
+ (
+ {
+ "timeoutSeconds": 1,
+ "initialDelaySeconds": 2,
+ "periodSeconds": 3,
+ "failureThreshold": 4,
+ },
+ {"timeoutSeconds": 11},
+ {
+ "exec": {"command": ["klist", "-s"]},
+ "timeoutSeconds": 11,
+ "initialDelaySeconds": 2,
+ "periodSeconds": 3,
+ "failureThreshold": 4,
+ },
+ ),
+ ],
+ ids=["enabled", "disabled", "custom"],
+ )
+ def test_overwrite_kerberos_sidecar_startup_probe(
+ self, startup_probe, worker_set_startup_probe, expected
+ ):
+ docs = render_chart(
+ values={
+ "workers": {
+ "celery": {
+ "enableDefault": False,
+ "kerberosSidecar": {"enabled": True, "startupProbe":
startup_probe},
+ "sets": [
+ {
+ "name": "test",
+ "kerberosSidecar": {"startupProbe":
worker_set_startup_probe},
+ }
+ ],
+ }
+ }
+ },
+ show_only=["templates/workers/worker-deployment.yaml"],
+ )
+
+ assert (
+ jmespath.search(
+ "spec.template.spec.containers[?name=='worker-kerberos'] |
[0].startupProbe", docs[0]
+ )
+ == expected
+ )
+
@pytest.mark.parametrize(
"values",
[
