fixes to the error handling as suggested by ekim

pull/129/head
Christopher Usher 5 years ago
parent 553b11bc84
commit dc1c31fef4

@ -286,10 +286,11 @@ class BackfillerManager(object):
# ignore error when file is already deleted # ignore error when file is already deleted
if e.errno == errno.ENOENT: if e.errno == errno.ENOENT:
self.logger.warn('{} already deleted'.format(path)) self.logger.warn('{} already deleted'.format(path))
# warn if not empty (will try to delete folder again next time) # warn if not empty (will try to delete folder again next time)
elif e.errno == errno.ENOTEMPTY: elif e.errno == errno.ENOTEMPTY:
self.logger.warn('Failed to delete non-empty folder {}'.format(path)) self.logger.warn('Failed to delete non-empty folder {}'.format(path))
else:
raise
else: else:
self.logger.info('{} deleted'.format(path)) self.logger.info('{} deleted'.format(path))
@ -331,7 +332,7 @@ class BackfillerManager(object):
if self.run_once: if self.run_once:
break break
# if get_nodes() raises an error, this will deletes will not occur # note that if get_nodes() raises an error, then deletes will not occur
if self.delete_old and self.start: if self.delete_old and self.start:
try: try:
self.delete_hours() self.delete_hours()

Loading…
Cancel
Save