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/01/10 10:00:51 UTC

[GitHub] [airflow] XD-DENG edited a comment on pull request #12010: Support external redis in helm chart

XD-DENG edited a comment on pull request #12010:
URL: https://github.com/apache/airflow/pull/12010#issuecomment-757448775


   I would like to further "challenge" this PR:
   
   Wouldn't something like below work and serve the same purpose?
   
   ```
   helm install airflow . \
       --namespace airflow \
       --set executor=CeleryExecutor \
       --set redis.enabled=false \
       --set config.celery.broker_url=redis://redis-user:password@redis-host:6379/0
   ```
   
   To use an external Redis as broker, we just need to tell the Chart not to start a Redis instance, plus provide the external Redis URI to Airflow configuration. Then it should work as it was, without any change (including the changes in this PR).
   
   Likely I missed something. Please let me know @FloChehab @mik-laj 


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