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)
Release Engineering
Release Automation
Tracking
(firefox60 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: mtabara, Assigned: mtabara)
Details
(Whiteboard: [releaseduty])
Attachments
(1 file)
4.90 KB,
patch
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•7 years ago
|
||
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 2•7 years ago
|
||
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" ?
Comment 3•7 years ago
|
||
This might be causing https://bugzilla.mozilla.org/show_bug.cgi?id=1436501
Pushed by mtabara@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/faaa816fdb9d
fix notifications lies. r=bhearsum a=release
Assignee | ||
Comment 5•7 years ago
|
||
Comment 6•7 years ago
|
||
bugherder |
Updated•7 years ago
|
Attachment #8949034 -
Flags: review?(bhearsum)
You need to log in
before you can comment on or make changes to this bug.
Description
•