Change in vdsm[master]: domain: use shared lock for extendStorageDomain

danken at redhat.com danken at redhat.com
Mon Jan 21 15:28:18 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: domain: use shared lock for extendStorageDomain
......................................................................


domain: use shared lock for extendStorageDomain

The extendStorageDomain command shouldn't hold an exclusive lock on
the storage domain since it can be executed also during other long
tasks (e.g.: moveImage).

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=876663
Change-Id: I2ea73b76b4db010a6eacd80c8328afe66e7cdc51
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2ea73b76b4db010a6eacd80c8328afe66e7cdc51
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: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list