Skip to content

If tests are taking too long, create a memory dump and abort - #1052

Closed
lostmsu wants to merge 7 commits into
pythonnet:masterfrom
losttech:PR/TestsDeadlockDump
Closed

lostmsu wants to merge 7 commits into
pythonnet:masterfrom
losttech:PR/TestsDeadlockDump

Conversation

@lostmsu

@lostmsu lostmsu commented Feb 21, 2020

Copy link
Copy Markdown
Member

This change is to investigate AppVeyor tests failing with timeout #1050

@lostmsu
lostmsu force-pushed the PR/TestsDeadlockDump branch from 9bee1bd to df4b2b7 Compare February 21, 2020 20:55
@codecov-io

codecov-io commented Feb 22, 2020 •

Copy link
Copy Markdown

Codecov Report

Merging #1052 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1052   +/-   ##
=======================================
  Coverage   86.75%   86.75%           
=======================================
  Files           1        1           
  Lines         302      302           
=======================================
  Hits          262      262           
  Misses         40       40
Flag Coverage Δ
#setup_linux 65.56% <ø> (ø) ⬆️
#setup_windows 71.52% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e1d29d...938a59a. Read the comment docs.

@lostmsu
lostmsu force-pushed the PR/TestsDeadlockDump branch from df4b2b7 to 8114af0 Compare February 27, 2020 07:37
@lostmsu

lostmsu commented Feb 27, 2020

Copy link
Copy Markdown
Member Author

@filmor this is still useful, as I experience a similar issue with tests hanging just after starting DomainReloadAndGC: https://ci.appveyor.com/project/pythonnet/pythonnet/builds/31086476/job/aigfnc5gv4ni8stk , which I can not reproduce locally.

Comment thread src/embed_tests/TestDomainReload.cs Resolved
@lostmsu lostmsu added this to the 2.4.1 milestone Feb 27, 2020
@lostmsu
lostmsu requested a review from filmor March 2, 2020 21:32
@lostmsu lostmsu removed this from the 2.4.1 milestone Apr 23, 2020
@filmor

filmor commented Feb 15, 2021

Copy link
Copy Markdown
Member

Since we haven't seen this in quite a while, I'm going to close this PR.

@filmor filmor closed this Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants