Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2021-03-05 13:49:08
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new.2378 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Fri Mar 5 13:49:08 2021 rev:35 rq:876941 version:1.20.4
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2020-08-04 20:20:04.424970083 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new.2378/python-openqa_review.changes
2021-03-05 13:51:26.319961804 +0100
@@ -1,0 +2,8 @@
+Fri Mar 05 09:40:21 UTC 2021 - [email protected]
+
+- Update to version 1.20.4:
+ * Change comments to not private when adding comments
+ * Fix readout of custom softfail references
+ * Adapt parsing of soft-failure details on openQA >=
4.5.1528009330.e68ebe2b-3956-gfac9d6dc2
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.20.2.obscpio
New:
----
python-openqa_review-1.20.4.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.fZfAsd/_old 2021-03-05 13:51:26.935962397 +0100
+++ /var/tmp/diff_new_pack.fZfAsd/_new 2021-03-05 13:51:26.935962397 +0100
@@ -1,7 +1,7 @@
#
-# spec file for package python
+# spec file for package python-openqa_review%{?name_ext}
#
-# Copyright (c) 2020 SUSE LLC
+# Copyright (c) 2021 SUSE LLC
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -31,7 +31,7 @@
%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.20.2
+Version: 1.20.4
Release: 0
Summary: A review helper script for openQA
License: MIT
@@ -60,7 +60,7 @@
Requires: python-setuptools
Requires: python-sortedcontainers
Requires(post): update-alternatives
-Requires(preun): update-alternatives
+Requires(preun):update-alternatives
%endif
BuildRoot: %{_tmppath}/%{name}-%{version}-build
BuildArch: noarch
++++++ python-openqa_review-1.20.2.obscpio ->
python-openqa_review-1.20.4.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/openqa_review/openqa_review.py
new/python-openqa_review-1.20.4/openqa_review/openqa_review.py
--- old/python-openqa_review-1.20.2/openqa_review/openqa_review.py
2020-08-03 14:22:00.000000000 +0200
+++ new/python-openqa_review-1.20.4/openqa_review/openqa_review.py
2021-02-26 12:18:47.000000000 +0100
@@ -667,7 +667,7 @@
self.bugzilla_browser.json_rpc_post('/jsonrpc.cgi',
'Bug.add_comment', {
'id': self.bugid,
'comment': comment,
- 'is_private': True,
+ 'is_private': False,
})
# self.issue_type == 'redmine':
else:
@@ -888,13 +888,10 @@
def _get_url_to_softfailed_module(self, job_url):
log.debug('job_url %s' % job_url)
- url = job_url + '/module_components_ajax'
try:
- test_details_html =
self.test_browser.get_soup(url).find(title='Soft Failed')
- if test_details_html is None: # pragma: no cover
- log.debug('Found older openQA, before
https://github.com/os-autoinst/openQA/pull/3080')
- url = job_url + '/details_ajax'
- test_details_html =
self.test_browser.get_soup(url).find(title='Soft Failed')
+ details = self.test_browser.get_json('/api/v1/jobs/' +
job_url.split('/')[-1] + '/details')
+ # fake an older module URL including the name, see
https://progress.opensuse.org/issues/72292
+ return format('///%s//' % [i for i in
details['job']['testresults'] if i['result'] == 'softfailed'][0]['name']) #
pragma: no cover
except DownloadError:
log.debug('Found older openQA, before
https://github.com/os-autoinst/openQA/pull/2932')
url = job_url
@@ -906,7 +903,8 @@
return test_details_html.get('data-url')
def _get_bugref_for_softfailed_module(self, result_item, module_name):
- details_url = '%s/file/details-%s.json' % (result_item['href'],
module_name)
+ rel_job_url = result_item['href']
+ details_url = '%s/file/details-%s.json' % (rel_job_url, module_name)
log.debug("Retrieving '%s'" % details_url)
details_json = self.test_browser.get_json(details_url)
details = details_json['details'] if 'details' in details_json else
details_json
@@ -915,14 +913,18 @@
if 'text_data' in field:
unformated_str = field['text_data']
else:
- unformated_str = self.test_browser.get_soup('%s/file/%s' %
(result_item['href'], quote(field['text']))).getText()
+ unformated_str = self.test_browser.get_soup('%s/file/%s' %
(rel_job_url, quote(field['text']))).getText()
return re.search('Soft Failure:\n(.*)',
unformated_str.strip()).group(1)
+ # custom results can have soft-fail as well
+ if 'result' in field and 'softfail' in field['result']:
+ match = re.search(bugref_regex, field['title'])
+ return match.group(1) + '#' + match.group(2)
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(bugref_regex, 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) for '%s'" %
- result_item['href'])
+ log.warn("Found workaround key without bugref that could
be understood, looking for a better bugref (if any) for '%s'" %
+ rel_job_url)
continue
return match.group(1) + '#' + match.group(2)
else: # pragma: no cover
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/tests/tags_labels/:tests:1316928
new/python-openqa_review-1.20.4/tests/tags_labels/:tests:1316928
--- old/python-openqa_review-1.20.2/tests/tags_labels/:tests:1316928
1970-01-01 01:00:00.000000000 +0100
+++ new/python-openqa_review-1.20.4/tests/tags_labels/:tests:1316928
2021-02-26 12:18:47.000000000 +0100
@@ -0,0 +1,20 @@
+<tr>
+ <td class="component">
+ <div>
+ <a href="/tests/1316928/modules/shutdown/steps/1/src">shutdown</a>
+ </div>
+ <div class="flags">
+ </div>
+ </td>
+ <td class="result resultsoftfailed">
+ soft-failed
+ </td>
+ <td class="links">
+ <div class="links_a">
+ <div class="fa fa-caret-up"></div>
+ <a class="no_hover"
data-url="/tests/1316928/modules/shutdown/steps/9" href="#step/shutdown/9"
+ title="Soft Failed">
+ <span class="resborder resborder_hash(0x9c726c8)">Soft
Failed</span>
+ </a></div>
+ </td>
+</tr>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/tests/tags_labels/:tests:1316928:file:details-shutdown.json
new/python-openqa_review-1.20.4/tests/tags_labels/:tests:1316928:file:details-shutdown.json
---
old/python-openqa_review-1.20.2/tests/tags_labels/:tests:1316928:file:details-shutdown.json
1970-01-01 01:00:00.000000000 +0100
+++
new/python-openqa_review-1.20.4/tests/tags_labels/:tests:1316928:file:details-shutdown.json
2021-02-26 12:18:47.000000000 +0100
@@ -0,0 +1,2 @@
+{"dents": 0, "details": [{"result": "softfail", "text": "shutdown-9.txt",
"title": "rogue workqueue lockup boo#1126782"}], "result": "softfail"}
+
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1507%26groupid%3D25
new/python-openqa_review-1.20.4/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1507%26groupid%3D25
---
old/python-openqa_review-1.20.2/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1507%26groupid%3D25
2020-08-03 14:22:00.000000000 +0200
+++
new/python-openqa_review-1.20.4/tests/tags_labels/:tests:overview%3Fdistri%3Dopensuse%26version%3D42.1%26build%3D1507%26groupid%3D25
2021-02-26 12:18:47.000000000 +0100
@@ -2866,6 +2866,21 @@
<td>-</td>
</tr>
+
+ <tr>
+ <td class="name">custom_bugref</td>
+ <td>-</td>
+ <td>-</td>
+ <td id="res_Gnome-DVD-i586_custom_bugref">
+ <span id="res-1316928">
+ <a href="/tests/1316928">
+ <i class="status fa fa-circle
result_softfailed" title="Done: softfail"></i>
+ </a>
+ </span>
+ </td>
+<td>-</td>
+</tr>
+
<tr>
<td class="name"><span
title="leap12_minimal_base+gcc5_create_hdd">leap12_minimal_base+gcc5_cr
...</span></td>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/tests/tags_labels/report25_T_bugrefs_softfails.md
new/python-openqa_review-1.20.4/tests/tags_labels/report25_T_bugrefs_softfails.md
---
old/python-openqa_review-1.20.2/tests/tags_labels/report25_T_bugrefs_softfails.md
2020-08-03 14:22:00.000000000 +0200
+++
new/python-openqa_review-1.20.4/tests/tags_labels/report25_T_bugrefs_softfails.md
2021-02-26 12:18:47.000000000 +0100
@@ -11,6 +11,7 @@
**New Product bugs:**
+* soft fails: [custom_bugref](https://openqa.opensuse.org/tests/1316928) ->
[boo#1126782](https://bugzilla.opensuse.org/show_bug.cgi?id=1126782)
* [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.opensuse.org/show_bug.cgi?id=931572)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.20.2/tests/tags_labels/report25_bugrefs_query_issues.md
new/python-openqa_review-1.20.4/tests/tags_labels/report25_bugrefs_query_issues.md
---
old/python-openqa_review-1.20.2/tests/tags_labels/report25_bugrefs_query_issues.md
2020-08-03 14:22:00.000000000 +0200
+++
new/python-openqa_review-1.20.4/tests/tags_labels/report25_bugrefs_query_issues.md
2021-02-26 12:18:47.000000000 +0100
@@ -11,6 +11,7 @@
**New Product bugs:**
+* soft fails: custom_bugref ->
[boo#1126782](https://bugzilla.opensuse.org/show_bug.cgi?id=1126782) (Request
to /jsonrpc.cgi?method=Bug.get¶ms=%5B%7B%22ids%22%3A+%5B1126782%5D%7D%5D
was not successful, file
:jsonrpc.cgi%3Fmethod%3DBug.get%26params%3D%255B%257B%2522ids%2522%253A%2B%255B1126782%255D%257D%255D
not found)
* toolchain_zypper ->
[boo#931571](https://bugzilla.opensuse.org/show_bug.cgi?id=931571 "no space
left on device when upgrading ???") (Ticket status: NEW, prio/severity:
P2/Major, assignee: [email protected])
* toolchain_zypper@bar ->
[boo#9315715](https://bugzilla.opensuse.org/show_bug.cgi?id=9315715) (Ticket
not found)
* soft fails: create_hdd_textmode ->
[boo#931572](https://bugzilla.opensuse.org/show_bug.cgi?id=931572 "no space
left on device when upgrading ???") (Ticket status: NEW, prio/severity:
P2/Major, assignee: [email protected])
++++++ python-openqa_review.obsinfo ++++++
--- /var/tmp/diff_new_pack.fZfAsd/_old 2021-03-05 13:51:27.263962712 +0100
+++ /var/tmp/diff_new_pack.fZfAsd/_new 2021-03-05 13:51:27.263962712 +0100
@@ -1,5 +1,5 @@
name: python-openqa_review
-version: 1.20.2
-mtime: 1596457320
-commit: d02138fd922c1b799a4e3f9a5b5bf9ab9f78a228
+version: 1.20.4
+mtime: 1614338327
+commit: 1e522d21ec095ae7b3610682a7690fa8048cd56a
