Closed Bug 1463730 Opened 2 years ago Closed 2 years ago

Small changes to the wpt manifest upload task

Categories

(Testing :: web-platform-tests, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: cactusmachete, Assigned: cactusmachete)

Details

Attachments

(1 file)

Gzipping the manifest speeds things up considerably. Also needed to remove some redundancy from the task name and have it run when there are changes in web-platform/mozilla/tests/*.
Comment on attachment 8979920 [details]
Bug 1463730 - Changes to wpt manifest upload task,

https://reviewboard.mozilla.org/r/246106/#review252184

::: taskcluster/ci/source-test/wpt-manifest.yml:5
(Diff revision 1)
> +job-defaults:
> +    platform: linux64/opt
> +    treeherder:
> +        kind: test
> +        tier: 1

I think this should be reduced to tier: 2 since the sheriffs aren't going to back out over failures.

::: taskcluster/ci/source-test/wpt-manifest.yml:23
(Diff revision 1)
> +    run:
> +        using: run-task
> +        command: >
> +            cd /builds/worker/checkouts/gecko
> +            && ./mach wpt-manifest-update
> +            && gzip testing/web-platform/meta/MANIFEST.json

We should also upload the mozilla-specific manifest here; that will be in `testing/web-platform/mozilla/meta/MANIFEST.json`. Maybe we should tar them together and upload as a single file; that seems like it should have better performance.
Attachment #8979920 - Flags: review?(james) → review-
Comment on attachment 8979920 [details]
Bug 1463730 - Changes to wpt manifest upload task,

https://reviewboard.mozilla.org/r/246106/#review252298
Attachment #8979920 - Flags: review?(james) → review+
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/d67b21a431e5
Changes to wpt manifest upload task, r=jgraham
https://hg.mozilla.org/mozilla-central/rev/d67b21a431e5
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.