Closed Bug 887234 Opened 7 years ago Closed 7 years ago

Make leakstats not fail the build job on OS X if the log file was incomplete, until bug 774844 is fixed

Categories

(Testing :: General, defect)

x86_64
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Bug 774844 is causing additional infra load due to people unnecessarily retriggering OS X build jobs on Try - which then spawns another 3 platforms worth of tests.

Both to save the infra load and the sheriffs sanity (this is a fairly frequent intermittent failure), we should make the "log file incomplete" failure mode [1] a warning on OS X, rather than allowing it to fail the entire build job.

[1] http://mxr.mozilla.org/mozilla-central/source/tools/trace-malloc/leakstats.c#140
Attached patch Patch v1Splinter Review
For now turns the failure into a warning on OS X. We can't exit(0) immediately, since otherwise the lack of a summary will cause us to hit https://hg.mozilla.org/build/buildbotcustom/file/4731c6859855/steps/test.py#l231 which in turn means we'll fail at https://hg.mozilla.org/build/buildbotcustom/file/4731c6859855/steps/test.py#l190

We'll still need to decide whether we even find the leak stats useful and if not remove the buildsteps entirely.
Attachment #767719 - Flags: review?(dbaron)
Blocks: 774844
Fun fact: the buildbot function that looks for the tipmost revision which is safe to build a nightly on considers debug builds, so hitting this constant failure means that we fail to ship things in m-c nightlies for absolutely no reason.
Comment on attachment 767719 [details] [diff] [review]
Patch v1

Can we fix this at a higher layer?  Printing out totally bogus leak stats seems likely to be harmful in the long run.
Attachment #767719 - Flags: review?(dbaron) → review-
WONTFIXing in favour of bug 617441.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.