Dan Kenigsberg has posted comments on this change.

Change subject: networkTests: stop NetworkManager from managing ifaces using 
NM_CONTROLLED=no
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.ovirt.org/#/c/37037/3/tests/functional/networkmanager.py
File tests/functional/networkmanager.py:

Line 49:         ifcfg.write('DEVICE={0}\n'
Line 50:                     'NM_CONTROLLED=no\n'.format(iface))
Line 51:     rc, _, _ = execCmd([_NM_CLI_BINARY.cmd, 'conn', 'load', 
ifcfg_path])
Line 52:     if rc:
Line 53:         raise SkipTest('Failed to unmanage ' + iface)
please include out and err to make the debgger life easier.
Line 54: 
Line 55: 
Line 56: def _restore(iface):
Line 57:     ifcfg_path = netinfo.NET_CONF_PREF + iface


Line 54: 
Line 55: 
Line 56: def _restore(iface):
Line 57:     ifcfg_path = netinfo.NET_CONF_PREF + iface
Line 58:     execCmd([_NM_CLI_BINARY.cmd, 'conn', 'delete', iface])
check rc and write out/err to log in case of error.


-- 
To view, visit http://gerrit.ovirt.org/37037
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icc8ea9e0b0d410203ed3045e735c9450eee316ae
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvob...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvob...@redhat.com>
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to