You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/08 16:17:17 UTC

[GitHub] [airflow] mik-laj commented on a change in pull request #13209: Add support for worker persistence with KEDA v2.0.0 in helm chart

mik-laj commented on a change in pull request #13209:
URL: https://github.com/apache/airflow/pull/13209#discussion_r589551080



##########
File path: docs/helm-chart/keda.rst
##########
@@ -54,19 +50,22 @@ to set ``worker.persistence.enabled`` to ``false``)
        --namespace airflow \
        --set executor=CeleryExecutor \
        --set workers.keda.enabled=true \
-       --set workers.persistence.enabled=false
+
+A ``ScaledObject`` and an ``hpa`` will be created in the airflow namespace.
 
 KEDA will derive the desired number of celery workers by querying
 Airflow metadata database:
 
-.. code-block:: none
+.. code-block:: sql
 

Review comment:
       this does not work because it is not valid SQL. It is mix of Jinja and SQL. 




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org