Closed Bug 1463730 Opened 2 years ago Closed 2 years ago
Small changes to the wpt manifest upload task
59 bytes, text/x-review-board-request
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 email@example.com: https://hg.mozilla.org/integration/autoland/rev/d67b21a431e5 Changes to wpt manifest upload task, r=jgraham
You need to log in before you can comment on or make changes to this bug.