glennhsh commented on code in PR #58547:
URL: https://github.com/apache/airflow/pull/58547#discussion_r2623547923
##########
chart/templates/workers/worker-deployment.yaml:
##########
@@ -20,10 +20,20 @@
################################
## Airflow Worker Deployment
#################################
+{{- $workerSets := .Values.workers.sets | default list -}}
Review Comment:
Thanks a lot for the review!
I completely agree with the direction. Since this change only impacts Celery
workers, I’ve moved it under workers.celery as suggested.
Appreciate the clarification and guidance 🙏
--
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]
