Closed Bug 977420 Opened 8 years ago Closed 8 years ago

Change birch config and add jacuzzis

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: glandium, Assigned: bhearsum)

Details

Attachments

(5 files)

Please reconfigure birch to only do linux and linux64 opt and debug (no asan, no valgrind, no windows, no mac, etc.). Please keep the periodic PGO and non-unified builds, but please remove the nightlies. Please also place the linux/linux64 opt/debug builds in a jacuzzi (but not the PGO an non-unified builds)
I just but the builders you requested into jacuzzis: https://github.com/bhearsum/static-jacuzzis/commit/d2881408055bb916eb8a79adcad2f75fad96f02e

I'll get to the config adjustments shortly.
Assignee: nobody → bhearsum
This patch removes all builders except:
Linux birch build
Linux birch pgo-build
Linux birch nightly
Firefox birch linux l10n nightly
Linux birch leak test build
Linux x86-64 birch build
Linux x86-64 birch pgo-build
Linux x86-64 birch non-unified
Linux x86-64 birch nightly
Firefox birch linux64 l10n nightly
Linux x86-64 birch leak test build
Linux x86-64 birch leak test non-unified
b2g_birch_emulator_dep
b2g_birch_emulator-debug_dep
b2g_birch_emulator-jb_dep
b2g_birch_emulator-jb-debug_dep
b2g_birch_linux32_gecko build
b2g_birch_linux64_gecko build
b2g_birch_macosx64_gecko build
b2g_birch_win32_gecko build


Your last sentence made me unsure whether you still wanted non-unified/PGO, and I wasn't sure about b2g either. If you don't need those I'll remove them in a second patch.
Attachment #8383057 - Flags: review?
Attachment #8383057 - Flags: review? → review+
Attachment #8383057 - Flags: checked-in+
Patch is in production. Mike, can you confirm whether or not the builder list from comment #2 is what you wanted?
Flags: needinfo?(mh+mozilla)
You can remove nightlies and b2g. The last sentence meant that i don't want PGO and non-unified builds in a jacuzzi, but i do want the others in one. Thanks.
Flags: needinfo?(mh+mozilla)
Attachment #8383275 - Flags: review?(rail)
Latest patch leaves us with:
Linux birch build NightlyBuildFactory
Linux birch pgo-build NightlyBuildFactory
Linux birch leak test build NightlyBuildFactory
Linux x86-64 birch build NightlyBuildFactory
Linux x86-64 birch pgo-build NightlyBuildFactory
Linux x86-64 birch non-unified NightlyBuildFactory
Linux x86-64 birch leak test build NightlyBuildFactory
Linux x86-64 birch leak test non-unified NightlyBuildFactory
Attachment #8383275 - Flags: review?(rail) → review+
Attachment #8383275 - Flags: checked-in+
I'm still getting b2g builds. I guess there hasn't been a reconfig yet?
Ah, I forgot to ask to remove tests jobs too. I only need builds at the moment, and don't want to waste test slave time on it.
(In reply to Mike Hommey [:glandium] from comment #8)
> Ah, I forgot to ask to remove tests jobs too. I only need builds at the
> moment, and don't want to waste test slave time on it.

Yeah, no reconfig for the second patch yet. I'll remove the tests too.
Attachment #8383637 - Flags: review?(rail)
Attachment #8383637 - Flags: review?(rail) → review+
Attachment #8383637 - Flags: checked-in+
buildbot-configs patch is merged and live in production! :)
I think we're all done here now!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
I have one last request, and I figured I'd just reuse this bug instead of filing a new one: could you disable coalescing on birch? Thanks.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Looks like there's extra stuff that would be needed if tests were on, but AFAICT this is all we need for builds.
Attachment #8385300 - Flags: review?(rail)
Attachment #8385300 - Flags: review?(rail) → review+
Attachment #8385300 - Flags: checked-in+
landed in production :)
(In reply to Jordan Lund (:jlund) from comment #15)
> landed in production :)

yay, we're all done here then :)
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
cleans up birch removing duplicate enable_nightly key (also removes snippet keys)
Attachment #8399009 - Flags: review?(bhearsum)
Attachment #8399009 - Flags: review?(bhearsum) → review+
Comment on attachment 8399009 [details] [diff] [review]
977420_remove_birch_nightly_keys-140329.patch

-> defualt: https://hg.mozilla.org/build/buildbot-configs/rev/70e201c6330f
Attachment #8399009 - Flags: checked-in+
in production.
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.