Change in vdsm[ovirt-3.6]: vm: check operation result for vm nic hotunplug

fromani at redhat.com fromani at redhat.com
Mon Dec 14 08:45:24 UTC 2015


Francesco Romani has submitted this change and it was merged.

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


vm: check operation result for vm nic hotunplug

After detaching a device we need to verify that this device
has actually been detached.

This mechanism was already implemented for disks. This
patch generalizes the disk waiting mechanism so that it
can also be applied to nics.

This patch is similar to:
https://gerrit.ovirt.org/#/c/45138
https://bugzilla.redhat.com/1044466

Change-Id: I3d3b2c839cbb4733aecc8d5a0e9a1ae691e14ac3
Signed-off-by: Marcin Mirecki <mmirecki at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1134256
Bug-Url: https://bugzilla.redhat.com/1199782
Reviewed-on: https://gerrit.ovirt.org/48473
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/49669
---
M tests/vmTests.py
M vdsm/virt/vm.py
M vdsm/virt/vmdevices/network.py
M vdsm/virt/vmdevices/storage.py
4 files changed, 43 insertions(+), 20 deletions(-)

Approvals:
  Marcin Mirecki: Verified
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d3b2c839cbb4733aecc8d5a0e9a1ae691e14ac3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list