[GitHub] [airflow] amoghrajesh commented on pull request #29624: Can't configure Kubernetes and Celery workers in Helm Chart

2023-02-23 Thread via GitHub
amoghrajesh commented on PR #29624: URL: https://github.com/apache/airflow/pull/29624#issuecomment-1441543267 @hussein-awala @potiuk I kinda like the idea of separating the differences between CeleryK8sExecutor and CeleryExecutor. @hussein-awala's idea seems reasonable but requires a fair

[GitHub] [airflow] amoghrajesh commented on pull request #29624: Can't configure Kubernetes and Celery workers in Helm Chart

2023-02-20 Thread via GitHub
amoghrajesh commented on PR #29624: URL: https://github.com/apache/airflow/pull/29624#issuecomment-1437336039 Thanks, my bad. Pulled it out of the comment @potiuk -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] amoghrajesh commented on pull request #29624: Can't configure Kubernetes and Celery workers in Helm Chart

2023-02-20 Thread via GitHub
amoghrajesh commented on PR #29624: URL: https://github.com/apache/airflow/pull/29624#issuecomment-1437079339 Thanks for explaining that @hussein-awala Made the changes in sync with what was asked in the issue description: #28880. Maybe I understood the issue wrong. Could you go through