This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 09603ff23f Adding versioned unit tests for kerberos init container 
(#36364)
09603ff23f is described below

commit 09603ff23fa648b6d7b510e9fca8cc5cfcb5bab0
Author: Amogh Desai <[email protected]>
AuthorDate: Sat Dec 30 16:25:07 2023 +0530

    Adding versioned unit tests for kerberos init container (#36364)
---
 helm_tests/airflow_core/test_worker.py | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/helm_tests/airflow_core/test_worker.py 
b/helm_tests/airflow_core/test_worker.py
index a65db04a5d..1cd1af5f7e 100644
--- a/helm_tests/airflow_core/test_worker.py
+++ b/helm_tests/airflow_core/test_worker.py
@@ -564,21 +564,28 @@ class TestWorker:
         } in jmespath.search("spec.template.spec.containers[2].volumeMounts", 
docs[0])
 
     @pytest.mark.parametrize(
-        "airflow_version, expected_init_containers",
+        "airflow_version, init_container_enabled, expected_init_containers",
         [
-            ("1.9.0", 2),
-            ("1.10.14", 2),
-            ("2.0.2", 2),
-            ("2.1.0", 2),
-            ("2.8.0", 3),
+            ("1.9.0", True, 2),
+            ("1.9.0", False, 2),
+            ("1.10.14", True, 2),
+            ("1.10.14", False, 2),
+            ("2.0.2", True, 2),
+            ("2.0.2", False, 2),
+            ("2.1.0", True, 2),
+            ("2.1.0", False, 2),
+            ("2.8.0", True, 3),
+            ("2.8.0", False, 2),
         ],
     )
-    def test_airflow_kerberos_init_container(self, airflow_version, 
expected_init_containers):
+    def test_airflow_kerberos_init_container(
+        self, airflow_version, init_container_enabled, expected_init_containers
+    ):
         docs = render_chart(
             values={
                 "airflowVersion": airflow_version,
                 "workers": {
-                    "kerberosInitContainer": {"enabled": True},
+                    "kerberosInitContainer": {"enabled": 
init_container_enabled},
                     "persistence": {"fixPermissions": True},
                 },
             },

Reply via email to