Closed Bug 1436305 Opened 7 years ago Closed 7 years ago

[in-tree scheduling] shipping releases emails are lies

Categories

(Release Engineering :: Release Automation, enhancement, P1)

enhancement

Tracking

(firefox60 fixed)

RESOLVED FIXED
Tracking Status
firefox60 --- fixed

People

(Reporter: mtabara, Assigned: mtabara)

Details

(Whiteboard: [releaseduty])

Attachments

(1 file)

Whenever we trigger shipping phase, the emails are triggered as well saying the "release has been shipped". This is a lie since we only schedule them in Balrog, pending RelMan & RelEng approval. We need to amend the notifications in such a way that either: a) they mention when the change is scheduled (most likely ETA from Ship-it passed down to action task and then to in-tree scheduling) b) at least say the release has `been scheduled to be shipped in Balrog pending signoffs` instead of the current message.
How about we start with this approach for now? In the cleanup section we can improve this by passing the ETA from Ship-it via action task down to tasks, to specify the timing as well, as we have in releasetasks old logic.
Assignee: nobody → mtabara
Attachment #8949034 - Flags: review?(bhearsum)
Comment on attachment 8949034 [details] [diff] [review] Fix lies by amending the notifications message. Review of attachment 8949034 [details] [diff] [review]: ----------------------------------------------------------------- ::: taskcluster/ci/release-notify-ship/kind.yml @@ +66,5 @@ > index: > product: firefox > + notifications: > + completed: > + subject: "{task[shipping-product]} {release_config[version]} build{release_config[build_number]}/{config[params][project]} has shipped with pending updates in Balrog for signoff!" "... has shipped with pending updates" is kindof a contradiction. How about "... updates are ready for signoff in Balrog" ?
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #8949034 - Flags: review?(bhearsum)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: