Change in vdsm[master]: fileSD: Move private constant near the only call site

nsoffer at redhat.com nsoffer at redhat.com
Tue Sep 9 22:00:20 UTC 2014


Nir Soffer has uploaded a new change for review.

Change subject: fileSD: Move private constant near the only call site
......................................................................

fileSD: Move private constant near the only call site

Keeping a private constant used by only one function at the top of the
module does not help anyone. The best place for it is just above the
code using it.

Change-Id: If8430a05dd454c34217bb051345633aec4351738
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M vdsm/storage/fileSD.py
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/24/32724/1

diff --git a/vdsm/storage/fileSD.py b/vdsm/storage/fileSD.py
index 8c2c550..9a22b66 100644
--- a/vdsm/storage/fileSD.py
+++ b/vdsm/storage/fileSD.py
@@ -57,8 +57,6 @@
 # Specific stat(2) block size as defined in the man page
 ST_BYTES_PER_BLOCK = 512
 
-_MOUNTLIST_IGNORE = ('/' + sd.BLOCKSD_DIR, '/' + sd.GLUSTERSD_DIR)
-
 getProcPool = oop.getGlobalProcPool
 
 
@@ -641,6 +639,9 @@
                 self.oop.utils.forceLink(tVol, tLink)
 
 
+_MOUNTLIST_IGNORE = ('/' + sd.BLOCKSD_DIR, '/' + sd.GLUSTERSD_DIR)
+
+
 def _getMountsList(pattern="*"):
     fileDomPattern = os.path.join(
         sd.StorageDomain.storage_repository, sd.DOMAIN_MNT_POINT,


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If8430a05dd454c34217bb051345633aec4351738
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list