Closed
Bug 1403490
Opened 7 years ago
Closed 7 years ago
enable mochitest-gpu tests on taskcluster windows 10
Categories
(Infrastructure & Operations :: RelOps: General, task)
Infrastructure & Operations
RelOps: General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: grenade, Assigned: grenade)
References
Details
Attachments
(2 files)
suite mochitest-gpu succeeds for taskcluster windows 10 builds, it would be good to have the windows 10 gpu tests that succeed running regularly.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8912640 [details]
Bug 1403490 - enable mochitest-gpu on tc win 10;
https://reviewboard.mozilla.org/r/183970/#review189164
excellent
Attachment #8912640 -
Flags: review?(jmaher) → review+
Assignee | ||
Updated•7 years ago
|
Assignee: relops → rthijssen
Keywords: checkin-needed
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e8737573b0e0
enable mochitest-gpu on tc win 10; r=jmaher
Keywords: checkin-needed
Comment 4•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 5•7 years ago
|
||
And backed out in https://hg.mozilla.org/mozilla-central/rev/e1f660fc493bc4bf2f91b6df94bc98e8e3840c42 because according to https://public-artifacts.taskcluster.net/RNMMR272QPWy3xRj2KUG4Q/0/public/logs/live_backing.log you have to do something about a supersederUrl.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•7 years ago
|
||
hey Dustin, do you know what the supersederUrl is and what i could or should do differently in this commit? just looking for pointers or a shrug as i haven't a clue about the exception thrown.
Flags: needinfo?(dustin)
Comment 7•7 years ago
|
||
Looks like this is coming from coscaling:
https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/transforms/task.py#833-835
Just speculating.... Maybe that is not a coscaled task?
Assignee | ||
Comment 8•7 years ago
|
||
pmoore: asking because i've seen your comments about windows test coalescing, do you have any idea what this supersederUrl exception is about? it seems odd that the attribute is set in the build_docker_worker_payload function since i don't understand why that should run for generic-worker payloads...
Flags: needinfo?(pmoore)
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8912640 [details]
Bug 1403490 - enable mochitest-gpu on tc win 10;
https://reviewboard.mozilla.org/r/183970/#review189722
Attachment #8912640 -
Flags: review+
Comment 10•7 years ago
|
||
I'm guessing this workerType doesn't have a new enough version of generic-worker. It was only implemented in generic-worker a month or two ago.
Flags: needinfo?(dustin)
Comment 11•7 years ago
|
||
(In reply to Dustin J. Mitchell [:dustin] from comment #10)
> I'm guessing this workerType doesn't have a new enough version of
> generic-worker. It was only implemented in generic-worker a month or two
> ago.
That's correct. Sorry about that!
To roll this out you'll need to do one of the following:
1) Upgrade gecko-t-win10-64-gpu to generic-worker 10.2.2 or higher (ideally 10.2.3)
OR
2) Add "gecko-t-win10-64-gpu" to this list https://hg.mozilla.org/mozilla-central/file/76a26ef7c493/taskcluster/taskgraph/transforms/coalesce.py#l34
OR
3) Depend on bug 1399401 (i.e. wait for it to land first)
It looks like gecko-t-win10-64-gpu is currently running generic-worker 8.3.0.
Depends on: 1399401
Flags: needinfo?(pmoore)
Comment 12•7 years ago
|
||
Option 2 is no doubt the simplest...
Comment 13•7 years ago
|
||
Attachment #8913295 -
Flags: review?(rthijssen)
Assignee | ||
Updated•7 years ago
|
Attachment #8913295 -
Flags: review?(rthijssen) → review+
Assignee | ||
Comment 14•7 years ago
|
||
please land both patches either together or first land the one from pmoore, followed by mine.
Keywords: checkin-needed
Comment 15•7 years ago
|
||
Pushed by pmoore@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ea65750e9e40
disable superseding for worker type gecko-t-win10-64-gpu,r=grenade
https://hg.mozilla.org/integration/mozilla-inbound/rev/82f07257d932
enable mochitest-gpu on tc win 10; r=jmaher
Keywords: checkin-needed
Comment 16•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ea65750e9e40
https://hg.mozilla.org/mozilla-central/rev/82f07257d932
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•