Chart: Change default executor to CeleryExecutor (#15603)

``CeleryExecutor`` is currently the most stable of all, so it should be the default when using the Helm Chart.
apache · Apr 30, 2021 · c7cd437 · c7cd437
1 parent 7f6ddda
commit c7cd437
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/chart/tests/test_basic_helm_chart.py b/chart/tests/test_basic_helm_chart.py
@@ -24,7 +24,7 @@
 
 from tests.helm_template_generator import render_chart
 
-OBJECT_COUNT_IN_BASIC_DEPLOYMENT = 22
+OBJECT_COUNT_IN_BASIC_DEPLOYMENT = 30
 
 
 class TestBaseChartTest(unittest.TestCase):
@@ -55,13 +55,21 @@ def test_basic_deployments(self):
             ('RoleBinding', 'TEST-BASIC-pod-log-reader-rolebinding'),
             ('Service', 'TEST-BASIC-postgresql-headless'),
             ('Service', 'TEST-BASIC-postgresql'),
+            ('Service', 'TEST-BASIC-flower'),
+            ('Service', 'TEST-BASIC-redis'),
             ('Service', 'TEST-BASIC-statsd'),
             ('Service', 'TEST-BASIC-webserver'),
+            ('Service', 'TEST-BASIC-worker'),
+            ('Deployment', 'TEST-BASIC-flower'),
             ('Deployment', 'TEST-BASIC-scheduler'),
             ('Deployment', 'TEST-BASIC-statsd'),
             ('Deployment', 'TEST-BASIC-webserver'),
             ('StatefulSet', 'TEST-BASIC-postgresql'),
+            ('StatefulSet', 'TEST-BASIC-redis'),
+            ('StatefulSet', 'TEST-BASIC-worker'),
             ('Secret', 'TEST-BASIC-fernet-key'),
+            ('Secret', 'TEST-BASIC-redis-password'),
+            ('Secret', 'TEST-BASIC-broker-url'),
             ('Job', 'TEST-BASIC-create-user'),
             ('Job', 'TEST-BASIC-run-airflow-migrations'),
         ]

diff --git a/chart/values.yaml b/chart/values.yaml
@@ -110,7 +110,7 @@ rbacEnabled: true
 
 # Airflow executor
 # Options: LocalExecutor, CeleryExecutor, KubernetesExecutor, CeleryKubernetesExecutor
-executor: "KubernetesExecutor"
+executor: "CeleryExecutor"
 
 # If this is true and using LocalExecutor/KubernetesExecutor/CeleryKubernetesExecutor, the scheduler's
 # service account will have access to communicate with the api-server and launch pods.

diff --git a/scripts/ci/libraries/_kind.sh b/scripts/ci/libraries/_kind.sh
@@ -333,7 +333,8 @@ function kind::deploy_airflow_with_helm() {
         --set "defaultAirflowTag=${AIRFLOW_PROD_BASE_TAG}-kubernetes" -v 1 \
         --set "config.api.auth_backend=airflow.api.auth.backend.default" \
         --set "config.api.enable_experimental_api=true" \
-        --set "config.logging.logging_level=DEBUG"
+        --set "config.logging.logging_level=DEBUG" \
+        --set "executor=KubernetesExecutor"
     echo
     popd > /dev/null 2>&1|| exit 1
 }