From 7a913b34d4877d75f70ed4dae46e898f41d0ee50 Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Tue, 27 Aug 2019 14:49:17 -0500 Subject: [PATCH] check_source: check for target package not found when checking out. When new packages are submitted the target package should not exist and thus does not constitute an error. Any other return code does. --- check_source.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/check_source.py b/check_source.py index 6376af26..1cffe324 100755 --- a/check_source.py +++ b/check_source.py @@ -126,8 +126,11 @@ class CheckSource(ReviewBot.ReviewBot): shutil.rmtree(os.path.join(target_package, '.osc')) os.rename(target_package, '_old') old_info = self.package_source_parse(target_project, target_package) - except HTTPError: - self.logger.error('failed to checkout %s/%s' % (target_project, target_package)) + except HTTPError as e: + if e.code == 404: + self.logger.info('target package does not exist %s/%s' % (target_project, target_package)) + else: + raise e CheckSource.checkout_package(self.apiurl, source_project, source_package, revision=source_revision, pathname=dir, server_service_files=True, expand_link=True)