Change in vdsm[master]: ifcfg: re-enable IPv6 before device configuration, or disabl...

danken at redhat.com danken at redhat.com
Sun Mar 27 14:11:23 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ifcfg: re-enable IPv6 before device configuration, or disable afterwards
......................................................................


ifcfg: re-enable IPv6 before device configuration, or disable afterwards

If a network has been restored with no IPv6 requested, all its devices
have disable_ipv6 set to 1 and up to now there was no other way to
re-enable IPv6 than doing so manually or rebooting.

This patch enables IPv6 on the top-level device ("the network") prior
to ifup, sets the underlying devices' disable_ipv6 to 1 as they are
not directly involved in communication. If IPv6 is not requested, it
is disabled on all devices after ifup.

Change-Id: Idddfb096e6ea384dbe6655c5c4178d4884a8db85
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/54555
Continuous-Integration: Jenkins CI
Reviewed-by: Edward Haas <edwardh at redhat.com>
Tested-by: Edward Haas <edwardh at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/network/configurators/ifcfg.py
M tests/functional/networkTests.py
2 files changed, 80 insertions(+), 26 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Edward Haas: Verified; Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/54555
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idddfb096e6ea384dbe6655c5c4178d4884a8db85
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list