Change in vdsm[master]: Fix detachStorageDomain for the ISO domains

ilvovsky at redhat.com ilvovsky at redhat.com
Thu Jul 19 07:52:29 UTC 2012


Igor Lvovsky has submitted this change and it was merged.

Change subject: Fix detachStorageDomain for the ISO domains
......................................................................


Fix detachStorageDomain for the ISO domains

When detaching an ISO domain we should acquire the host id before
trying to acquire the cluster lock.
Additionally since an ISO domain can be used by multiple pools we
should avoid to acquire the host id in the domain monitor.

In this patch:
* acquire the host id in detachSD for the ISO domains
* do not acquire the host id in the domain monitor for the ISO domains

Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Change-Id: Ifdf531f99d3d7304139def10fc6fd3ee4eedda28
---
M vdsm/storage/domainMonitor.py
M vdsm/storage/sp.py
2 files changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Moran Goldboim: Verified
  Igor Lvovsky: 


--
To view, visit http://gerrit.ovirt.org/6445
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifdf531f99d3d7304139def10fc6fd3ee4eedda28
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list