summaryrefslogtreecommitdiffstats
path: root/cnucnu
diff options
context:
space:
mode:
authorTill Maas <opensource@till.name>2014-02-06 12:37:30 +0100
committerTill Maas <opensource@till.name>2014-02-06 12:37:30 +0100
commit1ce25e7fe206cda02d5c009120b5b6ae856e0ec0 (patch)
tree18d11448f84e324340f1999110160e2019f9d1b3 /cnucnu
parent9d705b5efc0a137f23229c39f4ba3cb4a2d89e4f (diff)
downloadcnucnu-1ce25e7fe206cda02d5c009120b5b6ae856e0ec0.tar.gz
cnucnu-1ce25e7fe206cda02d5c009120b5b6ae856e0ec0.tar.xz
cnucnu-1ce25e7fe206cda02d5c009120b5b6ae856e0ec0.zip
PEP8 bugzilla_reporter.py
Diffstat (limited to 'cnucnu')
-rw-r--r--cnucnu/bugzilla_reporter.py44
1 files changed, 26 insertions, 18 deletions
diff --git a/cnucnu/bugzilla_reporter.py b/cnucnu/bugzilla_reporter.py
index 12214fa..2f688cd 100644
--- a/cnucnu/bugzilla_reporter.py
+++ b/cnucnu/bugzilla_reporter.py
@@ -52,8 +52,10 @@ class BugzillaReporter(object):
self.new_bug['platform'] = 'Unspecified'
self.new_bug['bug_severity'] = 'unspecified'
# Using ASSIGNED returns an exception:
- # <Fault 32000: 'You are not allowed to file new bugs with the\n ASSIGNED status.'>
- # if the account is in editbugs, fedora_bugs, fedora_contrib, setpriority
+ # <Fault 32000: 'You are not allowed to file new bugs with the\n
+ # ASSIGNED status.'>
+ # if the account is in editbugs, fedora_bugs, fedora_contrib,
+ # setpriority
# if not, then it is silently ignored
# 'status': 'ASSIGNED',
# Red Hat Bugzilla now supports filing bugs in state ASSIGNED:
@@ -69,8 +71,6 @@ class BugzillaReporter(object):
self._bz = Bugzilla(**rpc_conf)
return self._bz
-
-
def bug_url(self, bug):
if isinstance(bug, str):
bug_id = bug
@@ -82,7 +82,8 @@ class BugzillaReporter(object):
def report_outdated(self, package, dry_run=True):
if not package.exact_outdated_bug:
if not package.open_outdated_bug:
- new_bug, change_status = self.create_outdated_bug(package, dry_run)
+ new_bug, change_status = self.create_outdated_bug(package,
+ dry_run)
return self.bug_url(new_bug)
else:
open_bug = package.open_outdated_bug
@@ -91,29 +92,34 @@ class BugzillaReporter(object):
# short_desc should be '<name>-<version> <some text>'
# To extract the version get everything before the first space
# with split and then remove the name and '-' via slicing
- bug_version = short_desc.split(" ")[0][len(package.name)+1:]
+ bug_version = short_desc.split(" ")[0][len(package.name) + 1:]
if bug_version != package.latest_upstream:
- update = {'summary': self.config["short_desc template"] % package,
- 'comment': {'body': self.config["description template"] % package, 'is_private': False},
+ update = {'summary': self.config["short_desc template"]
+ % package,
+ 'comment': {'body':
+ self.config["description template"] %
+ package, 'is_private': False},
'ids': [open_bug.bug_id],
- }
- log.debug("About to update bug '%s' with '%r'" % (open_bug.bug_id, update))
+ }
+ log.debug("About to update bug '%s' with '%r'" % (
+ open_bug.bug_id, update))
res = self.bz._proxy.Bug.update(update)
log.debug("Result from bug update: %r" % res)
log.info("Updated bug: %s" % self.bug_url(open_bug))
return self.bug_url(open_bug)
else:
bug = package.exact_outdated_bug
- log.info("already reported:%s %s" % (self.bug_url(bug), bug.bug_status))
+ log.info("already reported:%s %s" % (self.bug_url(bug),
+ bug.bug_status))
return ""
-
def create_outdated_bug(self, package, dry_run=True):
- bug_dict = {'component': package.name,
- 'short_desc': self.config["short_desc template"] % package,
- 'description': self.config["description template"] % package
- }
+ bug_dict = {
+ 'component': package.name,
+ 'short_desc': self.config["short_desc template"] % package,
+ 'description': self.config["description template"] % package
+ }
bug_dict.update(self.new_bug)
if not dry_run:
new_bug = self.bz.createbug(**bug_dict)
@@ -122,7 +128,9 @@ class BugzillaReporter(object):
log.info("Created bug: %s" % self.bug_url(new_bug))
if new_bug.bug_status != self.config['bug status']:
- change_status = self.bz._proxy.bugzilla.changeStatus(new_bug.bug_id, self.config['bug status'], self.config['user'], "", "", False, False, 1)
+ change_status = self.bz._proxy.bugzilla.changeStatus(
+ new_bug.bug_id, self.config['bug status'],
+ self.config['user'], "", "", False, False, 1)
log.debug("Changed bug status %r" % change_status)
return (new_bug, change_status)
else:
@@ -153,7 +161,7 @@ class BugzillaReporter(object):
def get_open_outdated_bug(self, package):
q = {'component': [package.name],
'bug_status': [self.config['bug status']]
- }
+ }
q.update(self.base_query)
bugs = self.bz.query(q)