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:
[email protected]
