Eduardo has uploaded a new change for review.

Change subject: Revert "hsm: always check validateNotSPM when disconnecting 
from a pool"
......................................................................

Revert "hsm: always check validateNotSPM when disconnecting from a pool"

This reverts commit 185872e5e03999ed7376099d1c8b1a4044cfa5bb.

Change-Id: I791f626e31c9929d687d181b2d7fd43e1d2b155d
Signed-off-by: Eduardo <[email protected]>
---
M vdsm/storage/hsm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/26/13926/1

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index ae35b11..f3a702e 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -1026,12 +1026,12 @@
         self.validateNotSPM(spUUID)
 
         vars.task.getExclusiveLock(STORAGE, spUUID)
-        pool = self.getPool(spUUID)
+        self.validateNotSPM(spUUID)
 
+        pool = self.getPool(spUUID)
         return self._disconnectPool(pool, hostID, scsiKey, remove)
 
     def _disconnectPool(self, pool, hostID, scsiKey, remove):
-        self.validateNotSPM(pool.spUUID)
         res = pool.disconnect()
         del self.pools[pool.spUUID]
         return res


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I791f626e31c9929d687d181b2d7fd43e1d2b155d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to