Closed Bug 1152994 Opened 9 years ago Closed 9 years ago

Improve mochitest orphan process checks

Categories

(Testing :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(firefox40 affected)

RESOLVED FIXED
Tracking Status
firefox40 --- affected

People

(Reporter: gbrown, Assigned: gbrown)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In bug 1084614, I tried to resolve the problem of ssltunnel and xpcshell processes left over from previous test runs causing infrequent, intermittent test failures. The strategy was to kill orphan ssltunnel and xpcshell processes and log all such running processes found at the start of a test run.

That patch has helped, significantly on Android.

But bug 915212 (and maybe bug 919920) shows that there are additional cases to be considered.

1. B2G emulator mochitests do not use the code introduced in bug 1084614.

2. On Ubuntu VM 12.04 (only?) we still sometimes find ssltunnel and xpcshell running at the start of the test, started recently (30 to 60 minutes earlier, typically), but not orphaned. Presumably the parent is runtests.py, but we don't know for sure. We do not kill non-orphaned processes to account for Android foopies, where we expect multiple ssltunnel and xpcshell instances to be running concurrently.
Simple patch to handle the B2G case.
Attachment #8590823 - Flags: review?(kmoir)
I haven't figured out what to do about case #2 yet.
Keywords: leave-open
Attachment #8590823 - Flags: review?(kmoir) → review+
I wanted to keep this open to consider adding more diagnostics. With recent progress in bug 915212, I think we probably have enough information and understanding of root cause (for example, if the network connection is lost, buildbot will abandon the test and recycle the instance, but there may not be a reboot, so the old test may still be running when a new one is starting).
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: