Change in vdsm[master]: tmp

laravot at redhat.com laravot at redhat.com
Tue Sep 8 13:48:02 UTC 2015


Liron Aravot has uploaded a new change for review.

Change subject: tmp
......................................................................

tmp

Change-Id: I1d5b65a75b1b50d5f5991334cf6221c067a31f5b
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M vdsm/storage/hsm.py
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/66/45866/1

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index 5583f76..7050fcf 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -3512,7 +3512,11 @@
     def upgradeStoragePool(self, spUUID, targetDomVersion):
         targetDomVersion = int(targetDomVersion)
         pool = self.getPool(spUUID)
-        pool._upgradePool(targetDomVersion)
+        # This lock has to be mutual with the pool metadata operations (like
+        # activateSD/deactivateSD) as it uses the pool metadata.
+        with rmanager.acquireResource(STORAGE, spUUID,
+                                      rm.LockType.exclusive):
+            pool._upgradePool(targetDomVersion)
         return {"upgradeStatus": "started"}
 
     def _getDomsStats(self, domainMonitor, doms):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d5b65a75b1b50d5f5991334cf6221c067a31f5b
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>


More information about the vdsm-patches mailing list