Disable mailbloat on Mac Debug builds due to perma red (to enable comm-esr17 nightly builds to run)

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

5 years ago
All the other comm-esr17 branches have them, so we should have them to.
Weirdly, enable_nightly is already set to true. Dunno why we aren't getting them.
There's permared, hidden, mac debug builds. That's why wer'e not getting nightlies. https://tbpl.mozilla.org/?tree=Thunderbird-Esr17&noignore=1
(Assignee)

Comment 3

5 years ago
Created attachment 709117 [details] [diff] [review]
Disable leak tests on mac debug builds

We've had perma red on these builds for a while, plus they don't tend to give us stuff. However its just the leak test bit - the builds are still fine and the debug mozmill/xpcshell still run.

So lets turn off the leak test step. If at some time we get around to debugging them, then we can perhaps turn them back on again.
Attachment #709117 - Flags: review?(bhearsum)
Attachment #709117 - Flags: review?(bhearsum) → review+
(Assignee)

Updated

5 years ago
Summary: Enable nightly builds for comm-esr17 → Disable mailbloat on Mac Debug builds due to perma red (to enable comm-esr17 nightly builds to run)
(Assignee)

Comment 4

5 years ago
Comment on attachment 709117 [details] [diff] [review]
Disable leak tests on mac debug builds

http://hg.mozilla.org/build/buildbot-configs/rev/231b30d5adb3
Attachment #709117 - Flags: checked-in+
(Assignee)

Comment 5

5 years ago
Created attachment 709653 [details] [diff] [review]
Don't define leak_target globally

Looks like the previous patch didn't work, the leak tests are still run on Mac debug.

This should do it though - stop defining leak_target globally, and let the platforms decide if to define it or not.
Assignee: nobody → mbanner
Status: NEW → ASSIGNED
Attachment #709653 - Flags: review?(bhearsum)
(Assignee)

Updated

5 years ago
Attachment #709653 - Flags: review?(bhearsum) → review?(catlee)
Comment on attachment 709653 [details] [diff] [review]
Don't define leak_target globally

I think this will turn it off everywhere with the current implementation...

http://hg.mozilla.org/build/buildbotcustom/file/default/misc.py#l1110

should be changed to look at pf['leak_target'] instead of config['leak_target']. Then we can define this per-platform.
Attachment #709653 - Flags: review?(catlee) → review+
(Assignee)

Comment 7

5 years ago
Comment on attachment 709653 [details] [diff] [review]
Don't define leak_target globally

Thanks for the hint, I checked in both:

http://hg.mozilla.org/build/buildbotcustom/rev/f7e9b171d2c9
http://hg.mozilla.org/build/buildbot-configs/rev/ecd94170d162
Attachment #709653 - Flags: checked-in+
In production
(Assignee)

Comment 9

5 years ago
Created attachment 711739 [details] [diff] [review]
Really disable leak tests on Mac

I totally headed down the wrong route. leak_target is just overriding the action that happens to run the actual leak tests.

What I wanted to be doing was setting enable_leaktests to false on Mac.

This patch:
- Reverts the changes to leak_target
- Removes the duplicate platform definitions for leak_target when the global config will do.
- Turns off enable_leaktests on Mac.

When landing this, I'll also backout http://hg.mozilla.org/build/buildbotcustom/rev/f7e9b171d2c9 to revert the leak_target check to using the global config.
Attachment #711739 - Flags: review?(jhopkins)
Attachment #711739 - Flags: review?(catlee)
Attachment #711739 - Flags: review?(catlee) → review+
(Assignee)

Updated

5 years ago
Attachment #711739 - Flags: review?(jhopkins)
(Assignee)

Comment 12

5 years ago
This is live and working fine now.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
Blocks: 840958
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.