[NEW PATCH] Avoid to do lvm._bootstrap when importing lvm module. (via gerrit-bot)

Dan Kenigsberg danken at redhat.com
Mon Aug 8 12:10:15 UTC 2011


New patch submitted by Dan Kenigsberg (danken at redhat.com)

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

commit 1555a9f9c6bfca8f764284cd6d6804d796cd323d
Author: Eduardo Warszawski <ewarszaw at redhat.com>
Date:   Tue Aug 2 11:54:42 2011 +0300

    Avoid to do lvm._bootstrap when importing lvm module.
    
    Change-Id: I5012016f9b9d0fb1096e30b5d384d22c0e04e5eb

diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py
index d9d8e21..42d5a5c 100644
--- a/vdsm/storage/hsm.py
+++ b/vdsm/storage/hsm.py
@@ -149,6 +149,7 @@ class HSM:
             multipath.setupMultipath()
 
         self.__validateLvmLockingType()
+        lvm._lvminfo.bootstrap()
 
         def storageRefresh():
             SDF.refreshStorage()
diff --git a/vdsm/storage/lvm.py b/vdsm/storage/lvm.py
index 51e1dcf..ca32231 100644
--- a/vdsm/storage/lvm.py
+++ b/vdsm/storage/lvm.py
@@ -367,7 +367,6 @@ class LVMCache(object):
         self._pvs = {}
         self._vgs = {}
         self._lvs = {}
-        self._bootstrap()
 
     def cmd(self, cmd):
         finalCmd = self._addExtraCfg(cmd)
@@ -393,7 +392,7 @@ class LVMCache(object):
             pp.pformat(self._lvs)))
 
 
-    def _bootstrap(self):
+    def bootstrap(self):
         self._reloadpvs()
         self._reloadvgs()
         self._reloadAllLvs()




More information about the vdsm-patches mailing list