vyncint commented on PR #70205:
URL: https://github.com/apache/airflow/pull/70205#issuecomment-5074359276
Implemented the `podLabels` split requested in the review and earlier
discussion:
- Leaving `dagProcessor.podLabels` unset preserves the existing behavior:
`dagProcessor.labels` applies to pods, while the Deployment receives global
labels only.
- Setting `dagProcessor.podLabels` (including `{}`) opts into separation:
`dagProcessor.labels` applies to the Deployment and `dagProcessor.podLabels`
applies to pods.
- Component labels keep precedence over global labels, and `deepCopy`
prevents `mustMerge` from mutating the source values.
I also added values/schema documentation, a news fragment, and regression
coverage for the fallback, separated-label, collision, and explicit-empty-map
cases. The branch is rebased on current `main`.
Validation completed:
- `breeze testing helm-tests --use-xdist` — 2095 passed
- `prek run --from-ref upstream/main --stage pre-commit`
- `prek run --from-ref upstream/main --stage manual`
- `breeze ci selective-check --commit-ref HEAD`
---
Drafted-by: Codex (GPT-5) (no human review before posting)
--
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]
