1453270 - taskgraph: Make snap tasks support esr60

Status: RESOLVED FIXED

(Release Engineering Graveyard :: Release Automation: Snap, enhancement)

Description

[Johan Lorenzo [:jlorenzo]]

Generating a taskgraph with esr60 configs[1] leads to some discrepancies. The things I've noticed are:

* release-snap-push-firefox
  * bad scope "project:releng:snaocraft:firefox:mock"
  * bad worker type

* release-snap-repackage-firefox
  * I haven't checked, but I think langpacks URL is busted.

There are probably other issues I didn't see.

[1] https://bugzilla.mozilla.org/show_bug.cgi?id=1434889#c4

Comment 1

[Johan Lorenzo [:jlorenzo]]

NI'ing myself to look at it.

Comment 2

[Johan Lorenzo [:jlorenzo]]

We can't use the same Snap Store channel on ESR because ESR doesn't fit the regular Store model of edge -> beta -> candidate -> release. However, they support this case[1], by creating a new track. I asked for it at [2]

[1] https://docs.snapcraft.io/reference/channels#track
[2] https://forum.snapcraft.io/t/firefox-please-create-the-track-esr/5006

Comment 3

[Johan Lorenzo [:jlorenzo]]

(In reply to Johan Lorenzo [:jlorenzo] from comment #0)
> * release-snap-push-firefox
>   * bad scope "project:releng:snaocraft:firefox:mock"
>   * bad worker type

Fixed in upcoming patch.


> * release-snap-repackage-firefox
>   * I haven't checked, but I think langpacks URL is busted.
I checked. It looks good with the data discussed in bug 1453261 comment 4

Comment 4

[Johan Lorenzo [:jlorenzo]]

Attachment: Bug 1453270 - taskgraph: Make snap tasks support esr60

Review commit: https://reviewboard.mozilla.org/r/236914/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/236914/

Comment 5

[Johan Lorenzo [:jlorenzo]]

Attachment: [pushsnapscript] PR: Add support for the esr track

Comment 6

[Rail Aliiev [:rail]]

Comment on attachment 8968214 [details]
Bug 1453270 - taskgraph: Make snap tasks support esr60

https://reviewboard.mozilla.org/r/236914/#review242672

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/17905b182e88
taskgraph: Make snap tasks support esr60 r=rail

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/17905b182e88

Comment 9

[Johan Lorenzo [:jlorenzo]]

Reopening to get this landed in 60.

Comment 10

[Johan Lorenzo [:jlorenzo]]

Landed on:
* release https://hg.mozilla.org/releases/mozilla-release/rev/31788214d6b483767ed755c06b07196df5a595ed 
* esr60 https://hg.mozilla.org/releases/mozilla-esr60/rev/7055a4f07f2df099a8963556b333904184eb7786

Comment 11

[Johan Lorenzo [:jlorenzo]]

It turns out I forgot to to deploy the changes onto the machine: https://tools.taskcluster.net/groups/dqhcS8eKS0OIF2udsb292A/tasks/L0AZfbxQSWiHR3J_CQfnMw/runs/0/logs/public%2Flogs%2Flive_backing.log#L22. Reopening bug.

Comment 12

[Johan Lorenzo [:jlorenzo]]

Attachment: Bug 1453270 - pushsnap_scriptworker: bump pushsnapscript to 0.2.1 a=versionbump

Review commit: https://reviewboard.mozilla.org/r/244002/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/244002/

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/build/puppet/rev/6e7562b0ccc6
pushsnap_scriptworker: bump pushsnapscript to 0.2.1 a=versionbump

Comment 14

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8975795 [details]
Bug 1453270 - pushsnap_scriptworker: bump pushsnapscript to 0.2.1 a=versionbump

Landed on production at: https://hg.mozilla.org/build/puppet/rev/c2d963b335f576e37b94fb27430f037867100adc

Comment 15

[Johan Lorenzo [:jlorenzo]]

This was lacking credentials, which I'm adding in bug 1462066.