Closed Bug 1042056 Opened 7 years ago Closed 7 years ago

release runner failure e-mail should be sent to relman or release-drivers

Categories

(Release Engineering :: Release Automation: Other, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

Details

Attachments

(1 file)

We had releases fail to start early this morning because of failures during release runner. They didn't start, and no indication was given, except to release@mozilla.com. This is confusing, and makes it impossible for anyone else to try and debug on their own.

We should also increase the number of lines of context included. I'd suggest 100 lines for now, which should end up including all of the release sanity output.
Maybe use last 1000 lines and attach as text? 100 won't be enough I bet.
(In reply to Rail Aliiev [:rail] from comment #1)
> Maybe use last 1000 lines and attach as text? 100 won't be enough I bet.

Yeah, maybe. My only concern is that you may end up with context from a previous run of release runner. Maybe we can have max 1000, but walk backwards only until we see "Fetching release requests" to avoid getting a previous run?
... and replace the shell wrapper with something better :)
This should get the mail sent to the right place. bug 872316 is a better place to track improving the log, which I plan to pick up soon.
Assignee: nobody → bhearsum
Status: NEW → ASSIGNED
Attachment #8462696 - Flags: review?(rail)
Comment on attachment 8462696 [details] [diff] [review]
send failure mail to release-drivers

Hmm, Rail is away for awhile. Probably good to have someone from RelMan sign off on this too.
Attachment #8462696 - Flags: review?(rail)
Attachment #8462696 - Flags: review?(catlee)
Attachment #8462696 - Flags: feedback?(sledru)
Comment on attachment 8462696 [details] [diff] [review]
send failure mail to release-drivers

LGTM
Attachment #8462696 - Flags: feedback?(sledru) → feedback+
Comment on attachment 8462696 [details] [diff] [review]
send failure mail to release-drivers

Review of attachment 8462696 [details] [diff] [review]:
-----------------------------------------------------------------

::: manifests/moco-config.pp
@@ +189,5 @@
>  
>      $vmwaretools_version = "9.4.0-1280544"
>      $vmwaretools_md5 = "4a2d230828919048c0c3ae8420f8edfe"
>      $releaserunner_notify_from = "Release Eng <release@mozilla.com>"
> +    $releaserunner_notify_to = "<release-drivers@mozilla.org>"

I'm a little worried about having the email address as a bare address enclosed by <>. Is this supported?  Otherwise, LGTM.
Attachment #8462696 - Flags: review?(catlee) → review+
Comment on attachment 8462696 [details] [diff] [review]
send failure mail to release-drivers

(In reply to Chris AtLee [:catlee] from comment #7)
> Comment on attachment 8462696 [details] [diff] [review]
> send failure mail to release-drivers
> 
> Review of attachment 8462696 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: manifests/moco-config.pp
> @@ +189,5 @@
> >  
> >      $vmwaretools_version = "9.4.0-1280544"
> >      $vmwaretools_md5 = "4a2d230828919048c0c3ae8420f8edfe"
> >      $releaserunner_notify_from = "Release Eng <release@mozilla.com>"
> > +    $releaserunner_notify_to = "<release-drivers@mozilla.org>"
> 
> I'm a little worried about having the email address as a bare address
> enclosed by <>. Is this supported?  Otherwise, LGTM.

I added a string in front of the <> upon landing.
Attachment #8462696 - Flags: checked-in+
Puppet will deploy this soon, and the next run of release runner will pick it up.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.