Closed
Bug 911956
Opened 11 years ago
Closed 11 years ago
Clean up Android and B2G presentation on TBPL
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: RyanVM)
References
Details
Attachments
(4 files)
2.07 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
1.16 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
2.18 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
2.51 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
At the moment we have:
Data.js
{
/b2g.*_ics_armv7a_gecko_emulator_vm/i.test(name) ? "b2g-arm-vm" :
/b2g.*_ics_armv7a/i.test(name) ? "b2g-arm" :
/b2g.*_emulator-jb/i.test(name) ? "b2g-emu-jb" :
/b2g.*_emulator_vm/i.test(name) ? "b2g-emu-ics-vm" :
/b2g.*_emulator/i.test(name) ? "b2g-emu-ics" :
}
Config.js
{
"b2g-arm": "B2G Arm",
"b2g-arm-vm": "B2G Arm VM",
"b2g-emu-ics": "B2G ICS Emu",
"b2g-emu-ics-vm": "B2G ICS Emu VM",
"b2g-emu-jb": "B2G JB Emu",
}
I was thinking it would make to consolidate "b2g-emu-ics-vm" into "b2g-emu-ics", given that on everywhere apart from Cedar, we don't overlap the suites between EC2 emulator and releng hardware run emulator. Not sure why we still do that on Cedar - suspect the releng hardware jobs can be switched off - filed bug 911953 for that. If they get switched off, great we can consolidate entirely - if they don't, what do you think about renaming "B2G ICS Emu VM" to "B2G ICS Emu" and "B2G ICS Emu" to "B2G ICS Emu HW" (since the latter would only appear on Cedar). This would mean the B2G ICS opt builds appear on the same line as the emulator tests jobs, rather than having the debug build stuck inbetween them on all trees.
Also "b2g-arm-vm" doesn't seem to exist anywhere now, so can be gotten rid of (and even if it did, for the same reasons as above, think we should get rid of it, to consolidate into "b2g-arm").
Ryan - would you like to take a look at this too? :-)
Assignee | ||
Comment 1•11 years ago
|
||
I would *love* to take this. I was just thinking last night about filing another "clean up Android/B2G" bug. Mind if I repurpose this bug for that? :)
Assignee: nobody → ryanvm
Assignee | ||
Comment 2•11 years ago
|
||
I'm also thinking that it would be nice to get all device image builds on one line instead of the current format.
Reporter | ||
Comment 3•11 years ago
|
||
sgtm on both counts :-)
Assignee | ||
Updated•11 years ago
|
Summary: Consolidate B2G emulator job types → Clean up Android and B2G presentation on TBPL
Assignee | ||
Comment 4•11 years ago
|
||
Calling x86 builds "Android 2.2" is a bit of a misnomer. Since the soon-to-be-running tests will be on top of 4.2, let's just go ahead and rename it.
Attachment #798863 -
Flags: review?(emorley)
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #798867 -
Flags: review?(emorley)
Assignee | ||
Comment 6•11 years ago
|
||
This will look a bit funny on Cedar and b2g18* until the redundant rev3 jobs are turned off, but it doesn't sound like we'll get any real resistance to doing so.
Attachment #798868 -
Flags: review?(emorley)
Assignee | ||
Comment 7•11 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #2)
> I'm also thinking that it would be nice to get all device image builds on
> one line instead of the current format.
This is still something I want to do, but it's going to be more work than I originally thought. I'm going to split it off to a new bug.
Assignee | ||
Comment 8•11 years ago
|
||
Put Android l10n nightlies after the Android nightly build. Also, I moved the l10n group to the top of the groupedMachineTypes list to preserve alphabetical order.
Attachment #798878 -
Flags: review?(emorley)
Reporter | ||
Updated•11 years ago
|
Attachment #798863 -
Flags: review?(emorley) → review+
Reporter | ||
Updated•11 years ago
|
Attachment #798867 -
Flags: review?(emorley) → review+
Reporter | ||
Updated•11 years ago
|
Attachment #798868 -
Flags: review?(emorley) → review+
Reporter | ||
Comment 9•11 years ago
|
||
Comment on attachment 798878 [details] [diff] [review]
Put Android l10n nightlies after the Android nightly build
Thank you - all looks good :-)
Love cleanup like this!
Attachment #798878 -
Flags: review?(emorley) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Reporter | ||
Comment 12•11 years ago
|
||
In production :-)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•