Use mozcrash.kill_and_get_minidump in xpcshell harness

RESOLVED DUPLICATE of bug 1316309

Status

RESOLVED DUPLICATE of bug 1316309
4 years ago
2 years ago

People

(Reporter: ted, Assigned: ted)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

Since bug 890026 finally landed, we should switch the xpcshell harness to use mozcrash.kill_and_get_minidump instead of Automation.killAndGetStackNoScreenshot (which would remove the harness' dependency on automation.py, which is nice).
Assignee: nobody → cmanchester
Ted mentioned taking this on IRC (if that's no longer true I'm happy to do it).
Assignee: cmanchester → ted
It's cool, I'll do it.
Created attachment 8647603 [details]
MozReview Request: bug 1193738 - use mozcrash.kill_and_get_minidump in xpcshell harness. r?chmanchester

bug 1193738 - use mozcrash.kill_and_get_minidump in xpcshell harness. r?chmanchester
Attachment #8647603 - Flags: review?(cmanchester)
Comment on attachment 8647603 [details]
MozReview Request: bug 1193738 - use mozcrash.kill_and_get_minidump in xpcshell harness. r?chmanchester

https://reviewboard.mozilla.org/r/16019/#review14339

::: testing/xpcshell/runxpcshelltests.py:760
(Diff revision 1)
>              if self.checkForCrashes(self.tempDir, self.symbolsPath, test_name=name):
>                  if self.retry:
>                      self.clean_temp_dirs(path)
>                      return

We get here now when a test times out but is eligible to retry, so we print the dumps and turn the job orange.

All this retry logic is already pretty convoluted, but a thing to do would be to not check for crashes if we're going to retry.
Attachment #8647603 - Flags: review?(cmanchester)
This is looking a lot greener. I'll try another push with a forced hang to sanity check that it works on Try, and I'll see if I can update our existing hang test to more fully test this case.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e1543315a655 is that push with a forced hang. It's *close*, in that we kill the process and get a stack, but we hit an error trying to kill the process again. I'll fix that up and get a new patch up.
Blocks: 1192307
ahal has a patch in bug 1316309
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1316309
No longer blocks: 1192307
You need to log in before you can comment on or make changes to this bug.