Miretpl commented on PR #60396:
URL: https://github.com/apache/airflow/pull/60396#issuecomment-3735672577
Yes.
The idea is, if some `workers` section is used in both places
(`worker-deployment` and `pod-template-file`), we want to give an easy option
to overwrite the values for one, but if someone wants to specify the same
config for both, I think it should be possible (maybe with an exception for
`resources` as having the same `resources` for celery workers and kubernetes
executor pods would be a bad idea).
Regarding the `newsfragment`, until now I only added them when the change
was changing the behaviour of the chart, so e.g. in cases like:
```
{{ workers.celery.example | default workers.example }}
```
where `example` would be e.g. `1` by default, but if the default was unset
or `null`, I didn't add the newsfragment (only the change in NOTES if the value
was deprecated).
--
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]
