The default bug view has changed. See this FAQ.

adjust e-mail for update_shipping builder to be more intelligible

RESOLVED FIXED

Status

Release Engineering
Release Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bhearsum, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
This is a mail that gets sent out to release-drivers, but it's just says "update_shipping complete for Firefox 32.0b1 build2". It should probably say something like "Updates available on beta/release/esr" channel, like we do for betatest updates. The release channel is specified in the release config now, so this should be do-able...
(Assignee)

Comment 1

3 years ago
Created attachment 8462596 [details] [diff] [review]
allow channel as substitution for release mails

I was considering changing the "Updates available on betatest" message as part of this patch, too, but I think we should wait until we adjust the channel names in bug 986990 to do that. It may require splitting out "testChannels" into two vars, too.

I think I should hold off landing these until after Balrog is in production for the Beta channel - otherwise it could lead to confusion for QA.
Attachment #8462596 - Flags: review?(nthomas)
(Assignee)

Comment 2

3 years ago
Created attachment 8462597 [details] [diff] [review]
add mail template for update shipping builder
Attachment #8462597 - Flags: review?(nthomas)
(Assignee)

Comment 3

3 years ago
I did a test run in staging, and got mail like:
Date: Fri, 25 Jul 2014 07:40:08 -0700
From: release@mozilla.com
To: bhearsum@mozilla.com
Subject: [release] Firefox 32.0b1 build1: Updates available on beta

Updates to Firefox 32.0b1 build1 are available on the beta channel now.

-buildbot

Updated

3 years ago
Attachment #8462596 - Flags: review?(nthomas) → review+
Comment on attachment 8462597 [details] [diff] [review]
add mail template for update shipping builder

Or
Updates to %(releaseName)s are now available on the %(releaseChannel)s channel.

Probably just used to reading it in that form.
Attachment #8462597 - Flags: review?(nthomas) → review+
(Assignee)

Comment 5

3 years ago
(In reply to Nick Thomas [:nthomas] from comment #4)
> Comment on attachment 8462597 [details] [diff] [review]
> add mail template for update shipping builder
> 
> Or
> Updates to %(releaseName)s are now available on the %(releaseChannel)s
> channel.
> 
> Probably just used to reading it in that form.

Yeah, let's use that. It sounds better.
(Assignee)

Updated

3 years ago
Attachment #8462596 - Flags: checked-in+
(Assignee)

Comment 6

3 years ago
Comment on attachment 8462597 [details] [diff] [review]
add mail template for update shipping builder

I landed the first patch (which does nothing on its own), this one will hold off until we switch.
something(s) here went to production today
(Assignee)

Comment 8

3 years ago
(In reply to Justin Wood (:Callek) from comment #7)
> something(s) here went to production today

It didn't, it was backed out.
(Assignee)

Comment 9

3 years ago
Comment on attachment 8462597 [details] [diff] [review]
add mail template for update shipping builder

Planning to land this on Thursday as part of pushing Balrog to the beta channel in bug 1027608.
(Assignee)

Updated

3 years ago
Attachment #8462597 - Flags: checked-in+
In prod with reconfig on 2014-09-18 08:14 PT
(Assignee)

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.