Refresh email template for sending OST [CQ] failures to devel list
Description
Activity

Dafna Ron September 19, 2018 at 7:11 AM
owner = the "to" field is sent to the owner of the suspected patch
summary = we try to write down what is the issue when possible
I have an issue about adding the component until we automate the process because it may cause people to ignore an issue which they can help resolve and since this is the stability of the whole branch and not just a component I would like people to have a look at the error and be aware that there is an issue.

danken September 19, 2018 at 1:46 AM
My original subject for this ticket was was "Include summary, owner and component of suspected patch in OST Failure Report "
, if your reports include those, you've done what I wanted.

Dafna Ron September 18, 2018 at 8:16 AM
I am not sure what is needed to be fixed there as it has project name in the subject and has the patch + name in the mail.
is there anything else that you would like to add?

Eyal Edri September 14, 2017 at 2:35 PM
Since this is a manual process for now ( Until we implement the reporting back to Gerrit ), I'll change this ticket for a procedural task for updating the template sent to devel list.
https://docs.google.com/document/d/1Z6qY2S9W2m1qDO0fvfRjTnSW7LhANLCsMdTdrz_8uOk/edit

Barak Korren September 14, 2017 at 2:18 PM
ok, I realised you're referring to the message sent manually by the infra-owner, as opposed to the generated message delivered the CQ to the infra-list.
I agree that at the very least the project name should be there, and probably also documented in the OST failure tracking table we maintain.

Barak Korren September 14, 2017 at 2:08 PM
in "component" I meant ovirt-project. (engine/vdsm/node/...)
This already shows up. Both in the subject line and in the message body.

danken September 14, 2017 at 1:59 PM
in "component" I meant ovirt-project. (engine/vdsm/node/...)

Barak Korren September 14, 2017 at 1:56 PM
Yes - I've asked to have the patch summary line, and its component.
I'm probably not going to implement this, it will become redundant once CQ messages are posted to Gerrit and requires some extensive code changes to keep this information so it is available in time for reporting.
BTW not sure what you mean by "component".

danken September 14, 2017 at 12:28 PM
On Thu, Sep 14, 2017 at 3:17 PM, eyal edri (oVirt JIRA) <
Yes - I've asked to have the patch summary line, and its component.
On Thu, Sep 14, 2017 at 3:11 PM, danken (oVirt JIRA) <

Eyal Edri September 14, 2017 at 12:17 PM
We are planning to start commenting directly in Gerrit on the relevant
offending patch soon, we were waiting to see how CQ behaves and sending
emails to infra only until now.
However, it looks like it's quite accurate, so we might consider doing it
sooner, though we might also want to wait until we gate also OS changes
before during that.
Nevertheless, I guess we can in the meantime include the patch owner in the
'to' email when sending a report.
Any comments on the current summary? is there something missing in it other
than the patch owner?
On Thu, Sep 14, 2017 at 3:11 PM, danken (oVirt JIRA) <
–
Eyal edri
ASSOCIATE MANAGER
RHV DevOps
EMEA VIRTUALIZATION R&D
Red Hat EMEA <https://www.redhat.com/>
<https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted>
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
Details
Assignee
Dafna RonDafna Ron(Deactivated)Reporter
dankendanken(Deactivated)Blocked By
OVIRT-1447Priority
Low
Details
Details
Assignee

Reporter

$subject would make it easier and quicker to check if the suspicion is correct.