Change birch config and add jacuzzis

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
a year ago

People

(Reporter: glandium, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(5 attachments)

(Reporter)

Description

5 years ago
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)
(Assignee)

Comment 1

5 years ago
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
(Assignee)

Comment 2

5 years ago
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+
(Assignee)

Updated

5 years ago
Attachment #8383057 - Flags: checked-in+
(Assignee)

Comment 3

5 years ago
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)
(Reporter)

Comment 4

5 years ago
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)
(Assignee)

Comment 5

5 years ago
Attachment #8383275 - Flags: review?(rail)
(Assignee)

Comment 6

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

Updated

5 years ago
Attachment #8383275 - Flags: review?(rail) → review+
(Assignee)

Updated

5 years ago
Attachment #8383275 - Flags: checked-in+
(Reporter)

Comment 7

5 years ago
I'm still getting b2g builds. I guess there hasn't been a reconfig yet?
(Reporter)

Comment 8

5 years ago
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.
(Assignee)

Comment 9

5 years ago
(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.
(Assignee)

Comment 10

5 years ago
Attachment #8383637 - Flags: review?(rail)

Updated

5 years ago
Attachment #8383637 - Flags: review?(rail) → review+
(Assignee)

Updated

5 years ago
Attachment #8383637 - Flags: checked-in+
buildbot-configs patch is merged and live in production! :)
(Assignee)

Comment 12

5 years ago
I think we're all done here now!
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 13

5 years ago
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 → ---
(Assignee)

Comment 14

5 years ago
Looks like there's extra stuff that would be needed if tests were on, but AFAICT this is all we need for builds.
(Assignee)

Updated

5 years ago
Attachment #8385300 - Flags: review?(rail)

Updated

5 years ago
Attachment #8385300 - Flags: review?(rail) → review+
(Assignee)

Updated

5 years ago
Attachment #8385300 - Flags: checked-in+
landed in production :)
(Assignee)

Comment 16

5 years ago
(In reply to Jordan Lund (:jlund) from comment #15)
> landed in production :)

yay, we're all done here then :)
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
cleans up birch removing duplicate enable_nightly key (also removes snippet keys)
Attachment #8399009 - Flags: review?(bhearsum)
(Assignee)

Updated

5 years ago
Attachment #8399009 - Flags: review?(bhearsum) → review+
in production.
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.