sudeepgupta90 commented on issue #20999: URL: https://github.com/apache/airflow/issues/20999#issuecomment-1376930278
@jedcunningham I agree that redis can be disabled. I actually got confused after seeing only `if and .Values.redis.enabled` in redis manifests inside the Airflow chart and missed that we can use external redis. But my point was that this is extra appendage being added to the Airflow chart which is not really needed. From what I understand, the earlier versions of Airflow only had CeleryExecutor so having a simplified redis distribution bundled inside the chart - made it easy to deploy. From my earlier comments, and users raising issues in redis components - it would fall on the Airflow maintenance to fix them, when it should be infact managed at redis chart level. Doing this refactoring, does not add immediate value to the Airflow chart, but this is something which may be looked into for long term maintenance goal to reduce clutter and issues. Wdyt? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
