From b848458251c7a0e2bdcf89bc41aeb5bd3e6b0455 Mon Sep 17 00:00:00 2001 From: Mike McLean Date: Oct 30 2017 16:09:04 +0000 Subject: safe_rmtree: make return value consistent with past behavior --- diff --git a/koji/tasks.py b/koji/tasks.py index e5824b1..b8ad060 100644 --- a/koji/tasks.py +++ b/koji/tasks.py @@ -78,10 +78,11 @@ def safe_rmtree(path, unmount=False, strict=True): raise else: logger.warn("Error removing: %s", exc_info=True) - return + return 1 + return 0 if not os.path.exists(path): logger.debug("No such path: %s" % path) - return + return 0 logger.debug('Scrubbing files in %s' % path) try: @@ -91,6 +92,7 @@ def safe_rmtree(path, unmount=False, strict=True): if strict: raise return 1 + return 0 class ServerExit(Exception):