Closed Bug 1191655 Opened 9 years ago Closed 9 years ago

Cleanup the B2G buildbot configs

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: RyanVM)

Details

Attachments

(1 file)

Lots of old cruft from branches we no longer support, dead test suites, etc.
Attached patch b2g-desktopSplinter Review
mozilla-tests/b2g_config.py     |  745 +---------------------------------------
mozilla/b2g_config.py           |  134 +------
mozilla/b2g_project_branches.py |   12 
3 files changed, 66 insertions(+), 825 deletions(-)

Builders added:
+ b2g_mozilla-b2g37_v2_2r_linux64-b2g-haz_dep
Builders removed
- b2g_emulator_vm_large cedar opt test mochitest-media
- b2g_mozilla-b2g37_v2_2_macosx64_gecko build
- b2g_mozilla-b2g37_v2_2_macosx64_gecko nightly
- b2g_mozilla-b2g37_v2_2_macosx64_gecko-debug build
- b2g_mozilla-b2g37_v2_2_win32_gecko build
- b2g_mozilla-b2g37_v2_2_win32_gecko nightly
- b2g_mozilla-b2g37_v2_2_win32_gecko-debug build

The added/removed builders are intentional here. I got approval from jryans to drop the OSX/Win B2G desktop builds from b2g37 since it's code complete. And we should have been running the B2G hazard build on v2.2r from the start since it verifies security and stability in our GC engine.

Explicitly locking platforms on b2g37 will also make it easier to further prune more builds down the road without having to have tons of special-casing scattered about.
Attachment #8644125 - Flags: review?(catlee)
I guess one change that might be considered a bit suspect is the graphene change in mozilla/b2g_config.py. Arguably, my change makes it harder to selectively enable on other branches down the road. But I made the change because it seems likely that when that day comes, we're likely to be enabling it across the board as it's presumably ready to not live only on Try and a project branch anymore.

And in general, I think it's more understandable for non-default jobs to be opted into rather than added globally and then selectively removed later.
Comment on attachment 8644125 [details] [diff] [review]
b2g-desktop

Review of attachment 8644125 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, shipit! thanks for cleaning this up!
Attachment #8644125 - Flags: review?(catlee) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: