Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change loadbalancer type for HCP deployments #3307

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

aswinsuryan
Copy link
Contributor

The Kubevirt(HCP) clusters do not have their loadbalancer installedand rely on the host loadblancer( like netallb) when a loadbalancer is created.So when the Submariner exposes the submariner-gateway loadbalancer service with ServiceExternalTrafficPolicy locally does not work, as the traffic from the remotecluster first lands on the host cluster. Hence the traffic policy is updated to clusterin these deployments.

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr3307/aswinsuryan/hcp-lb
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@yboaron yboaron added the ready-to-test When a PR is ready for full E2E testing label Dec 23, 2024
@aswinsuryan aswinsuryan force-pushed the hcp-lb branch 3 times, most recently from 8c5b9b3 to 99bab08 Compare January 10, 2025 19:18
@aswinsuryan aswinsuryan force-pushed the hcp-lb branch 2 times, most recently from 2f7ce87 to 0597df2 Compare January 13, 2025 22:14
The Kubevirt(HCP) clusters do not have their loadbalancer installed and rely
on the host loadblancer( like netallb) when a loadbalancer is created.So when
the Submariner exposes the submariner-gateway loadbalancer service with
ServiceExternalTrafficPolicy locally does not work, as the traffic from
the remotecluster first lands on the host cluster. Hence the traffic policy
need to be updated to cluster in these deployments.

A new parameter hostedCluster is added to submariner config, if set the
traffic policy will be set to cluster

Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
@yboaron yboaron merged commit f14c74e into submariner-io:devel Jan 15, 2025
36 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr3307/aswinsuryan/hcp-lb]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants