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

[keepalived-vip] Should allow for multiple NICs #1

Open
shift opened this issue May 16, 2017 · 0 comments
Open

[keepalived-vip] Should allow for multiple NICs #1

shift opened this issue May 16, 2017 · 0 comments

Comments

@shift
Copy link
Owner

shift commented May 16, 2017

The current implementation doesn't have support for multiple network interfaces. It's more of a first one wins.

A better implementation would be to use labels on the nodes to signify which is the internal and which is the external interfaces.

The internal interface is the one which the VRRP keepalive/heartbeat is used for.
The external interface is the one which the VIP is assigned too.

I'm currently swaying towards using a labels like contrib.kubernetes.io/keepalived-vip/internal and contrib.kubernetes.io/keepalived-vip/external. And if present to not bother with the current interface code and instead to use those values in the keepalived.conf generated.

Even with RBAC enabled the default example policy allows access to the nodes resource, so no other changes should be required on the deployment side of things.

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

No branches or pull requests

1 participant