Change in vdsm[master]: vm: check operation result for vm nic hotunplug

mmirecki at redhat.com mmirecki at redhat.com
Mon Nov 30 14:02:29 UTC 2015


Marcin Mirecki has posted comments on this change.

Change subject: vm: check operation result for vm nic hotunplug
......................................................................


Patch Set 16:

(2 comments)

https://gerrit.ovirt.org/#/c/48473/16//COMMIT_MSG
Commit Message:

Line 9:   
> nit: 2 spaces
Done


https://gerrit.ovirt.org/#/c/48473/16/vdsm/virt/vmdevices/network.py
File vdsm/virt/vmdevices/network.py:

Line 194:         The path is relative to the root element
Line 195:         """
Line 196:         return "./devices/interface/mac[@address='%s']" % self.macAddr
Line 197: 
Line 198:     def _make_name(self):
> Looks like the failing tests are the new waitForDeviceRemoval tests, since 
Removed from this patch completely.
This should belong to the last patch with the nic test (moved initialization to test there)
Line 199:         """
Line 200:         Creates an interface name if one was not set.
Line 201:         There is no guarantee that the interface name will be set in
Line 202:         constructor, so we need this to make sure we have a value for name.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d3b2c839cbb4733aecc8d5a0e9a1ae691e14ac3
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list