Francesco Romani has submitted this change and it was merged.

Change subject: config: Resolve crash when IPv6 disabled in kernel
......................................................................


config: Resolve crash when IPv6 disabled in kernel

The configuration for creating the management bridge attempts to do some
things with IPv6 which causes an uncaught exception when IPv6 is
disabled at the kernel level with `ipv6.disable=1`.

This patch checks that IPv6 is actually available before attempting to
do anything with it.

Change-Id: I774cac868ddee08ac072adf2104a811594779052
Bug-Url: https://bugzilla.redhat.com/1277204
Backport-To: 3.6
Signed-off-by: Ollie Armstrong <ol...@fubra.com>
Signed-off-by: Ondřej Svoboda <osvob...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/47978
Continuous-Integration: Jenkins CI
Reviewed-by: Ido Barkan <ibar...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/50457
Reviewed-by: Francesco Romani <from...@redhat.com>
---
M lib/vdsm/netinfo.py
M vdsm/network/configurators/ifcfg.py
M vdsm/network/configurators/iproute2.py
3 files changed, 14 insertions(+), 2 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Ondřej Svoboda: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I774cac868ddee08ac072adf2104a811594779052
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Ondřej Svoboda <osvob...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to