Change in vdsm[master]: caps: Fix access to /sys/class/iommu

danken at redhat.com danken at redhat.com
Thu Mar 12 10:18:57 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: caps: Fix access to /sys/class/iommu
......................................................................


caps: Fix access to /sys/class/iommu

This patch fixes OSError during getVdsCapabilities introduced in
https://gerrit.ovirt.org/#/c/37701/

On some systems (such as el6), iommu is not present, causing 
getVdsCapabilities to not return properly. This patch can be reverted when
we no longer want to be able to run on el6.

Change-Id: Ie11f10163984a8bb4cf2e27225d2e19d6cb2f9ae
Signed-off-by: Dima Kuznetsov <dkuznets at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38641
Reviewed-by: Yeela Kaplan <ykaplan at redhat.com>
Reviewed-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/caps.py
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Yeela Kaplan: Looks good to me, but someone else must approve
  Dima Kuznetsov: Verified
  Dan Kenigsberg: Looks good to me, approved
  Martin Polednik: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie11f10163984a8bb4cf2e27225d2e19d6cb2f9ae
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list