yudaike/openSUSE-release-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #491 from lnussel/master

Browse Source 
Make sure to not post unicode strings to OBS
This commit is contained in:
Ludwig Nussel 2016-04-27 15:34:47 +02:00
commit 0617ecce2f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openqa-maintenance.py
View File

@ -332,7 +332,7 @@ class OpenQABot(ReviewBot.ReviewBot):
self.logger.debug("adding comment to %s, state %s result %s", req.reqid, state, result)
self.logger.debug("message: %s", msg)
if not self.dryrun:
self.commentapi.add_comment(request_id=req.reqid, comment=comment)
self.commentapi.add_comment(request_id=req.reqid, comment=str(comment))
def openqa_overview_url_from_settings(self, settings):
return osc.core.makeurl(self.openqa.baseurl, ['tests'], {'match': settings['BUILD']})
@ -396,7 +396,7 @@ class OpenQABot(ReviewBot.ReviewBot):
for job in jobs:
modules = self.find_failed_modules(job)
if modules != []:
msg += '\n* [%s](%s) failed %s in %s' % (
msg += '\n- [%s](%s) failed %s in %s' % (
job['id'],
osc.core.makeurl(self.openqa.baseurl, ['tests', str(job['id'])]),
job['settings']['TEST'], ','.join(modules))