Skip to content

Commit

Permalink
ttm: Print all failing packages before exiting
Browse files Browse the repository at this point in the history
Don't exit early but print all known issues instead.
  • Loading branch information
Vogtinator committed Jun 3, 2024
1 parent 99d3f4c commit 718cb77
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions ttm/releaser.py
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand All @@ -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.
Expand Down

0 comments on commit 718cb77

Please sign in to comment.