Nir Soffer 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 <mmire...@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 <nsof...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Francesco Romani <from...@redhat.com> --- 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: Nir Soffer: Looks good to me, approved Marcin Mirecki: Verified Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48473 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d3b2c839cbb4733aecc8d5a0e9a1ae691e14ac3 Gerrit-PatchSet: 21 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmire...@redhat.com> Gerrit-Reviewer: Amit Aviram <aavi...@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: Marcin Mirecki <mmire...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@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