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

Use cloud-provider-kind instead of MetalLB #1622

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

skitt
Copy link
Member

@skitt skitt commented Apr 18, 2024

kind has a cloud-provider which can take care of LoadBalancer-type services; this works fine with LB-mode Submariner, so use that instead of MetalLB.

Currently the command is installed as cmd, which is why it is renamed during container construction. This is fixed for the next release of the cloud-provider.

cloud-provider-kind runs in the foreground, so this wraps it in a container run alongside the kind clusters, on the host network. The container is stopped on cleanup.

The dependency version will be tracked using dependabot.

kind has a cloud-provider which can take care of LoadBalancer-type
services; this works fine with LB-mode Submariner, so use that instead
of MetalLB.

Currently the command is installed as cmd, which is why it is renamed
during container construction. This is fixed for the next release of
the cloud-provider.

cloud-provider-kind runs in the foreground, so this wraps it in a
container run alongside the kind clusters, on the host network. The
container is stopped on cleanup.

The dependency version will be tracked using dependabot.

Signed-off-by: Stephen Kitt <skitt@redhat.com>
@submariner-bot
Copy link

🤖 Created branch: z_pr1622/skitt/cloud-provider-kind

Copy link
Member

@dfarrell07 dfarrell07 left a comment

Choose a reason for hiding this comment

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

Nice!

@skitt skitt marked this pull request as draft April 18, 2024 14:36
@skitt
Copy link
Member Author

skitt commented Apr 18, 2024

Switching to draft because this leaves haproxy containers running (and shouldn’t).

Copy link

github-actions bot commented May 3, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further
activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label May 3, 2024
@dfarrell07 dfarrell07 removed the stale label May 7, 2024
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further
activity occurs. Thank you for your contributions.

@yboaron
Copy link
Contributor

yboaron commented Jun 4, 2024

I think we should consider keeping MetalLB as well since it is the default LB provider for baremetal. Do you think we can support both methods?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

None yet

5 participants