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

System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited hangs in CI #38943

Closed
ViktorHofer opened this issue Jul 8, 2020 · 9 comments
Labels
area-System.Diagnostics.Process blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' runtime-mono specific to the Mono runtime
Milestone

Comments

@ViktorHofer
Copy link
Member

ViktorHofer commented Jul 8, 2020

Configuration: net5.0-OSX-Release-x64-Mono_release-OSX.1014.Amd64.Open

https://helix.dot.net/api/2019-06-17/jobs/6fafb787-3aa8-4715-a1f4-2573e59b0da3/workitems/System.Diagnostics.Process.Tests/files/console.af56a1ee.log

https://dnceng.visualstudio.com/public/_build/results?buildId=720820&view=ms.vss-test-web.build-test-results-tab&runId=22326810&paneView=debug

  Discovering: System.Diagnostics.Process.Tests (method display = ClassAndMethod, method display options = None)
  Discovered:  System.Diagnostics.Process.Tests (found 220 of 290 test cases)
  Starting:    System.Diagnostics.Process.Tests (parallel test collections = on, max threads = 12)
    System.Diagnostics.Tests.ProcessStartInfoTests.ShellExecute_Nano_Fails_Start [SKIP]
      Condition(s) not met: "IsWindowsNanoServer"
Darwin
    System.Diagnostics.Tests.ProcessTests.TestProcessRecycledPid [SKIP]
      Condition(s) not met: "IsStressModeEnabledAndRemoteExecutorSupported"
    System.Diagnostics.Tests.ProcessTests.CanBeFinalized [SKIP]
      Condition(s) not met: "IsPreciseGcSupported"
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:02:10
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:04:11
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:06:12
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:08:13
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:10:14
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:12:15
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAfterExited', Elapsed: 00:14:15
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-System.Diagnostics.Process untriaged New issue has not been triaged by the area owner labels Jul 8, 2020
@ghost
Copy link

ghost commented Jul 8, 2020

Tagging subscribers to this area: @eiriktsarpalis
Notify danmosemsft if you want to be subscribed.

@safern safern added the blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' label Jul 14, 2020
@eiriktsarpalis eiriktsarpalis added this to the 5.0.0 milestone Aug 11, 2020
@eiriktsarpalis eiriktsarpalis added runtime-mono specific to the Mono runtime and removed untriaged New issue has not been triaged by the area owner labels Aug 11, 2020
@adamsitnik
Copy link
Member

I've sent a PR to disable this test for Mono to unblock the CI #40837 and moving this to 6.0

@jeffhandley @marek-safar how area owners should handle Mono-specific issues?

@adamsitnik adamsitnik modified the milestones: 5.0.0, 6.0.0 Aug 14, 2020
@jeffhandley
Copy link
Member

I've sent a PR to disable this test for Mono to unblock the CI #40837 and moving this to 6.0

@jeffhandley @marek-safar how area owners should handle Mono-specific issues?

Thanks, Adam! We'll want to put a little more time into this later to try to narrow it down to see if we can find the appropriate mono area that we would move it into. But since we've already punted this out to 6.0.0 and disabled the test for now, there's no urgency on it as far as I'm concerned.

/cc @lewing @steveisok

@marek-safar
Copy link
Contributor

/cc @lambdageek @CoffeeFlux

@CoffeeFlux
Copy link
Contributor

Interesting that this doesn't happen with SingleProcess_WaitAsyncAfterExited.

@jkotas
Copy link
Member

jkotas commented Sep 11, 2020

Failed in #42112 for SingleProcess_WaitAsyncAfterExited

        --- End of stack trace from previous location ---
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAsyncAfterExited', Elapsed: 00:02:05
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAsyncAfterExited', Elapsed: 00:04:05
   System.Diagnostics.Process.Tests: [Long Running Test] 'System.Diagnostics.Tests.ProcessWaitingTests.SingleProcess_WaitAsyncAfterExited', Elapsed: 00:06:06
...

@steveisok
Copy link
Member

Interesting that there are only 7 records in Kusto that have passed and taken longer than 3 seconds. And 28 records taking longer than 1 second.

Also interesting that there's a 2 min run for each iteration in both the original and what @jkotas just indicated.

@jaredpar
Copy link
Member

jaredpar commented Oct 6, 2020

@ViktorHofer no failures in last 14 days. Do u want to keep this open and make a live issue or close?

https://runfo.azurewebsites.net/search/tests/?bq=definition%3Aruntime+started%3A%7E14&tq=SingleProcess_WaitAfterExited

@ViktorHofer
Copy link
Member Author

14 days without a hit should be enough to justify closing the issue. I expect us to reopen when we hit it again in CI.

@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Diagnostics.Process blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' runtime-mono specific to the Mono runtime
Projects
None yet
Development

No branches or pull requests