mjlshen commented on PR #41173: URL: https://github.com/apache/superset/pull/41173#issuecomment-4772400322
> Fix looks right and thanks for sorting out the chart bump. Out of curiosity — the celery `worker` and `beat` commands in `values.yaml` run the same `sh -c "...; celery ..."` pattern without `exec`. Want to give them the same treatment here, or keep this PR scoped to the web node? Approving for now either way. Got it, yeah I had the celery `worker` fixes in https://github.com/apache/superset/pull/41175 and happy to pull in the celery `beat` fixes there too. I prefer keeping this PR scoped to the web node because it fixes the specific issue of losing availability of the web service during rollouts -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
