Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2017-07-25 11:41:07
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Tue Jul 25 11:41:07 2017 rev:14 rq:512317 version:1.7.2
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2017-07-23 12:16:41.048384791 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2017-07-25 11:41:23.607895027 +0200
@@ -1,0 +2,6 @@
+Mon Jul 24 15:08:40 UTC 2017 - ok...@suse.com
+
+- Update to version 1.7.2:
+ * Extend soft-fail issue tracker support
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.7.1.tar.gz
New:
----
python-openqa_review-1.7.2.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.TAbC1t/_old 2017-07-25 11:41:24.331792824 +0200
+++ /var/tmp/diff_new_pack.TAbC1t/_new 2017-07-25 11:41:24.331792824 +0200
@@ -18,7 +18,7 @@
%define short_name openqa_review
Name: python-%{short_name}
-Version: 1.7.1
+Version: 1.7.2
Release: 0
Summary: A review helper script for openQA
License: MIT
++++++ python-openqa_review-1.7.1.tar.gz -> python-openqa_review-1.7.2.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.7.1/openqa_review/openqa_review.py
new/openqa_review-1.7.2/openqa_review/openqa_review.py
--- old/openqa_review-1.7.1/openqa_review/openqa_review.py 2017-07-23
09:55:07.000000000 +0200
+++ new/openqa_review-1.7.2/openqa_review/openqa_review.py 2017-07-24
17:06:59.000000000 +0200
@@ -223,6 +223,11 @@
interesting_states_names = [i for i in set(change_state.values()) if i !=
'STABLE'] + ['INCOMPLETE']
+issue_tracker = { # pragma: no branch
+ 'bsc': lambda i: 'https://bugzilla.suse.com/show_bug.cgi?id=%s' % i,
+ 'boo': lambda i: 'https://bugzilla.opensuse.org/show_bug.cgi?id=%s' % i,
+ 'poo': lambda i: 'https://progress.opensuse.org/issues/%s' % i,
+}
def status(entry):
"""Return test status from entry, e.g. 'result_passed'."""
@@ -787,13 +792,21 @@
v['bugref'] = self._get_bugref_for_softfailed_module(v,
module_name)
if not v['bugref']: # pragma: no cover
continue
- assert re.match('(bsc|boo)#?', v['bugref']), 'TODO
implement: more bugref matching'
- # TODO the following re.search fails to find the bugref in
e.g. https://openqa.opensuse.org/tests/447927#step/bootloader/7 where the bugref
- # is not at the end of line
- v['bugref_href'] =
"https://bugzilla.suse.com/show_bug.cgi?id=%s"; % re.search("[0-9]*$",
v['bugref']).group(0)
+ except AttributeError: # pragma: no cover
+ log.info('Could find neither soft failed info box nor
needle, assuming an old openQA job, skipping.')
+ continue
except DownloadError as e: # pragma: no cover
log.error("Failed to process %s with error %s. Skipping
current result" % (v, e))
continue
+ match = re.search('([a-z]{3})#?([0-9]+)', v['bugref'])
+ if not match: # pragma: no cover
+ log.info('Could not find bug reference in text \'%s\',
skipping.' % v['bugref'])
+ continue
+ bugref, bug_id = match.group(1), match.group(2)
+ assert bugref, "No bugref found for %s" % v
+ assert bug_id, "No bug_id found for %s" % v
+ v['bugref_href'] = issue_tracker[bugref](bug_id)
+
@property
def total_issues(self):
@@ -820,7 +833,7 @@
return re.search("Soft Failure:\n([^/]*)",
unformated_str.strip()).group(1)
elif 'properties' in field and len(field['properties']) > 0 and
field['properties'][0] == 'workaround':
log.debug('Evaluating potential workaround needle \'%s\'' %
field['needle'])
- match = re.search('([a-z]{3})([0-9]+)-[0-9]+', field['needle'])
+ match = re.search('([a-z]{3})#?([0-9]+)-[0-9]+',
field['needle'])
if not match: # pragma: no cover
log.warn('Found workaround needle without bugref that
could be understood, looking for a better bugref (if any)')
continue
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.7.1/tests/tags_labels/report25_T_bugrefs_softfails.md
new/openqa_review-1.7.2/tests/tags_labels/report25_T_bugrefs_softfails.md
--- old/openqa_review-1.7.1/tests/tags_labels/report25_T_bugrefs_softfails.md
2017-07-23 09:55:07.000000000 +0200
+++ new/openqa_review-1.7.2/tests/tags_labels/report25_T_bugrefs_softfails.md
2017-07-24 17:06:59.000000000 +0200
@@ -13,7 +13,7 @@
* [toolchain_zypper](https://openqa.opensuse.org/tests/384324 "Failed modules:
install") -> [boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571)
* [toolchain_zypper@bar](https://openqa.opensuse.org/tests/3843245 "Failed
modules: install") ->
[boo#9315715](https://bugzilla.opensuse.org/show_bug.cgi?id=9315715)
-* soft fails: [create_hdd_textmode](https://openqa.opensuse.org/tests/447901)
-> [boo#931572](https://bugzilla.suse.com/show_bug.cgi?id=931572)
+* soft fails: [create_hdd_textmode](https://openqa.opensuse.org/tests/447901)
-> [boo#931572](https://bugzilla.opensuse.org/show_bug.cgi?id=931572)
**Existing Product bugs:**
