Merge pull request #103 from damianh/add-warning-to-process-kill

damianh · web-flow · commit cfdba1ca3c8a · 2020-09-25T09:55:26.000+02:00
Log a warning of the timeout before killing the process
diff --git a/src/LittleForker/ProcessSupervisor.cs b/src/LittleForker/ProcessSupervisor.cs
@@ -327,6 +327,10 @@ await Task
                     // time.
                     try
                     {
+                        _logger.LogWarning(
+                            $"Timed out waiting to signal the process to exit or the " +
+                            $"process {_process.ProcessName} ({_process.Id}) did not shutdown in " +
+                            $"the given time ({timeout})");
                         _process.Kill();
                     }
                     catch (Exception ex)

Original file line number	Diff line number	Diff line change
`@@ -327,6 +327,10 @@ await Task`
`327`	`327`	`// time.`
`328`	`328`	`try`
`329`	`329`	`{`
	`330`	`+ _logger.LogWarning(`
	`331`	`+ $"Timed out waiting to signal the process to exit or the " +`
	`332`	`+ $"process {_process.ProcessName} ({_process.Id}) did not shutdown in " +`
	`333`	`+ $"the given time ({timeout})");`
`330`	`334`	`_process.Kill();`
`331`	`335`	`}`
`332`	`336`	`catch (Exception ex)`