Skip to content

Fix cluster non default additional service when lb enabled#2137

Open
AndrewChubatiuk wants to merge 2 commits intomasterfrom
fix-cluster-non-default-additional-service-when-lb-enabled
Open

Fix cluster non default additional service when lb enabled#2137
AndrewChubatiuk wants to merge 2 commits intomasterfrom
fix-cluster-non-default-additional-service-when-lb-enabled

Conversation

@AndrewChubatiuk
Copy link
Copy Markdown
Contributor

@AndrewChubatiuk AndrewChubatiuk commented May 5, 2026

fixed infinite recreation of non-default additional select/insert service when requestsLoadBalancer.enabled: true


Summary by cubic

Fixes infinite recreation of non-default additional Select/Insert Services across VMCluster, VLCluster, and VTCluster when requestsLoadBalancer.enabled is true and useAsDefault: false. Aligns LB proxy selectors and service generation to avoid name clashes and reconcile loops.

  • Bug Fixes
    • Additional Services now derive from a component-prefixed base Service (cr.PrefixedName(...)) to avoid name collisions.
    • In LB mode, orphan cleanup uses the component-prefixed base, preserving the correct additional Service.
    • LB proxy Services use r.SelectorLabels for the balancer selector across clusters.

Written for commit 660e583. Summary will update on new commits.

@AndrewChubatiuk AndrewChubatiuk requested a review from vrutkovs as a code owner May 5, 2026 14:48
@AndrewChubatiuk AndrewChubatiuk force-pushed the fix-cluster-non-default-additional-service-when-lb-enabled branch 2 times, most recently from bbcf893 to bb9da70 Compare May 5, 2026 14:53
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 11 files

@AndrewChubatiuk AndrewChubatiuk force-pushed the fix-cluster-non-default-additional-service-when-lb-enabled branch from bb9da70 to 8392e16 Compare May 5, 2026 19:37
…-when-lb-enabled

Signed-off-by: Vadim Rutkovsky <vadim@vrutkovs.eu>
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.

2 participants