Open
Bug 1279132
Opened 8 years ago
Updated 2 years ago
mach try doesn't appear to know about mochitest-chrome
Categories
(Developer Infrastructure :: Try, defect)
Developer Infrastructure
Try
Tracking
(Not tracked)
NEW
People
(Reporter: aswan, Unassigned)
References
Details
I'm not sure I've got the details exactly right here but the best of my understanding is:
On linux64 debug builds, try uses the flag "-u mochitest-chrome", as opposed to "-u mochitest-o" on other platforms (perhaps related to taskcluster migration?). But mach try doesn't appear to know anything about mochitest-chrome: running "mach try -b d -p linux64 path/with/chrome-mochitests" produces a try command with mochitest-o. A resulting try job then runs a bunch of non-chrome tests and doesn't actually run the chrome mochitests. Very frustrating.
The discrepancy between different platforms may well be unfixable (e.g., no valid try syntax accurately captures "mach try -b d -plinux64,win64 path/with/chrome-mochitests") but for the time being, making a try run that includes chrome mochitests on linux64 debug produce a fatal error would remove the easy pitfall.
Comment 1•8 years ago
|
||
:armenzg -- Can you clarify the history here? I'm wondering why mochitest-oth was split into mochitest-a11y and mochitest-chrome for taskcluster, and if that's the "right" approach for mochitest-other on other platforms. (I blindly copied the same mochitest-a11y/chrome task definitions for linux64-opt and linux64-pgo, but am second-guessing that in view of this bug.)
Flags: needinfo?(armenzg)
Comment 2•8 years ago
|
||
I don't remember the exact reason of the split. jmaher and I workerd on it. IIRC it was to make the jobs run faster.
Just to be on the same page, here's a summary:
I see the mochitest-chrome split for Linux 64 (opt,debug,pgo) and Linux (debug only) [1]
I see mochitest-other running for all platforms except Linux64 debug (running on TaskCluster) [2]
Should we split mochitest-other in here?
I believe we can temporarily adjust the try syntax parser for the TC jobs [3] to include mochitest-chrome when mochitest-o is specified.
That way we won't have to split all the jobs in here.
[1] https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=3c5025f98e561a20e24d97c91a9e4e0ec28015ea&filter-searchStr=mochitest-chrome&group_state=expanded
[2] https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=3c5025f98e561a20e24d97c91a9e4e0ec28015ea&filter-searchStr=mochitest-other&group_state=expanded
[3] https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/try_option_syntax.py#69
Flags: needinfo?(armenzg)
Comment 3•8 years ago
|
||
(In reply to Armen Zambrano [:armenzg] - Engineering productivity from comment #2)
Thanks Armen.
> I believe we can temporarily adjust the try syntax parser for the TC jobs
> [3] to include mochitest-chrome when mochitest-o is specified.
> That way we won't have to split all the jobs in here.
That sounds like a good idea to me.
Updated•6 years ago
|
Component: General → Try
Product: Testing → Firefox Build System
Version: Version 3 → unspecified
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•