Skip to content
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

CI system should print out integration test failures #54843

Open
CyrusNajmabadi opened this issue Jul 15, 2021 · 3 comments
Open

CI system should print out integration test failures #54843

CyrusNajmabadi opened this issue Jul 15, 2021 · 3 comments
Assignees
Labels
Area-Infrastructure Contributor Pain The issue impedes progress for project collaborators.
Milestone

Comments

@CyrusNajmabadi
Copy link
Member

Right now all you see are things like:

Proc dump location: D:\workspace\_work\1\s\.tools\ProcDump\procdump.exe 
Running 2 test assemblies in 2 partitions 
 1 running, 1 queued, 0 completed 
Roslyn Error: test timeout exceeded, dumping remaining processes 
Dumping dotnet 5796 to D:\workspace\_work\1\s\artifacts\log\Debug\dotnet-1.dmp ... succeeded (100469002 bytes) 
Dumping dotnet 5744 to D:\workspace\_work\1\s\artifacts\log\Debug\dotnet-2.dmp ... succeeded (105067654 bytes) 
Dumping testhost.net472.x86 2520 to D:\workspace\_work\1\s\artifacts\log\Debug\testhost.net472.x86-3.dmp ... succeeded (325525651 bytes) 
Copying ServiceHub logs to D:\workspace\_work\1\s\artifacts\log\Debug 
##[error](NETCORE_ENGINEERING_TELEMETRY=Test) Tests failed 

This is cryptic and hard to diagnose. Can we instead print out the test log failure info? It looks like we have it somewhere, so it would just be about printing it to the console.

@CyrusNajmabadi CyrusNajmabadi added the Contributor Pain The issue impedes progress for project collaborators. label Jul 15, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Jul 15, 2021
@RikkiGibson
Copy link
Contributor

@sharwell
Copy link
Member

This type of failure is diagnosed by downloading the heap dump and finding the deadlock. Historically, test logs have not made the investigation any easier so the emphasis is on ensuring the heap dump is available.

@CyrusNajmabadi
Copy link
Member Author

I have run into this many times before and it's an utter PITA. Please provide just the test that was running. We can still keep providing the heap dumps if hte dev needs that.

@jinujoseph jinujoseph removed the untriaged Issues and PRs which have not yet been triaged by a lead label Jul 16, 2021
@jinujoseph jinujoseph added this to the 17.0 milestone Jul 16, 2021
@jinujoseph jinujoseph modified the milestones: 17.0, 17.1 Oct 21, 2021
@jinujoseph jinujoseph modified the milestones: 17.1, 17.3 Apr 27, 2022
@arkalyanms arkalyanms modified the milestones: 17.3, 17.6 P2 Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure Contributor Pain The issue impedes progress for project collaborators.
Projects
None yet
Development

No branches or pull requests

6 participants