1552960 - Build bustage after bug 1551738: Exception: No platform matching u'thunderbird-source' nor 'default' found while determining item `s3_bucket_paths` in `s3_bucket_paths`

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Jorg K (CEST = GMT+2)]

Attachment: WIP, not working all all.

Running the taskcluster tool locally, I get:
0:03.91 Error loading tasks for kind beetmover-source:
[snip]
Exception: No platform matching u'thunderbird-source' nor 'default' found while determining item s3_bucket_paths in s3_bucket_paths

I tried to port a few bits of bug 1551738 with no success. I'll attach a WIP.

Try runs:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=e6a347496e9762f8d17db9d8a3ad656e31d8f9d6
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=e28eae60d261c508ef3ed670fbe434f325da07b2

This may also need M-C changes, perhaps here:
https://hg.mozilla.org/try/rev/d730625e539d7268b6afe838f7c5b30dcb87b448

Maybe Geoff wants to spend some time on this. Or Tom can give us a hint.

Comment 1

[Jorg K (CEST = GMT+2)]

Ported some from here: https://hg.mozilla.org/mozilla-central/rev/7144d26d331ccd17dc9c5b3a79f424fc53a0f1dc but still with "Firefox" in the config which doesn't appear to be right.

Comment 2

[Tom Prince [:tomprince]]

Rather than modifying m-c, the solution is to copy the artifact manifest to c-c, and change the artifact_map's to point at the c-c copies.

Comment 3

[Jorg K (CEST = GMT+2)]

Attachment: pp.patch

Thanks, Tom. This one works but needs some clean-up.

Comment 4

[Jorg K (CEST = GMT+2)]

Attachment: pp.patch

I'm going to land this now and then Rob can to a clean-up around the edges.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/7566abd1aa2c
Port parts of bug 1551738 (rev 7144d26d331ccd17dc9c5b3a79f424fc53a0f1dc) and add artifact manifests. rs=bustage-fix

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/ac1f260073e5
Follow-up: remove yaml separator --- from copied files. rs=bustage-fix DONTBUILD

Comment 7

[Rob Lemley [:rjl]]

Attachment: 36375.patch

We don't need the custom transform function anymore. Removing it will fix the validation error as the .xpi file was listed twice.

Comment 8

[Jorg K (CEST = GMT+2)]

Comment on attachment 9066474 [details] [diff] [review]
36375.patch

OK. Are you actually saying that all the stuff I landed was alright? Have you looked at it? Does some of it need to be removed? I just dumped all the FF files in with only some minor tweaks.

Comment 9

[Rob Lemley [:rjl]]

Attachment: 36376.patch

It was trying to upload checksums to an s3 bucket named "default" instead of "pub/thunderbird/nightly"

Comment 10

[Jorg K (CEST = GMT+2)]

Comment on attachment 9066483 [details] [diff] [review]
36376.patch

Interesting, I should really know more about that stuff.

Comment 11

[Rob Lemley [:rjl]]

Attachment: 36377.patch

Cleaned up the manifest files.

Comment 12

[Jorg K (CEST = GMT+2)]

Comment on attachment 9066512 [details] [diff] [review]
36377.patch

Thanks, I knew some of that stuff needed to be pulled out, but at 12:30 AM I didn't have the patience.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/49774aa1a8ef
Follow-up: Remove custom Thunderbird Langpack transform. r=jorgk
https://hg.mozilla.org/comm-central/rev/cca897225fa9
Follow-up: Fix S3 bucket path for checksum uploads. r=jorgk
https://hg.mozilla.org/comm-central/rev/cfeeca7996f6
Follow-up: Cleanup Beetmover manifests. r=jorgk
https://hg.mozilla.org/comm-central/rev/93545cdce0cc
Follow-up: Restore yaml separator --- since it's required now. r=me DONTBUILD