Skip to content

feat(kubernetes): Allow adding custom labels to worker pods#4526

Merged
mnonnenmacher merged 1 commit intomainfrom
worker-labels
Feb 19, 2026
Merged

feat(kubernetes): Allow adding custom labels to worker pods#4526
mnonnenmacher merged 1 commit intomainfrom
worker-labels

Conversation

@mnonnenmacher
Copy link
Contributor

Add a config property for the KubernetesSenderConfig to allow adding custom labels for the created jobs.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: hte -> the

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it is just me, but the exact meaning of this sentence is not clear to me. It is supposed to mean that the reserved labels are ignored in the map, but my first interpretation was that they will not be added (i.e., the adding is ignored) if they are included in the map. To make this fully clear, maybe use something like "... are added by default and cannot be overridden via this mechanism." ? Also in other places.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have reworded the sentences, it should be clearer now.

Add a config property for the `KubernetesSenderConfig` to allow adding
custom labels for the created jobs.

Signed-off-by: Martin Nonnenmacher <martin.nonnenmacher@doubleopen.org>
@mnonnenmacher mnonnenmacher added this pull request to the merge queue Feb 19, 2026
Merged via the queue into main with commit 978eb82 Feb 19, 2026
34 checks passed
@mnonnenmacher mnonnenmacher deleted the worker-labels branch February 19, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments