Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from

Conversation

lostmsu
Copy link
Member

@lostmsu lostmsu commented Feb 21, 2020

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
Copy link

codecov-io commented Feb 22, 2020

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
Copy link
Member Author

lostmsu commented Feb 27, 2020

@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.

@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
Copy link
Member

filmor commented Feb 15, 2021

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