]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph-daemon: use 'iglob' instead of 'glob'
authorMichael Fritch <mfritch@suse.com>
Sun, 24 Nov 2019 01:05:40 +0000 (18:05 -0700)
committerMichael Fritch <mfritch@suse.com>
Mon, 25 Nov 2019 23:45:14 +0000 (16:45 -0700)
Use a glob iterator for instances where the matched pathnames return a
very long list of files.

Signed-off-by: Michael Fritch <mfritch@suse.com>
src/ceph-daemon/ceph-daemon

index dabfd6a8522263373d0345c1b98ae5f03ad119e1..2fccc9d5a6945f1c82df0823aaf6ee17151c203b 100755 (executable)
@@ -61,7 +61,7 @@ import uuid
 
 from distutils.spawn import find_executable
 from functools import wraps
-from glob import glob
+from glob import iglob
 
 
 container_path = None
@@ -1515,7 +1515,7 @@ def command_adopt():
                         (daemon_type, args.cluster, daemon_id))
         data_dir_src = os.path.abspath(args.legacy_dir + data_dir_src)
         data_dir_dst = make_data_dir(fsid, daemon_type, daemon_id)
-        for data_file in glob(os.path.join(data_dir_src, '*')):
+        for data_file in iglob(os.path.join(data_dir_src, '*')):
             move_file(data_file, data_dir_dst)
         logger.debug('Remove dir \'%s\'' % (data_dir_src))
         if os.path.ismount(data_dir_src):
@@ -1535,7 +1535,7 @@ def command_adopt():
                         (args.cluster, daemon_type, daemon_id))
         log_dir_src = os.path.abspath(args.legacy_dir + log_dir_src)
         log_dir_dst = make_log_dir(fsid, uid=uid, gid=gid)
-        for log_file in glob(log_dir_src):
+        for log_file in iglob(log_dir_src):
             move_file(log_file, log_dir_dst)
 
         logger.info('Creating new units...')