scripts: Make default mode of cache-cleaning scripts much less verbose.

Print a detailed report only if `--verbose` flag is specified.

Fixes: #6632.
This commit is contained in:
Harshit Bansal
2017-09-23 19:15:13 +00:00
committed by Tim Abbott
parent 480ae626fb
commit df7ea375c1
2 changed files with 10 additions and 6 deletions

View File

@@ -237,7 +237,8 @@ def purge_unused_caches(caches_dir, caches_in_use, args, cache_type):
caches_to_purge = get_caches_to_be_purged(caches_dir, caches_in_use, args.threshold_days)
caches_to_keep = all_caches - caches_to_purge
may_be_perform_purging(caches_to_purge, caches_to_keep, cache_type, args.dry_run)
may_be_perform_purging(
caches_to_purge, caches_to_keep, cache_type, args.dry_run, args.verbose)
print("Done!\n")
def generate_sha1sum_emoji(zulip_path):
@@ -263,17 +264,19 @@ def generate_sha1sum_emoji(zulip_path):
return sha.hexdigest()
def may_be_perform_purging(dirs_to_purge, dirs_to_keep, dir_type, dry_run):
# type: (Set[Text], Set[Text], Text, bool) -> None
def may_be_perform_purging(dirs_to_purge, dirs_to_keep, dir_type, dry_run, verbose):
# type: (Set[Text], Set[Text], Text, bool, bool) -> None
if dry_run:
print("Performing a dry run...")
else:
print("Cleaning unused %ss..." % (dir_type,))
for directory in dirs_to_purge:
print("Cleaning unused %s: %s" % (dir_type, directory))
if verbose:
print("Cleaning unused %s: %s" % (dir_type, directory))
if not dry_run:
subprocess.check_call(["sudo", "rm", "-rf", directory])
for directory in dirs_to_keep:
print("Keeping used %s: %s" % (dir_type, directory))
if verbose:
print("Keeping used %s: %s" % (dir_type, directory))