Skip to content

[cilium-envoy] Make affinity and updateStrategy extensible#42359

Merged
marseel merged 1 commit intocilium:mainfrom
nezdolik:cilium-envoy-extensions
Oct 24, 2025
Merged

[cilium-envoy] Make affinity and updateStrategy extensible#42359
marseel merged 1 commit intocilium:mainfrom
nezdolik:cilium-envoy-extensions

Conversation

@nezdolik
Copy link
Copy Markdown
Contributor

@nezdolik nezdolik commented Oct 23, 2025

This change makes it possible to extend affinity and updateStrategy properties for Envoy daemonset. While users can now configure those values via helm chart there is no possibility to configure conditional values for those settings (e.g. if property X is enabled configure affinity with A and if X is disabled configure affinity with B).

Fixes: #issue-number

<!-- Enter the release note text here if needed or remove this section! -->

@nezdolik nezdolik requested review from a team as code owners October 23, 2025 13:05
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 23, 2025
@nezdolik nezdolik force-pushed the cilium-envoy-extensions branch 2 times, most recently from 51fdc2c to 47404e0 Compare October 23, 2025 13:08
@viktor-kurchenko
Copy link
Copy Markdown
Contributor

/test

@viktor-kurchenko viktor-kurchenko added area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. release-note/misc This PR makes changes that have no direct user impact. labels Oct 23, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 23, 2025
Signed-off-by: Kateryna Nezdolii <kateryna.nezdolii@isovalent.com>
@nezdolik nezdolik force-pushed the cilium-envoy-extensions branch from 47404e0 to 83ec0f1 Compare October 23, 2025 15:07
@viktor-kurchenko
Copy link
Copy Markdown
Contributor

/test

@marseel marseel enabled auto-merge October 24, 2025 08:45
@sayboras sayboras removed the request for review from jrajahalme October 24, 2025 09:32
@marseel marseel added this pull request to the merge queue Oct 24, 2025
Merged via the queue into cilium:main with commit d44238a Oct 24, 2025
75 checks passed
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Oct 24, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Oct 24, 2025
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
nezdolik added a commit to nezdolik/community that referenced this pull request Mar 13, 2026
nezdolik added a commit to nezdolik/community that referenced this pull request Mar 13, 2026
nezdolik added a commit to nezdolik/community that referenced this pull request Mar 13, 2026
joestringer pushed a commit to cilium/community that referenced this pull request Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

6 participants