split try and non-try tegra pools

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: aki, Assigned: aki)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [android][android_tier_1])

Attachments

(2 attachments, 3 obsolete attachments)

(Assignee)

Description

7 years ago
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

7 years ago
Created attachment 558966 [details] [diff] [review]
non-functional attempt #1, configs
(Assignee)

Comment 2

7 years ago
Created attachment 558968 [details] [diff] [review]
non-functional attempt #1, custom

Spent several hours trying to change one thing and failing.
Still wishing for non-programmatic configs.
(Assignee)

Comment 3

7 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

7 years ago
Created attachment 559018 [details] [diff] [review]
split try and non-try tegra pools, buildbotcustom

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

7 years ago
Created attachment 559020 [details] [diff] [review]
split try and non-try tegra pools, buildbot-configs

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

7 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

6 years ago
Created attachment 559252 [details] [diff] [review]
hm, uploaded wrong patch earlier (buildbotcustom)
Attachment #559018 - Attachment is obsolete: true
Attachment #559018 - Flags: review?(rail)
Attachment #559252 - Flags: review?(coop)
(Assignee)

Updated

6 years ago
Attachment #559020 - Flags: review?(rail) → review?(coop)

Comment 8

6 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

6 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

6 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

6 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

6 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

6 years ago
Flags: needs-reconfig?

Comment 13

6 years ago
Merged to production and reconfigured.
Flags: needs-reconfig? → needs-reconfig+
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.