Miretpl commented on code in PR #58547:
URL: https://github.com/apache/airflow/pull/58547#discussion_r2636247715
##########
chart/values.yaml:
##########
@@ -637,6 +637,12 @@ kerberos:
# Airflow Worker Config
workers:
+ # Enable the default worker defined by the root 'workers' configuration.
+ # If false, only workers defined in 'sets' will be created.
+ enableDefault: true
+
+ # Queue name for the worker
Review Comment:
```suggestion
# Queue name for the default workers
```
##########
chart/values.yaml:
##########
@@ -637,6 +637,12 @@ kerberos:
# Airflow Worker Config
workers:
+ # Enable the default worker defined by the root 'workers' configuration.
+ # If false, only workers defined in 'sets' will be created.
Review Comment:
```suggestion
# Enable the default workers defined by the root 'workers' configuration
to be created.
# If false, only dedicated workers defined in 'sets' will be created.
```
I feel like the description from one point is good and from the other
something is missing for me, so just a small proposal
--
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]
