Add retry loop around Process.Kill(entireProcessTree: true)

Workaround dotnet#93321
jkotas · Jun 1, 2024 · a7993bb · a7993bb
1 parent 9db5486
commit a7993bb
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/src/tests/Common/Coreclr.TestWrapper/CoreclrTestWrapperLib.cs b/src/tests/Common/Coreclr.TestWrapper/CoreclrTestWrapperLib.cs
@@ -305,7 +305,18 @@ static bool CollectCrashDumpWithCreateDump(Process process, string crashDumpPath
             }
             else
             {
-                createdump.Kill(true);
+                // Workaround for https://github.com/dotnet/runtime/issues/93321
+                for (int i = 0; i < 5; i++)
+                {
+                    try
+                    {
+                        createdump.Kill(entireProcessTree: true);
+                    }
+                    catch (Exception e)
+                    {
+                        Console.WriteLine($"Process.Kill(entireProcessTree: true) failed with {e}. Retrying.");
+                    }
+                }
             }
 
             return fSuccess && createdump.ExitCode == 0;