-
Notifications
You must be signed in to change notification settings - Fork 162
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
[WIP] Stable IPv6 addresses #2227
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems to be sitting on top of that other questionable PR. Any chance we can split those? (so that IPv6 stuff gets in separately) @eriknordmark ?
4f20c90
to
84c913c
Compare
That was why it was a WIP/Draft. Cleaned it up. But need some tweak because the IPv6 addresses still change on EVE-OS reboot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Any reason not to merge this @eriknordmark ?
Yes, because it isn't sufficient. IPv6 addresses still change on every device reboot. |
Signed-off-by: eriknordmark <[email protected]>
84c913c
to
f1d5d13
Compare
Signed-off-by: eriknordmark <[email protected]>
Hey @eriknordmark -- this has been open for quite a long time now -- any updates on what do you plan to do with this? |
@@ -128,14 +131,48 @@ func addBridge(log *base.LogObject, ifname string) error { | |||
log.Error(err) | |||
return err | |||
} | |||
// update cached ifindex | |||
// updatecached ifindex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you intend to remove the space between "update" and "cached"?
@eriknordmark Erik, I assume we need to set 'addr_gen_mode = 2' as well:
|
We switched to RFC7217 but are not setting a secret hence the IPv6 addresses differ on each boot.