Run reftest-stylo jobs on inbound, too.

RESOLVED FIXED in Firefox 57

Status

()

Core
CSS Parsing and Computation
P3
normal
RESOLVED FIXED
9 months ago
8 months ago

People

(Reporter: KWierso, Assigned: jryans)

Tracking

unspecified
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

(Whiteboard: [Stylo])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

9 months 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?

Updated

9 months ago
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

8 months 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

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

Comment 6

8 months 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

8 months 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: 8 months 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.