Closed Bug 1398256 Opened 7 years ago Closed 7 years ago

Only schedule reftest-gpu on Win7

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla57

People

(Reporter: RyanVM, Assigned: jmaher)

Details

Attachments

(1 file)

In my Try runs that run basically every test suite under the sun, I can see reftest-gpu (Rg) getting scheduled on Win10 TC runs. That suite is only applicable to Win7 and should be skipped everywhere else.
we do not allow win8/win10 to run on any branches, so something fishy is going on here:
http://searchfox.org/mozilla-central/source/taskcluster/ci/test/tests.yml#1158

I assume this is something more deep in taskcluster configuration, :dustin- could you help figure this out?
Flags: needinfo?(dustin)
run-on-projects only applies for -p all / -u all.  If you specify a job explicitly in try, the job you specified will run.  If we truly never run win8/win10, then the test tasks should not be generated in the first place (which would mean removing it from the relevant test-sets).
Flags: needinfo?(dustin)
:dustin, thanks for explaining how that works, I learned something new!

:ryanvm, this seems pretty straightforward
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8906256 - Flags: review?(ryanvm)
Comment on attachment 8906256 [details] [diff] [review]
reftest-gpu as a unique test-set and only on win32

Review of attachment 8906256 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. Can we maintain alphabetical ordering still in test-platforms.yml? Thanks!
Attachment #8906256 - Flags: review?(ryanvm) → review+
Pushed by jmaher@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7d96ffadc552
Only schedule reftest-gpu on Win7. r=RyanVM
https://hg.mozilla.org/mozilla-central/rev/7d96ffadc552
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.