Skip to content

Commit

Permalink
Merged PR 693279: Increase ProcessDumper default depth to 20
Browse files Browse the repository at this point in the history
This is a short term mitigation to allow some builds with timeouts to be investigated where the process tree depth is greater than 20.

Related work items: #2015851
  • Loading branch information
mpysson committed Dec 13, 2022
1 parent 8ec2777 commit 087359a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Public/Src/Engine/Processes/ProcessDumper.cs
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ public static bool TryDumpProcess(IntPtr processHandle, int processId, string du
/// <summary>
/// Attempts to dump all processes in a process tree. Any files existing in the dump directory will be deleted.
/// </summary>
public static bool TryDumpProcessAndChildren(int parentProcessId, string dumpDirectory, out Exception primaryDumpCreationException, int maxTreeDepth = 10, Action<string> debugLogger = null)
public static bool TryDumpProcessAndChildren(int parentProcessId, string dumpDirectory, out Exception primaryDumpCreationException, int maxTreeDepth = 20, Action<string> debugLogger = null)
{
if (OperatingSystemHelper.IsMacOS)
{
Expand Down

0 comments on commit 087359a

Please sign in to comment.