Closed
Bug 1340911
Opened 7 years ago
Closed 7 years ago
need a way to perform linux64 + linux64-stylo T-pushes (π-pushes) to reduce budget impact of stylo try runs
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Infrastructure & Operations Graveyard
CIDuty
Tracking
(firefox54 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: bholley, Assigned: bholley)
References
Details
Attachments
(1 file)
1.15 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
This might be possible in the status quo, but trychooser doesn't have an option for linux64-stylo T-pushes, so filing a bug. The answer here may just be to tell us what the right syntax actually is. When doing try pushes, we really want to build on all platforms (to avert the kind of bustage that's currently happening on autoland), and run tests on the linux64-stylo platform, as well as one non-stylo platform (linux64 works well). Otherwise, we have to choose between inadequate results, two separate pushes, or a wasteful all-platform push. I basically want something like: try: -b do -p all -u all[x64,stylo] -t none Would that syntax actually work?
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(dustin)
Comment 1•7 years ago
|
||
I tried the syntax and it doesn't have the results you would expect. I don't think we have implemented t-tests with trychooser yet, see bug 1141280
Comment 2•7 years ago
|
||
The try option syntax does support t-pushes (bug 1247343). However, I don't think "stylo" means anything in the context of a T push. The values in brackets need to be either full platform names or a pretty name defined in UNITTEST_PLATFORM_PRETTY_NAMES in taskcluster/taskgraph/try_option_syntax.py So something like `-u all [linux64,linux64-stylo]` would probably do the trick (x64 also includes linux64-asan, which you probably don't need here)
Flags: needinfo?(dustin)
Assignee | ||
Comment 3•7 years ago
|
||
Sounds good - thanks!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 4•7 years ago
|
||
So this seems to mostly work, but seems to exclude our reftest-stylo job: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a4510ac771b354ec4a0a6427394a9e0dd6459703 Dustin, what's the fix here?
Status: RESOLVED → REOPENED
Flags: needinfo?(dustin)
Resolution: WORKSFORME → ---
Comment 5•7 years ago
|
||
So, looking at the decision task: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a4510ac771b354ec4a0a6427394a9e0dd6459703&selectedJob=80539968 and looking at its full-task-graph (warning, huge) https://queue.taskcluster.net/v1/task/TgHt8oM9STKGs2QbPL9QgQ/runs/0/artifacts/public/full-task-graph.json I see the a reftest-stylo task (but I don't see it in target-tasks.json, meaning it was filtered out by try syntax): "test-linux64-stylo/debug-reftest-stylo-1": { "attributes": { "build_platform": "linux64-stylo", "build_type": "debug", "e10s": false, "kind": "test", "run_on_projects": [ "stylo", "autoland", "mozilla-inbound", "mozilla-central" ], "test_chunk": "1", "test_platform": "linux64-stylo", "unittest_flavor": "reftest-stylo", "unittest_suite": "reftest", "unittest_try_name": "reftest-stylo" }, The syntax was "try: -b do -p all -u all[linux64-stylo] -t none" I think the issue is the run_on_projects attribute. For try, that applies when `-p all` is specified (I know, I know, it'd be great if try did what you wanted it to do, but somebody else wanted it to not do what you wanted it to do). Using something with explicit platforms, like `-p linux64-stylo`, should get that test run. Alternately, adding 'try' to run_on_projects would make it run.
Flags: needinfo?(dustin)
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Dustin J. Mitchell [:dustin] from comment #5) > Using something with explicit platforms, like `-p linux64-stylo`, should get > that test run. Well sure, but that defeats the purpose of the T-push / π-push. > Alternately, adding 'try' to run_on_projects would make it run. Ok, I'll give that a shot.
Comment 7•7 years ago
|
||
OK, just keep in mind it means that those tests will run whenever anyone uses `-p all`.
Assignee | ||
Comment 8•7 years ago
|
||
(In reply to Dustin J. Mitchell [:dustin] from comment #7) > OK, just keep in mind it means that those tests will run whenever anyone > uses `-p all`. That's fine. I have a patch that seems to do the right thing: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0cc5c32cae1451b87adc8d31529d66666b3aba7d
Assignee | ||
Comment 9•7 years ago
|
||
MozReview-Commit-ID: 9skNPBTvTsi
Attachment #8842707 -
Flags: review?(dustin)
Updated•7 years ago
|
Assignee: nobody → bobbyholley
Updated•7 years ago
|
Attachment #8842707 -
Flags: review?(dustin) → review+
Comment 10•7 years ago
|
||
Pushed by bholley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dcd244eca427 Run stylo reftests on try by default. r=dustin
Comment 11•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/dcd244eca427
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Updated•6 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Updated•4 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•