Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2018-11-15 12:41:08
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Thu Nov 15 12:41:08 2018 rev:26 rq:648990 version:1.13.1
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2018-09-24 13:13:33.865675176 +0200
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2018-11-15 12:41:13.014197616 +0100
@@ -1,0 +2,9 @@
+Wed Nov 14 14:01:40 UTC 2018 - [email protected]
+
+- Update to version 1.13.1:
+ * dashboard: Provide explanation on top as requested
+ * dashboard: Try to discover the template file from current dir
+ * dashboard: Replace the fake openQA footer with an explanation of the
report content
+ * Handle connection error while sending POST
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.13.0.obscpio
New:
----
python-openqa_review-1.13.1.obscpio
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.xuUwBa/_old 2018-11-15 12:41:13.682196855 +0100
+++ /var/tmp/diff_new_pack.xuUwBa/_new 2018-11-15 12:41:13.682196855 +0100
@@ -31,7 +31,7 @@
%define binaries openqa-review openqa-review-daily-email
openqa-review-sles-ha tumblesle-release
openqa-review-functional_autoyast_concise
%define oldpython python
Name: python-%{short_name}%{?name_ext}
-Version: 1.13.0
+Version: 1.13.1
Release: 0
Summary: A review helper script for openQA
License: MIT
++++++ python-openqa_review-1.13.0.obscpio ->
python-openqa_review-1.13.1.obscpio ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.13.0/bin/openqa-review-daily-email
new/python-openqa_review-1.13.1/bin/openqa-review-daily-email
--- old/python-openqa_review-1.13.0/bin/openqa-review-daily-email
2018-09-22 12:12:52.000000000 +0200
+++ new/python-openqa_review-1.13.1/bin/openqa-review-daily-email
2018-11-14 14:58:24.000000000 +0100
@@ -17,7 +17,7 @@
# duplicate reminder comments
openqa_review_save_args="${openqa_review_save_args:-"--report-links
--reminder-comment-on-issues --save --save-dir ${tmp}"}"
openqa_review="${openqa_review:-"$(which openqa-review)"}"
-TPL="${TPL:-"dashboard_files/dashboard.html.in"}"
+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} "$@")"
email_report="$(${openqa_review} $openqa_review_args $openqa_review_email_args
"$@")"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/python-openqa_review-1.13.0/dashboard_files/dashboard.html.in
new/python-openqa_review-1.13.1/dashboard_files/dashboard.html.in
--- old/python-openqa_review-1.13.0/dashboard_files/dashboard.html.in
2018-09-22 12:12:52.000000000 +0200
+++ new/python-openqa_review-1.13.1/dashboard_files/dashboard.html.in
2018-11-14 14:58:24.000000000 +0100
@@ -75,18 +75,25 @@
</div>
</div>
+ <div class="container">
+ <div id="explanation" class="text-center">
+ <b>Explanation for the status fields:</b><br/>
+ The status Green/Amber/Red is set simply based on the number of
+ failed test scenarios. If there are any issues reported as "New
+ Issues", i.e. have not been seen in the reference build, the
+ status is always considered "Red". "Amber" when there are
+ multiple, existing issues, "Green" otherwise.
+ <br/>
+ <br/>
+ Special sections simply repeat the corresponding status from the
complete report above.
+ <br/>
+ <br/>
+ Any feedback regarding the script processing or contact can be
communicated using issues on the github repo <a
href="https://github.com/okurz/openqa_review";>okurz/openqa_review</a> or
directly to [email protected]
+ </div>
+ </div>
</div>
${TPL_OPENQA_CONTENT}
</div>
-
- <footer class="footer">
- <div class="container">
- <div id="footer-legal" class="text-center">
- openQA is licensed
- <a href="https://github.com/os-autoinst/openQA";>GPL-2.0</a>
- </div>
- </div>
- </footer>
</body></html>
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/python-openqa_review-1.13.0/openqa_review/browser.py
new/python-openqa_review-1.13.1/openqa_review/browser.py
--- old/python-openqa_review-1.13.0/openqa_review/browser.py 2018-09-22
12:12:52.000000000 +0200
+++ new/python-openqa_review-1.13.1/openqa_review/browser.py 2018-11-14
14:58:24.000000000 +0100
@@ -147,8 +147,18 @@
else: # pragma: no cover
absolute_url = url if not url.startswith('/') else
urljoin(str(self.root_url), str(url))
data = json.dumps({'method': method, 'params': [params]})
- r = requests.post(absolute_url, data=data, auth=self.auth,
headers={'content-type': 'application/json'})
- r.raise_for_status()
+ for i in range(1, 7):
+ try:
+ r = requests.post(absolute_url, data=data, auth=self.auth,
headers={'content-type': 'application/json'})
+ r.raise_for_status()
+ except requests.exceptions.ConnectionError:
+ log.info("Connection error encountered accessing %s,
retrying try %s" % (absolute_url, i))
+ continue
+ break
+ else:
+ msg = "Request to %s was not successful after multiple
retries, giving up" % absolute_url
+ log.warn(msg)
+ return None
return r.json() if r.text else None
def json_rest(self, url, method, data):
++++++ python-openqa_review.obsinfo ++++++
--- /var/tmp/diff_new_pack.xuUwBa/_old 2018-11-15 12:41:13.998196495 +0100
+++ /var/tmp/diff_new_pack.xuUwBa/_new 2018-11-15 12:41:14.002196490 +0100
@@ -1,5 +1,5 @@
name: python-openqa_review
-version: 1.13.0
-mtime: 1537611172
-commit: aaab7bd1d23b0e4d65683c03f8c9033eb2fcdb16
+version: 1.13.1
+mtime: 1542203904
+commit: 47a907bb5f4dfe3c5e7fe29c5803c11f51de060e
