Closed Bug 657631 Opened 13 years ago Closed 13 years ago

code coverage builds timeout in "make check" and xpcshell-tests

Categories

(Release Engineering :: General, defect, P2)

x86
Linux
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ted, Assigned: ted)

References

()

Details

(Whiteboard: [coverage][unittest])

Attachments

(1 file)

From the log: ======== BuildStep started ======== 'bash -c ...' failed === Output === bash -c 'make -k check 2>&1 | bzip2 > ../logs/check.log.bz2' in dir /builds/slave/cen-lnx-codecoverage/build/objdir (timeout 600 secs) <...> command timed out: 600 seconds without output, attempting to kill process killed by signal 9 ======== BuildStep started ======== 'bash -c ...' failed === Output === bash -c 'make xpcshell-tests 2>&1 | bzip2 > ../logs/xpcshell.log.bz2' <...> command timed out: 3600 seconds without output, attempting to kill process killed by signal 9 The codecoverage steps pipe their output to a log file, so these are probably being killed for producing no output: http://hg.mozilla.org/build/buildbotcustom/file/4eaf32eb9bfd/process/factory.py#l5792 We could swap that to be: | tee >(bzip2 -c > log.bz2) so that output would go to stdout as well, and things might work again. Longer-term, it might be nice to get these builds using the standard build steps, since we now upload compressed logs for build steps anyway.
Blocks: 449532
I haven't tested this, but it seems pretty straightforward.
Attachment #533015 - Flags: review?(catlee)
Assignee: nobody → ted.mielczarek
Status: NEW → ASSIGNED
I filed bug 657703 on simplifying the codecoverage setup.
A downside of using tee is that the tinderbox log is going to be a single massive log.
Attachment #533015 - Flags: review?(catlee) → review+
Priority: -- → P2
Whiteboard: [coverage][unittest]
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: