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:45:04 UTC

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

ashb commented on a change in pull request #13209:
URL: https://github.com/apache/airflow/pull/13209#discussion_r589575026



##########
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:
       We could set it to `:: jinja` I think, but that probably expects it to be template to output HTML, so would likely be worse than no highlighting.




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