[NEW PATCH] Related to BZ#726960 - Unify SD.validate signature. (via gerrit-bot)

Saggi Mizrahi smizrahi at redhat.com
Thu Aug 18 07:08:37 UTC 2011


New patch submitted by Saggi Mizrahi (smizrahi at redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/827

commit 7f39053070b1cbe974d36aa3ed89ffbdf6f2a991
Author: Eduardo Warszawski <ewarszaw at redhat.com>
Date:   Wed Aug 17 16:26:57 2011 +0300

    Related to BZ#726960 - Unify SD.validate signature.
    
    Change-Id: Ie63225e7012e2236a358a89a1f2ccacec0d498fb

diff --git a/vdsm/storage/fileSD.py b/vdsm/storage/fileSD.py
index 39a9ae7..6219303 100644
--- a/vdsm/storage/fileSD.py
+++ b/vdsm/storage/fileSD.py
@@ -169,7 +169,7 @@ class FileStorageDomain(sd.StorageDomain):
                             volUUID, desc, srcImgUUID, srcVolUUID)
 
 
-    def validate(self):
+    def validate(self, useCache=False):
         """
         Validate that the storage domain is accessible.
         """
diff --git a/vdsm/storage/sd.py b/vdsm/storage/sd.py
index 26e19a2..5b3d798 100644
--- a/vdsm/storage/sd.py
+++ b/vdsm/storage/sd.py
@@ -430,12 +430,6 @@ class StorageDomain:
         # do any required cleanup
 
 
-    def validate(self):
-        """
-        Validate that the storage domain is accessible.
-        """
-        pass
-
     # I personally don't think there is a reason to pack these
     # but I already changed too much.
     def changeLeaseParams(self, leaseParamPack):




More information about the vdsm-patches mailing list