Closed
Bug 685242
Opened 13 years ago
Closed 13 years ago
split try and non-try tegra pools
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: mozilla)
Details
(Whiteboard: [android][android_tier_1])
Attachments
(2 files, 3 obsolete files)
12.86 KB,
patch
|
coop
:
review+
bear
:
feedback+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
3.29 KB,
patch
|
coop
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
Tegras are multi-colored right now. One hypothesis is that try runs are leaving the tegras in a fragile state. If we can split the tegras up into try- and non-try- it may make us more stable. https://tbpl.mozilla.org/?tree=Mozilla-Inbound&usebuildbot=1&jobname=Android https://tbpl.mozilla.org/?tree=Firefox&usebuildbot=1&jobname=Android https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&jobname=Android
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Comment 2•13 years ago
|
||
Spent several hours trying to change one thing and failing. Still wishing for non-programmatic configs.
Assignee | ||
Comment 3•13 years ago
|
||
Actually, this patch works. dump_masters.sh doesn't output slave names. Cleaning up the patches and asking for r?
Assignee | ||
Comment 4•13 years ago
|
||
Let me know if this is the right approach. When I alter dump_master.py to print b['slavenames'], then I see the separate tegra pools.
Attachment #558968 -
Attachment is obsolete: true
Attachment #559018 -
Flags: review?(rail)
Assignee | ||
Comment 5•13 years ago
|
||
configs portion. Bear, let me know if the split makes sense. Ideally this is the first stab at it and we can adjust the pools at will.
Attachment #558966 -
Attachment is obsolete: true
Attachment #559020 -
Flags: review?(rail)
Attachment #559020 -
Flags: feedback?(bear)
Comment 6•13 years ago
|
||
Comment on attachment 559020 [details] [diff] [review] split try and non-try tegra pools, buildbot-configs your upper end could be changing very soon with the new additions, but yea rougly a 1/3 makes sense now
Attachment #559020 -
Flags: feedback?(bear) → feedback+
Assignee | ||
Comment 7•13 years ago
|
||
Attachment #559018 -
Attachment is obsolete: true
Attachment #559018 -
Flags: review?(rail)
Attachment #559252 -
Flags: review?(coop)
Assignee | ||
Updated•13 years ago
|
Attachment #559020 -
Flags: review?(rail) → review?(coop)
Comment 8•13 years ago
|
||
Comment on attachment 559252 [details] [diff] [review] hm, uploaded wrong patch earlier (buildbotcustom) Review of attachment 559252 [details] [diff] [review]: ----------------------------------------------------------------- Looks good.
Attachment #559252 -
Flags: review?(coop) → review+
Comment 9•13 years ago
|
||
Comment on attachment 559020 [details] [diff] [review] split try and non-try tegra pools, buildbot-configs Review of attachment 559020 [details] [diff] [review]: ----------------------------------------------------------------- Guess I don't know much about how try works for mobile, but are no changes required in buildbot-configs/mozilla/*_config.py? Looks good otherwise.
Attachment #559020 -
Flags: review?(coop) → review+
Assignee | ||
Comment 10•13 years ago
|
||
(In reply to Chris Cooper [:coop] from comment #9) > Comment on attachment 559020 [details] [diff] [review] > split try and non-try tegra pools, buildbot-configs > > Review of attachment 559020 [details] [diff] [review]: > ----------------------------------------------------------------- > > Guess I don't know much about how try works for mobile, but are no changes > required in buildbot-configs/mozilla/*_config.py? > > Looks good otherwise. I *think* since I'm only updating pools for the tegras inside misc.generateTalosBranchObjects(), I'm ok here.
Assignee | ||
Comment 11•13 years ago
|
||
Comment on attachment 559020 [details] [diff] [review] split try and non-try tegra pools, buildbot-configs http://hg.mozilla.org/build/buildbot-configs/rev/b1174cccecbf
Attachment #559020 -
Flags: checked-in+
Assignee | ||
Comment 12•13 years ago
|
||
Comment on attachment 559252 [details] [diff] [review] hm, uploaded wrong patch earlier (buildbotcustom) http://hg.mozilla.org/build/buildbotcustom/rev/7c9b75d464c2
Attachment #559252 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Flags: needs-reconfig?
Comment 13•13 years ago
|
||
Merged to production and reconfigured.
Flags: needs-reconfig? → needs-reconfig+
Assignee | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•