Use mozcrash.kill_and_get_minidump in xpcshell harness

RESOLVED DUPLICATE of bug 1316309

Status

Testing
XPCShell Harness
RESOLVED DUPLICATE of bug 1316309
3 years ago
a year ago

People

(Reporter: ted, Assigned: ted)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
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
(Assignee)

Comment 2

3 years ago
It's cool, I'll do it.
(Assignee)

Comment 4

3 years ago
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)
(Assignee)

Comment 7

3 years ago
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.
(Assignee)

Comment 8

3 years ago
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.
(Assignee)

Updated

3 years ago
Blocks: 1192307
(Assignee)

Comment 9

a year ago
ahal has a patch in bug 1316309
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1316309
(Assignee)

Updated

a year ago
No longer blocks: 1192307
You need to log in before you can comment on or make changes to this bug.