Applying flake8 rules to pp_lib/quota_check.py

diff --git a/pp_lib/quota_check.py b/pp_lib/quota_check.py
index 07262033cd22a2513d353a349fc7114e8ee42858..4db1ee81a6d5dae3ffc195aca4638b995c225b9e 100644 (file)
--- a/pp_lib/quota_check.py
+++ b/pp_lib/quota_check.py
@@ -23,6 +23,7 @@ import gzip
 import shutil
 import time
 import locale
+import sys
 
 from subprocess import Popen, PIPE
 
@@ -35,7 +36,7 @@ from .common import pp, to_str
 
 from .homes_admin import PpHomesAdminError, PpHomesAdminApp
 
-__version__ = '0.6.2'
+__version__ = '0.6.3'
 LOG = logging.getLogger(__name__)
 ZERO = datetime.timedelta(0)
 
@@ -479,9 +480,7 @@ class PpQuotaCheckApp(PpHomesAdminApp):
         return result
 
     # -------------------------------------------------------------------------
-    def check_home_utilization(self):
-
-        LOG.info("Checking utilization of home directories ...")
+    def _silly_open_homedirs(self):
 
         # Senseless opening of all user home directories to activate automounter
         for user_name in self.passwd_data:
@@ -491,7 +490,8 @@ class PpQuotaCheckApp(PpHomesAdminApp):
                 continue
             if not pwd_home_dir.startswith(self.home_root_abs):
                 if self.verbose > 2:
-                    LOG.debug("Home dir {!r} is not below {!r}.".format(pwd_home_dir, self.home_root_abs))
+                    LOG.debug("Home dir {!r} is not below {!r}.".format(
+                        pwd_home_dir, self.home_root_abs))
                 continue
             abs_home_dir = os.path.join(self.chroot_homedir, os.path.relpath(pwd_home_dir, os.sep))
             LOG.debug("Trying to open {!r} ...".format(abs_home_dir))
@@ -499,9 +499,16 @@ class PpQuotaCheckApp(PpHomesAdminApp):
                 os.listdir(abs_home_dir)
                 if self.verbose > 2:
                     LOG.debug("Found home directory {!r} ...".format(abs_home_dir))
-            except OSError as e:
+            except OSError:
                 LOG.warn("Directory {!r} does not exists.".format(abs_home_dir))
 
+    # -------------------------------------------------------------------------
+    def check_home_utilization(self):
+
+        LOG.info("Checking utilization of home directories ...")
+
+        self._silly_open_homedirs()
+
         glob_pattern = os.path.join(self.home_root_real, '*')
         all_home_entries = glob.glob(glob_pattern)