Change in vdsm[master]: hostdev: destroy VM with hostdev even without IOMMU support

danken at redhat.com danken at redhat.com
Thu Jun 4 12:37:52 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: destroy VM with hostdev even without IOMMU support
......................................................................


hostdev: destroy VM with hostdev even without IOMMU support

Such VM can occur if it is started via vdsClient. Any attempt to
destroy that VM would fail with exception without destroying the VM,
therefore leaving it hanging in vdsm. This patch logs if reattach of
hostdev without valid IOMMU configuration occurs and allows destroy to
finish.

Change-Id: I857ec18c0932de247469f6fb780d1b28aa3873e8
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41886
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/hostdev.py
M vdsm/virt/vm.py
2 files changed, 9 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve
  Martin Polednik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I857ec18c0932de247469f6fb780d1b28aa3873e8
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan 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: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list