maintenance: Do not send notifications for *starting* to test
This commit is contained in:
parent
080dd83110
commit
95e314ea90
@ -862,9 +862,6 @@ class OpenQABot(ReviewBot.ReviewBot):
|
|||||||
self.commentapi.delete(comment_id)
|
self.commentapi.delete(comment_id)
|
||||||
self.commentapi.add_comment(request_id=req.reqid, comment=str(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']})
|
|
||||||
|
|
||||||
# escape markdown
|
# escape markdown
|
||||||
def emd(self, str):
|
def emd(self, str):
|
||||||
return str.replace('_', '\_')
|
return str.replace('_', '\_')
|
||||||
@ -920,12 +917,11 @@ class OpenQABot(ReviewBot.ReviewBot):
|
|||||||
self.add_comment(req, msg, 'done', 'accepted')
|
self.add_comment(req, msg, 'done', 'accepted')
|
||||||
ret = True
|
ret = True
|
||||||
else:
|
else:
|
||||||
url = self.openqa_overview_url_from_settings(jobs[0]['settings'])
|
# no notification until the result is done
|
||||||
self.logger.debug("url %s", url)
|
osc.core.change_review_state(req.apiurl, req.reqid, newstate='new',
|
||||||
msg = "now testing in [openQA](%s)" % url
|
by_group=self.review_group, by_user=self.review_user,
|
||||||
self.add_comment(req, msg, 'seen')
|
msg='now testing in openQA')
|
||||||
elif qa_state == QA_FAILED or qa_state == QA_PASSED:
|
elif qa_state == QA_FAILED or qa_state == QA_PASSED:
|
||||||
url = self.openqa_overview_url_from_settings(jobs[0]['settings'])
|
|
||||||
# don't take test repo results into the calculation of total
|
# don't take test repo results into the calculation of total
|
||||||
# this is for humans to decide which incident broke the test repo
|
# this is for humans to decide which incident broke the test repo
|
||||||
jobs += self.request_get_openqa_jobs(req, incident=False, test_repo=True)
|
jobs += self.request_get_openqa_jobs(req, incident=False, test_repo=True)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user