jscheffl commented on PR #67569: URL: https://github.com/apache/airflow/pull/67569#issuecomment-4557932752
> > Agree a rolling restart is a workaround and should not be a permanent chart feature. This PR adds an opt-in CronJob disabled by default so teams can use it as a short term mitigation. > > I would be really against adding workaround features to the chart. We have, e.g., PostgreSQL within the chart currently, which was meant only for development purposes, and there are teams which are using it for production. I believe that we should not encourage users to use this particular workaround by implementing it and making it easy to use. If there is a team which will need to do it, they could just create the CronJob definition and apply it to the Kubernetes cluster. Then - if really somebody needs it - would propose to add this being a Kustomize Layer example we can add to the repo with some docs how to apply but not adding this to main chart. -- 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]
