AY: added IPv6 option (bnc#633310) #270
Conversation
svn path=/trunk/network/; revision=62494 Conflicts: VERSION package/yast2-network.changes src/modules/Lan.ycp
| @@ -255,6 +255,8 @@ define map ToAY (map settings) { | |||
|
|
|||
| map ret = $[]; | |||
| ret["managed"] = settings["managed"]:false; | |||
| if(haskey(settings, "ipv6")) | |||
| ret["ipv6"] = settings["ipv6"]:true; | |||
There was a problem hiding this comment.
so by default is ipv6 enabled. Is it intention and is it documented?
There was a problem hiding this comment.
it is port of already existing implementation. It behaves the same way like in OS > 11.4 and SLE12.
I'll explicitly ask for decision in the fate.
There was a problem hiding this comment.
But the whole point of https://fate.suse.com/316938 is to make an option configurable which currently defaults to true already. If users want to disable it, they will explicitly set so in the profile. For others, nothing changes, right?
|
if it is intention to have it by default to true, then LGTM |
|
I have tested that it works (but only after a reboot, since is is implemented by blacklisting the module; that should be IMHO mentioned in the release notes). |
AY: added IPv6 option (bnc#633310)
fate#316938