Change in vdsm[master]: Symmetrize *StorageDomain.*activateVolumes().

ewarszaw at redhat.com ewarszaw at redhat.com
Thu Jul 4 07:12:36 UTC 2013


Eduardo has uploaded a new change for review.

Change subject: Symmetrize *StorageDomain.*activateVolumes().
......................................................................

Symmetrize *StorageDomain.*activateVolumes().

Related to BZ#960952, BZ#769502.

Change-Id: Id182158d54df75c712222061b005be97fc933529
Signed-off-by: Eduardo <ewarszaw at redhat.com>
---
M vdsm/storage/fileSD.py
M vdsm/storage/sd.py
2 files changed, 6 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/25/16425/1

diff --git a/vdsm/storage/fileSD.py b/vdsm/storage/fileSD.py
index dcc6baf..6d8d356 100644
--- a/vdsm/storage/fileSD.py
+++ b/vdsm/storage/fileSD.py
@@ -421,6 +421,12 @@
         for vol in [self.produceVolume(imgUUID, x) for x in volUUIDs]:
             self.oop.fileUtils.copyUserModeToGroup(vol.getVolumePath())
 
+    def deactivateVolumes(self, imgUUID, volUUIDs):
+        """
+        Deactivate all the volumes listed in volUUIDs
+        """
+        pass
+
     @classmethod
     def format(cls, sdUUID):
         """
diff --git a/vdsm/storage/sd.py b/vdsm/storage/sd.py
index fd79059..36c4877 100644
--- a/vdsm/storage/sd.py
+++ b/vdsm/storage/sd.py
@@ -631,18 +631,6 @@
             self.mountMaster()
             self.createMasterTree()
 
-    def activateVolumes(self, imgUUID, volUUIDs):
-        """
-        Activate all the volumes listed in volUUIDs
-        """
-        pass
-
-    def deactivateVolumes(self, imgUUID, volUUIDs):
-        """
-        Deactivate all the volumes listed in volUUIDs
-        """
-        pass
-
     def _getRepoPath(self):
         # This is here to make sure no one tries to get a repo
         # path from an ISO domain.


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id182158d54df75c712222061b005be97fc933529
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>


More information about the vdsm-patches mailing list