Hello community,
here is the log from the commit of package python-openqa_review for
openSUSE:Factory checked in at 2017-03-03 17:56:13
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-openqa_review (Old)
and /work/SRC/openSUSE:Factory/.python-openqa_review.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-openqa_review"
Fri Mar 3 17:56:13 2017 rev:9 rq:462046 version:1.5.2
Changes:
--------
---
/work/SRC/openSUSE:Factory/python-openqa_review/python-openqa_review.changes
2017-02-24 02:54:53.528341688 +0100
+++
/work/SRC/openSUSE:Factory/.python-openqa_review.new/python-openqa_review.changes
2017-03-03 17:56:14.497910926 +0100
@@ -1,0 +2,15 @@
+Fri Mar 03 05:19:17 UTC 2017 - [email protected]
+
+- Update to version 1.5.2:
+ * tumblesle_release: Add optional AMQP authentication
+ * Add CII badge after review
+ * Replace obsolete gitlab with github reference
+ * Fix version file parsing when there is no git tag
+ * tumblesle-release: Retry on AMQP connection lost
+
+-------------------------------------------------------------------
+Fri Mar 3 05:09:49 UTC 2017 - [email protected]
+
+- Fix import error on backports.configparser by workaround (boo#1025589)
+
+-------------------------------------------------------------------
Old:
----
python-openqa_review-1.5.1.tar.gz
New:
----
python-openqa_review-1.5.2.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-openqa_review.spec ++++++
--- /var/tmp/diff_new_pack.p0VlHQ/_old 2017-03-03 17:56:15.117823363 +0100
+++ /var/tmp/diff_new_pack.p0VlHQ/_new 2017-03-03 17:56:15.121822798 +0100
@@ -18,7 +18,7 @@
%define short_name openqa_review
Name: python-%{short_name}
-Version: 1.5.1
+Version: 1.5.2
Release: 0
Summary: A review helper script for openQA
License: MIT
@@ -31,6 +31,9 @@
Requires: python-beautifulsoup4
Requires: python-certifi
Requires: python-configparser
+# workaround because of python-configparser not providing the '__init__.py'
+# file within site-packages/backports
+Requires: python-backports.ssl_match_hostname
Requires: python-future
Requires: python-humanfriendly
Requires: python-pika
@@ -50,6 +53,7 @@
%install
python setup.py install --prefix=%{_prefix} --root=%{buildroot}
+rm %{buildroot}/%{python_sitelib}/version.*
%files
%defattr(-,root,root,-)
++++++ python-openqa_review-1.5.1.tar.gz -> python-openqa_review-1.5.2.tar.gz
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.5.1/README.md
new/openqa_review-1.5.2/README.md
--- old/openqa_review-1.5.1/README.md 2017-02-09 14:17:42.000000000 +0100
+++ new/openqa_review-1.5.2/README.md 2017-03-03 06:14:56.000000000 +0100
@@ -1,4 +1,4 @@
-# openqa_review_script [](https://travis-ci.org/okurz/openqa_review)
+# openqa_review_script [](https://travis-ci.org/okurz/openqa_review)
[](https://bestpractices.coreinfrastructure.org/projects/699)
A review helper script for openQA.
@@ -36,9 +36,9 @@
This project lives in https://github.com/okurz/openqa_review
-Feel free to add issues in gitlab or send pull requests.
+Feel free to add issues in github or send pull requests.
-TODOs and ideas are tracked in the file `TODO` as well as gitlab issues.
+TODOs and ideas are tracked in the file `TODO` as well as github issues.
### Rules for commits
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.5.1/TODO new/openqa_review-1.5.2/TODO
--- old/openqa_review-1.5.1/TODO 2017-02-09 14:17:42.000000000 +0100
+++ new/openqa_review-1.5.2/TODO 2017-03-03 06:14:56.000000000 +0100
@@ -9,14 +9,9 @@
- think about updating a wiki page on redmine with a report
http://www.redmine.org/projects/redmine/wiki/Rest_WikiPages
- - Review https://gitlab.suse.de/security/tools/tree/master and
- especially https://gitlab.suse.de/security/tools/blob/master/bzmass for
- mass filing bugs
-
- Add mode to count bug occurences (e.g. parse previous)
- - check install error when cloned from gitlab w/o tags maybe based on
pylibrary cookiecutter template? missing tag is causing version to fail
- - tags added but should also work without, try "bumpversion"? see
cookiecutter-pylibrary template
+ - try pylibrary cookiecutter template or "bumpversion"? see
cookiecutter-pylibrary template
- offer to automatically post comment on webpage
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore'
old/openqa_review-1.5.1/openqa_review/tumblesle_release.py
new/openqa_review-1.5.2/openqa_review/tumblesle_release.py
--- old/openqa_review-1.5.1/openqa_review/tumblesle_release.py 2017-02-09
14:17:42.000000000 +0100
+++ new/openqa_review-1.5.2/openqa_review/tumblesle_release.py 2017-03-03
06:14:56.000000000 +0100
@@ -64,6 +64,8 @@
#[notification]
#host = localhost
+#username = guest
+#password = guest
"""
@@ -111,12 +113,16 @@
self.browser = Browser(args, args.openqa_host)
if not config.has_section('notification'):
return
- notify_host = config.get('notification', 'host',
fallback='kazhua.suse.de')
- self.notify_connection =
pika.BlockingConnection(pika.ConnectionParameters(host=notify_host))
+ self.credentials = pika.PlainCredentials(config.get('notification',
'username', fallback='guest'), config.get('notification', 'password',
fallback='guest'))
+ self.notify_host = config.get('notification', 'host',
fallback='kazhua.suse.de')
+ self.notify_connect()
+
+ def notify_connect(self):
+ self.notify_connection =
pika.BlockingConnection(pika.ConnectionParameters(host=self.notify_host,
credentials=self.credentials, heartbeat_interval=10))
self.notify_channel = self.notify_connection.channel()
self.notify_channel.exchange_declare(exchange='pubsub', type='topic',
passive=True, durable=True)
self.notify_topic = 'suse.tumblesle'
- self.notify_seen = deque(maxlen=args.seen_maxlen)
+ self.notify_seen = deque(maxlen=self.args.seen_maxlen)
def __del__(self):
"""Cleanup notification objects."""
@@ -133,7 +139,17 @@
if body in self.notify_seen:
log.debug("notification message already sent out recently, not
resending: %s" % body)
return
- self.notify_channel.basic_publish(exchange='pubsub',
routing_key='.'.join([self.notify_topic, topic]), body=body)
+ tries = 7 # arbitrary
+ for t in range(tries):
+ try:
+ self.notify_channel.basic_publish(exchange='pubsub',
routing_key='.'.join([self.notify_topic, topic]), body=body)
+ break
+ except pika.exceptions.ConnectionClosed as e: # pragma: no cover
+ log.warn('sending notification did not work: %s. Retrying try
%s out of %s' % (e, t, tries))
+ self.notify_connect()
+ else: # pragma: no cover
+ log.error('could not send out notification for %s tries,
aborting.' % tries)
+ raise pika.exceptions.ConnectionClosed()
self.notify_seen.append(body)
def run(self, do_run=True):
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/openqa_review-1.5.1/setup.py
new/openqa_review-1.5.2/setup.py
--- old/openqa_review-1.5.1/setup.py 2017-02-09 14:17:42.000000000 +0100
+++ new/openqa_review-1.5.2/setup.py 2017-03-03 06:14:56.000000000 +0100
@@ -16,7 +16,7 @@
version_git = check_output(["git", "describe",
"--tags"]).rstrip().decode('ascii')
except:
try:
- version_git =
open(version_py).read().strip().split('=')[-1].replace('"', '')
+ version_git =
open(version_py).read().strip().split('=')[-1].replace('\'', '').strip()
except:
version_git = '0.0.0'
