1297829 - Use push inspector URL in release emails

Status: RESOLVED FIXED

(Release Engineering :: Release Automation, defect)

Description

[Rail Aliiev [:rail]]

because https://tools.taskcluster.net/push-inspector is cool!

Comment 1

[Connor Sheehan [:sheehan]]

Attachment: Adds push-inspector url for task group to SES emails.

Comment 2

[Rail Aliiev [:rail]]

Comment on attachment 8786070 [details] [review]
Adds push-inspector url for task group to SES emails.

Oooop, sorry, I wasn't clear enough. The release emails are good, they point you to a particular task. What I meant is the initial email sent to release-drivers, https://dxr.mozilla.org/build-central/source/tools/lib/python/kickoff/__init__.py#105

BTW, once you touched pulse-notify, can you simplify the email template and remove the redundant CSS styles? ;)

Comment 3

[Mihai Tabara [:mtabara]⌚️GMT]

I think it's worth switching to push-inspector in releasewarrior too (e.g. task graph from https://github.com/mozilla/releasewarrior/blob/master/releases/firefox-beta-49.0b7.md)

Comment 4

[Rail Aliiev [:rail]]

(In reply to Mihai Tabara [:mtabara] from comment #3)
> I think it's worth switching to push-inspector in releasewarrior too (e.g.
> task graph from
> https://github.com/mozilla/releasewarrior/blob/master/releases/firefox-beta-
> 49.0b7.md)

Good idea, but the PR link is 404! :P

Comment 5

[Connor Sheehan [:sheehan]]

Attachment: Changes task-graph-inspector link to push-inspector.

As far as I can tell the push-inspector uses the same id as the task-graph-inspector, so changing to the new URL should be enough.

Comment 6

[Rail Aliiev [:rail]]

Comment on attachment 8787296 [details] [review]
Changes task-graph-inspector link to push-inspector.

https://hg.mozilla.org/build/tools/rev/fc03b8cd3062

Comment 7

[Mihai Tabara [:mtabara]⌚️GMT]

The email for Build of Firefox-49.0b9-build1 has successfully changed \o/

--
A new build has been submitted through ship-it:

Commit: https://hg.mozilla.org/releases/mozilla-beta/rev/e3cc699ccef2abb7075c39e7b9a081718eb9c159
Task group: https://tools.taskcluster.net/push-inspector/#/...

Created by ...
Started by ...
--

As to ...

comment #4)
> (In reply to Mihai Tabara [:mtabara] from comment #3)
> > I think it's worth switching to push-inspector in releasewarrior too (e.g.
> > task graph from
> > https://github.com/mozilla/releasewarrior/blob/master/releases/firefox-beta-
> > 49.0b7.md)
> 
> Good idea, but the PR link is 404! :P

Fair enough :) I'll redirect to the source of truth then:

https://github.com/mozilla/releasewarrior/blob/master/templates/firefox_beta.md.tmpl
https://github.com/mozilla/releasewarrior/blob/master/templates/firefox_esr.md.tmpl
https://github.com/mozilla/releasewarrior/blob/master/templates/firefox_release-rc.md.tmpl
https://github.com/mozilla/releasewarrior/blob/master/templates/firefox_release.md.tmpl

Comment 8

[Rail Aliiev [:rail]]

Thanks!

Comment 9

[Connor Sheehan [:sheehan]]

Attachment: Changes task-graph-inspector link to push-inspector in releasewarrior

Comment 10

[Rail Aliiev [:rail]]

Comment on attachment 8787645 [details] [review]
Changes task-graph-inspector link to push-inspector in releasewarrior

r+ed in the PR, merged

Comment 11

[Rail Aliiev [:rail]]

Thanks, Connor!