Closed
Bug 1186764
Opened 10 years ago
Closed 9 years ago
trychooser thinks graphene and horizon builds are part of -p all
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: glandium, Assigned: philor)
References
Details
(Keywords: trychooser)
Attachments
(1 file)
3.76 KB,
patch
|
Callek
:
review+
philor
:
checked-in+
|
Details | Diff | Splinter Review |
STR:
- open trychooser
- select "both" build types
- select "all" platforms
- note that computed syntax uses "-p all" and that graphene and horizon jobs are checked.
- push to try with that syntax, graphene and horizon jobs are not started.
The same is true for linux64-mulet, but that's a separate problem, because those run on m-i, so they should be part of -p all.
Updated•10 years ago
|
Keywords: trychooser
Comment 1•10 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #0)
> STR:
> - open trychooser
> - select "both" build types
> - select "all" platforms
> - note that computed syntax uses "-p all" and that graphene and horizon jobs
> are checked.
> - push to try with that syntax, graphene and horizon jobs are not started.
Graphene and horizon are only enabled on larch, so you'd probably need to be pushing a larch-based tree that included the relevant mozconfigs. Is that what you were doing and it still didn't work, or is your point that we shouldn't be scheduling graphene and horizon builds via "-p all" in the first place?
Reporter | ||
Comment 2•10 years ago
|
||
My point is that what actually happens and what trychooser says will happen don't match.
Assignee | ||
Comment 3•9 years ago
|
||
He meant this.
Updated•9 years ago
|
Attachment #8716756 -
Flags: review?(coop) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8716756 [details] [diff] [review]
Give Graphene and Horizon the "(not run by default)" text and behavior, since they are "try_by_default": False
https://hg.mozilla.org/build/tools/rev/787ac3797a6f
Attachment #8716756 -
Flags: checked-in+
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•