Closed
Bug 1463730
Opened 6 years ago
Closed 6 years ago
Small changes to the wpt manifest upload task
Categories
(Testing :: web-platform-tests, enhancement)
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 hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
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 hidden (mozreview-request) |
Comment 4•6 years ago
|
||
mozreview-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
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d67b21a431e5
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•