[Qa-jenkins-scm] [jenkins.debian.net] 01/03: reproducible: html_package: complain laudly if there is no debbindiff for an unreproducible package, or there is no buildlog for a built package

[Holger Levsen]

This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit e013648a1982364aeb60893f0028f41ebbf213a8
Author: Mattia Rizzolo <mattia at mapreri.org>
Date:   Fri Mar 20 18:49:55 2015 +0100

    reproducible: html_package: complain laudly if there is no debbindiff for an unreproducible package, or there is no buildlog for a built package
---
 bin/reproducible_html_packages.py | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/bin/reproducible_html_packages.py b/bin/reproducible_html_packages.py
index 9421531..b969313 100755
--- a/bin/reproducible_html_packages.py
+++ b/bin/reproducible_html_packages.py
@@ -103,7 +103,7 @@ def gen_status_link_icon(status, icon, suite, arch):
     return html.format(status=status, icon=icon, suite=suite, arch=arch)
 
 
-def gen_extra_links(package, version, suite, arch):
+def gen_extra_links(package, version, suite, arch, status):
     eversion = strip_epoch(version)
     notes = NOTES_PATH + '/' + package + '_note.html'
     rbuild = RBUILD_PATH + '/' + suite + '/' + arch + '/' + package + '_' + \
@@ -130,6 +130,9 @@ def gen_extra_links(package, version, suite, arch):
             default_view = url
     else:
         log.debug('debbindiff not detetected at ' + dbd)
+        if status == 'unreproducible':
+            log.critical('The package ' + package + '/' + suite + '/' + arch +
+                         ' is unreproducible, but without debbindiff output')
     if pkg_has_buildinfo(package, version, suite):
         url = BUILDINFO_URI + '/' + suite + '/' + arch + '/' + package + \
               '_' + eversion + '_amd64.buildinfo'
@@ -146,9 +149,9 @@ def gen_extra_links(package, version, suite, arch):
                 sizeof_fmt(log_size) + ')</a>\n'
         if not default_view:
             default_view = url
-    else:
-        log.info('The package ' + package +
-                    ' did not produce any buildlog in ' + suite + '/' + arch + '! Maybe it has not been build yet.')
+    elif status != 'untested':
+        log.critical('The package ' + package + '/' + suite + '/' + arch +
+                     ' didn\'t produce a buildlog, even if it has been built')
     default_view = '/untested.html' if not default_view else default_view
     return (links, default_view)
 
@@ -212,7 +215,7 @@ def gen_packages_html(packages, suite=None, arch=None, no_clean=False, nocheck=F
         log.debug('Generating the page of ' + pkg + '/' + suite + '@' + version +
                  ' built at ' + build_date)
 
-        links, default_view = gen_extra_links(pkg, version, suite, arch)
+        links, default_view = gen_extra_links(pkg, version, suite, arch, status)
         suites_links = gen_suites_links(pkg, suite)
         status, icon = join_status_icon(status, pkg, version)
         status = gen_status_link_icon(status, icon, suite, arch)

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git