Run reftest-stylo jobs on inbound, too.

RESOLVED FIXED in Firefox 57

Status

()

P3
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: KWierso, Assigned: jryans)

Tracking

unspecified
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [Stylo])

Attachments

(1 attachment)

(Reporter)

Description

a year ago
Bug 1399461 is a near-permafailing reftest-stylo job that started on inbound. Inbound is not included in the repos that reftest-stylo currently run on. 

I think that was decided because these jobs were originally tier-2 and could theoretically only break via a landing from servo, which only happens on autoland, so these jobs were only enabled on autoland, m-c and try.

These tests can clearly break from stuff landing on inbound, so we should probably have these jobs running on inbound also.


We should probably also make sure other stylo jobs aren't being restricted from running on inbound.

If capacity issues are going to be a problem, is there a way to run these jobs less frequently on inbound, but still have them scheduled to run occasionally to ease backfilling?
Priority: -- → P3
Per https://groups.google.com/forum/#!msg/mozilla.dev.platform/H_Sl_-hCF5w/W7X_loXGAAAJ , stylo became the default ("on mozilla-central" at least), as of September 5th. (via bug 1330412)

So I'd thought that *all* test runs are now "stylo" flavored, and we now have accessory "stylo-disabled" test platforms as well.  Is that not the case on inbound?
status-firefox57: --- → fix-optional
Flags: needinfo?(wkocher)
(Assignee)

Comment 2

a year ago
reftest-stylo is strange beast that (despite its name) actually runs Stylo vs. Gecko mode, as opposed to the typical test vs. reference in the same engine.

So, it runs both Gecko and Stylo style systems for comparison purposes.

This special run only happens for autoland, m-c, and try at the moment:

http://searchfox.org/mozilla-central/rev/d08b24e613cac8c9c5a4131452459241010701e0/taskcluster/ci/test/tests.yml#1209
Flags: needinfo?(wkocher)
(Assignee)

Updated

a year ago
Assignee: nobody → jryans
Status: NEW → ASSIGNED
Whiteboard: [Stylo]
Aha, thanks for clarifying!
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 6

a year ago
mozreview-review
Comment on attachment 8910527 [details]
Bug 1399742 - Use default branch list for reftest-stylo.

https://reviewboard.mozilla.org/r/181958/#review187340

glad we are making this 100% complete- lets ship Stylo!!!
Attachment #8910527 - Flags: review?(jmaher) → review+

Comment 7

a year ago
Pushed by jryans@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d1d44405008e
Use default branch list for reftest-stylo. r=jmaher
https://hg.mozilla.org/mozilla-central/rev/d1d44405008e
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox57: fix-optional → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.