[Qa-jenkins-scm] [jenkins.debian.net] 02/02: reproducible: make the notes jobs not fail if there is a note for an untested package. add irc notification.

[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 3c9906db5bbd695dfb7d2de66ecd0ec9e8a1bbf8
Author: Holger Levsen <holger at layer-acht.org>
Date:   Fri May 1 10:44:33 2015 +0200

    reproducible: make the notes jobs not fail if there is a note for an untested package. add irc notification.
---
 TODO                        | 1 -
 bin/reproducible_notes.py   | 7 +++++--
 logparse/reproducible.rules | 1 +
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/TODO b/TODO
index c2cc60c..084d62f 100644
--- a/TODO
+++ b/TODO
@@ -194,7 +194,6 @@ grep '(=.*).,.$' rygel_0.24.2-1.debbindiff.html | uniq -c | sort | grep -c '^\s*
 * notes related
 ** new page with annoted packages without categorized issues
 ** new page with notes that doesnt make sense: a.) packages which are reproducible but should not, packages that build but shouldn't, etc.
-** notes job shouldn't fail when it hits unknown packages (=removed ones + typos) but rather point this out, probably via irc notify
 
 * pkg sets related:
 ** for all pkg sets: only display FTBR+FTBFS and reproducible packages with notes by default, provide a way to show them all...
diff --git a/bin/reproducible_notes.py b/bin/reproducible_notes.py
index e2b44f9..80328fa 100755
--- a/bin/reproducible_notes.py
+++ b/bin/reproducible_notes.py
@@ -13,6 +13,7 @@ import json
 import yaml
 from apt_pkg import version_compare
 from reproducible_common import *
+from subprocess import call
 
 NOTES = 'packages.yml'
 ISSUES = 'issues.yml'
@@ -46,10 +47,12 @@ def load_notes():
         query = query.format(pkg=pkg)
         result = query_db(query)
         if not result:
-            print_critical_message('This query produces no results: ' + query
+            print_critical_message('Warning: This query produces no results: ' + query
                                    + '\nThis means there is no tested ' +
                                    'package with the name ' + pkg)
-            sys.exit(1)
+            kgb = ['kgb-client', '--conf', '/srv/jenkins/kgb/debian-reproducible.conf',
+                   '--relay-msg', 'There is problem with the note for '+pkg+' - please investigate']]
+            call(kgb)
         else:
             notes[pkg] = []
             for suite in result:
diff --git a/logparse/reproducible.rules b/logparse/reproducible.rules
index b9a5c56..f07fd01 100644
--- a/logparse/reproducible.rules
+++ b/logparse/reproducible.rules
@@ -17,3 +17,4 @@ warning /Warning: the following failed builds have been found/
 warning /Warning: cannot update html pages for.+/
 warning /Warning: package .+is probably already building elsewhere, exiting./
 warning /Warning: too much difference for .+, aborting. Please investigate and update manually./
+warning /Warning: This query produces no results:.+/

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