-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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
Fixes #13211 linkLocalIPv6FromMac #13226
Conversation
Kudos to gissehel for his great work on investigation and the fabulous bug report! Signed-off-by: Malte Janduda <mail@janduda.net>
d2c58c9
to
f09b789
Compare
In libnetwork we are deprecating this as inspect value. The eth0 interface inside the container will have this linklocal ipv6 address assigned to the interface itself. So if you do |
so I think this needs to be opened on libnetwork now? then pulled in here am i correct |
@jfrazelle according to @mrjana's #13226 (comment) this shouldn't be a problem in Libnetwork. But I'm interested to hear if there's a regression test in place |
cool @MalteJ can you rebase please :) |
@jfrazelle well, it looks like the files I have edited disappeared from master branch 😄 |
@MalteJ is the test case still useful for libnetwork? |
@mrjana can you suggest the course of action for this PR ? |
@MalteJ @thaJeztah @jfrazelle @mavenugo This test is moot because in the libnetwork bridge driver we don't generate the linklocal address from MAC at all. We leave it to the kernel to do it for us because of the way we plumb the interfaces. So this(the generation part) doesn't need to be tested. What may be tested is add an integration-cli to ensure that this is the case when container is brought up and the the container eth0 address has the linklocalFromMac setup correctly. But, we should probably remove the |
Sorry @MalteJ, another one that we seem to have dropped with the big move to libnetwork =/ I'm closing this as my understanding from the above comments is that it's not relevant anymore. |
no problem. I am happy we have the new network stack! |
❤️ |
Kudos to @gissehel for his great work on investigation and the fabulous bug report!
fixes #13211