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

Jump to bottom

spec.loadBalancerIP always fail for ipv6 #301

Closed
uablrek opened this issue Aug 14, 2018 · 2 comments · Fixed by #313
Closed

spec.loadBalancerIP always fail for ipv6 #301

uablrek opened this issue Aug 14, 2018 · 2 comments · Fixed by #313
Assignees
@danderson
Labels
bug

Comments

@uablrek
Copy link
Contributor

uablrek commented Aug 14, 2018

An unnecessary To4() here causes specifying a loadBalancerIP to fail always for ipv6.

metallb version: v0.7.2
@uablrek
Copy link
Contributor Author

uablrek commented Aug 14, 2018

No rush there is an issue in k8s as well (sigh...).

@danderson danderson added the bug label Aug 14, 2018
@danderson danderson self-assigned this Aug 14, 2018
@danderson
Copy link
Contributor

Thanks for the report! As you can tell, ipv6 support is not tested right now :(. I'm working on it, but I expect more gremlins like this are hiding in the code...

uablrek pushed a commit to Nordix/metallb that referenced this issue Sep 14, 2018
Fix for issue metallb#301
bd2fbc6 
Remove a check for ipv4 to allow also ipv6 addresses.
@uablrek uablrek mentioned this issue Sep 14, 2018
Merged
danderson pushed a commit that referenced this issue Sep 26, 2018
@danderson
Fix for issue #301
0677136 
Remove a check for ipv4 to allow also ipv6 addresses.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danderson danderson

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix for issue #301 Nordix/metallb
2 participants
@danderson @uablrek