From a18c6d9278440292f5661fcacffd97dc82cfa90d Mon Sep 17 00:00:00 2001 From: Christopher Usher Date: Wed, 30 Oct 2019 22:22:28 +0000 Subject: [PATCH] error handling changes as suggested by ekim --- segment_coverage/segment_coverage/main.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/segment_coverage/segment_coverage/main.py b/segment_coverage/segment_coverage/main.py index b3fa57a..f83a3bb 100644 --- a/segment_coverage/segment_coverage/main.py +++ b/segment_coverage/segment_coverage/main.py @@ -193,7 +193,7 @@ class CoverageChecker(object): except OSError as e: if e.errno == errno.ENOENT: self.logger.warning('{} does not exist'.format(path)) - break + continue hours.sort() previous_hour_segments = None @@ -211,8 +211,8 @@ class CoverageChecker(object): segment_names = [name for name in os.listdir(hour_path) if not name.startswith('.')] except OSError as e: if e.errno == errno.ENOENT: - self.logger.warning('{} does not exist'.format(hour_path)) - break + self.logger.warning('Hour {} was deleted between finding it and processing it, ignoring'.format(hour)) + continue segment_names.sort() parsed = [] bad_segment_count = 0