From 718cb7748a4e613b4c6099f6326c0c45269aa792 Mon Sep 17 00:00:00 2001 From: Fabian Vogt Date: Fri, 31 May 2024 09:47:34 +0200 Subject: [PATCH] ttm: Print all failing packages before exiting Don't exit early but print all known issues instead. --- ttm/releaser.py | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/ttm/releaser.py b/ttm/releaser.py index eba80817d..0e48341ba 100644 --- a/ttm/releaser.py +++ b/ttm/releaser.py @@ -200,22 +200,24 @@ def is_snapshotable(self): if not self.all_repos_done(self.project.name): return False + all_ok = True + resultxml = self.api.retried_GET(self.api.makeurl(['build', self.project.name, '_result'])) prjresult = ET.parse(resultxml).getroot() for product in self.project.ftp_products + self.project.main_products: if not self.package_ok(prjresult, self.project.name, product, self.project.product_repo, self.project.product_arch): - return False + all_ok = False for product in self.project.image_products + self.project.container_products: for arch in product.archs: if not self.package_ok(prjresult, self.project.name, product.package, self.project.product_repo, arch): - return False + all_ok = False for product in self.project.containerfile_products: for arch in product.archs: if not self.package_ok(prjresult, self.project.name, product.package, 'containerfile', arch): - return False + all_ok = False if len(self.project.livecd_products): liveprjname = f'{self.project.name}:Live' @@ -228,7 +230,10 @@ def is_snapshotable(self): for arch in product.archs: if not self.package_ok(liveprjresult, liveprjname, product.package, self.project.product_repo, arch): - return False + all_ok = False + + if not all_ok: + return False # The FTP tree isn't released with setrelease, so it needs to contain # the product version already.