Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2020-01-27 20:17:07
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new.26092 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Mon Jan 27 20:17:07 2020 rev:29 rq:767592 version:1.16.0
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2019-10-02 11:58:45.291053349 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new.26092/python-openqa_review.changes
2020-01-27 20:17:23.452520813 +0100
@@ -1,0 +2,9 @@
+Mon Jan 27 08:52:34 UTC 2020 - [email protected]
+
+- Update to version 1.16.0:
+ * Fix error on repeated test modules when reading out details from soft-fail
bugrefs
+ * Fix failed assertion on build with only incompletes
+ * Rely only on POSIX 'command -v' rather than non-standard which
+ * In the bug reminder comment explain options to prevent further reminders
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.15.0.obscpio
New:
----
python-openqa_review-1.16.0.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.APX7cF/_old 2020-01-27 20:17:25.756522220 +0100
+++ /var/tmp/diff_new_pack.APX7cF/_new 2020-01-27 20:17:25.784522237 +0100
@@ -1,7 +1,7 @@
#
# spec file for package python
#
-# Copyright (c) 2019 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2020 SUSE LLC
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -31,13 +31,13 @@
%define binaries openqa-review openqa-review-daily-email
openqa-review-sles-ha tumblesle-release openqa-review-functional_yast_concise
%define oldpython python
Name: python-%{short_name}%{?name_ext}
-Version: 1.15.0
+Version: 1.16.0
Release: 0
Summary: A review helper script for openQA
License: MIT
Group: Development/Languages/Python
Source: python-%{short_name}-%{version}.tar.xz
-Url: https://github.com/okurz/%{short_name}
+URL: https://github.com/okurz/%{short_name}
BuildRequires: python-rpm-macros
%if 0%{?_test}
BuildRequires: %{oldpython}-%{short_name} == %{version}
++++++ python-openqa_review-1.15.0.obscpio ->
python-openqa_review-1.16.0.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/python-openqa_review-1.15.0/bin/dashboard
new/python-openqa_review-1.16.0/bin/dashboard
--- old/python-openqa_review-1.15.0/bin/dashboard 2019-09-30
22:58:26.000000000 +0200
+++ new/python-openqa_review-1.16.0/bin/dashboard 2020-01-21
22:44:30.000000000 +0100
@@ -5,7 +5,7 @@
TPL="${TPL:-"dashboard_files/dashboard.html.in"}"
OUT="${OUT:-"dashboard.html"}"
-if which openqa-review >/dev/null 2>&1 ; then
+if command -v openqa-review >/dev/null ; then
OPENQA_REVIEW="${OPENQA_REVIEW:-"openqa-review"}"
else
OPENQA_REVIEW="${OPENQA_REVIEW:-"./openqa_review/openqa_review.py"}"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.15.0/bin/openqa-review-daily-email
new/python-openqa_review-1.16.0/bin/openqa-review-daily-email
--- old/python-openqa_review-1.15.0/bin/openqa-review-daily-email
2019-09-30 22:58:26.000000000 +0200
+++ new/python-openqa_review-1.16.0/bin/openqa-review-daily-email
2020-01-21 22:44:30.000000000 +0100
@@ -18,7 +18,7 @@
# and if we call it here and also in a later step we would end up with
# duplicate reminder comments
openqa_review_save_args="${openqa_review_save_args:-"--reminder-comment-on-issues
--save --save-dir ${tmp}"}"
-openqa_review="${openqa_review:-"$(which openqa-review)"}"
+openqa_review="${openqa_review:-"$(command -v openqa-review)"}"
TPL="${TPL:-"$(dirname $0)/../dashboard_files/dashboard.html.in"}"
# We have to preserve the '$@' until here to prevent too early evaluation of
parameters with spaces
save_report="$(${openqa_review} ${openqa_review_args}
${openqa_review_save_args} "$@")"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.15.0/openqa_review/openqa_review.py
new/python-openqa_review-1.16.0/openqa_review/openqa_review.py
--- old/python-openqa_review-1.15.0/openqa_review/openqa_review.py
2019-09-30 22:58:26.000000000 +0200
+++ new/python-openqa_review-1.16.0/openqa_review/openqa_review.py
2020-01-21 22:44:30.000000000 +0100
@@ -192,6 +192,11 @@
This bug is still referenced in a failing openQA test: $name
$url
+
+To prevent further reminder comments one of the following options should be
followed:
+1. The test scenario is fixed by applying the bug fix to the tested product or
the test is adjusted
+2. The openQA job group is moved to "Released"
+3. The label in the openQA scenario is removed
""")
status_badge_str = {
@@ -900,7 +905,7 @@
details_json =
json.loads(self.test_browser.get_soup('%s/file/details-%s.json' %
(result_item['href'], module_name)).getText())
for field in details_json:
if 'title' in field and 'Soft Fail' in field['title']:
- unformated_str = self.test_browser.get_soup('%s/file/%s' %
(result_item['href'], field['text'])).getText()
+ unformated_str = self.test_browser.get_soup('%s/file/%s' %
(result_item['href'], quote(field['text']))).getText()
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'])
@@ -955,7 +960,9 @@
current_details = browser.get_soup(current_url)
previous_details = browser.get_soup(previous_url)
for details in current_details, previous_details:
- assert sum(int(badge.text) for badge in
details.find_all(class_='badge')) > 0, \
+ # build pages matching no tests show no job as well as builds only
consisting of incomplete jobs
+ # see https://progress.opensuse.org/issues/60458
+ assert sum(int(badge.text) for badge in
details.find_all(class_='badge')) > 0 or len(details.find_all(class_='status'))
> 0, \
'invalid page with no test results found reading %s and %s,
make sure you specified valid builds (leading zero missing?)' \
% (current_url, previous_url)
current_summary = parse_summary(current_details)
++++++ python-openqa_review.obsinfo ++++++
--- /var/tmp/diff_new_pack.APX7cF/_old 2020-01-27 20:17:26.656522770 +0100
+++ /var/tmp/diff_new_pack.APX7cF/_new 2020-01-27 20:17:26.656522770 +0100
@@ -1,5 +1,5 @@
name: python-openqa_review
-version: 1.15.0
-mtime: 1569877106
-commit: a28a115190a55cfee6db7d23540a0c7b34e141d3
+version: 1.16.0
+mtime: 1579643070
+commit: 175df1037206206e2bec8793ae87031d9fbba6d7
