actions/runner
1
0
Fork 0
mirror of https://github.com/actions/runner.git synced 2025-12-11 21:06:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added execution context error

Browse Source 
This will make a failed health check more visible in the UI without disrupting the execution of the program.
This commit is contained in:
AStancu
2022-09-06 21:53:37 +00:00
committed by Ava S
parent 94e504c40e
commit cfcf0831f3
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/Runner.Worker/ContainerOperationProvider.cs
View File

@@ -429,8 +429,9 @@ namespace GitHub.Runner.Worker
} }
else else
{ {
executionContext.Output($"##[group] Container {container.ContainerImage} failed healthchecks, printing logs:"); executionContext.Output($"##[group]Container {container.ContainerImage} failed healthchecks, printing logs:");
await _dockerManager.DockerLogs(context: executionContext, containerId: container.ContainerId); await _dockerManager.DockerLogs(context: executionContext, containerId: container.ContainerId);
executionContext.Error($"Failed to initialize container {container.ContainerImage}");
executionContext.Output("##[endgroup]"); executionContext.Output("##[endgroup]");
} }
} }

2
src/Test/L0/Worker/ContainerOperationProviderL0.cs
View File

@@ -64,7 +64,7 @@ namespace GitHub.Runner.Common.Tests.Worker
//Assert //Assert
_ec.Verify(dm => dm.Write(It.IsAny<string>(), It.IsAny<string>()), Times.Exactly(2)); _ec.Verify(dm => dm.Write(It.IsAny<string>(), It.IsAny<string>()), Times.Exactly(2));
_ec.Verify(dm => dm.Write(null, $"##[group] Container {containerInfo.ContainerImage} failed healthchecks, printing logs:"), Times.AtLeastOnce()); _ec.Verify(dm => dm.Write(null, $"##[group]Container {containerInfo.ContainerImage} failed healthchecks, printing logs:"), Times.AtLeastOnce());
_ec.Verify(dm => dm.Write(null, "##[endgroup]"), Times.AtLeastOnce()); _ec.Verify(dm => dm.Write(null, "##[endgroup]"), Times.AtLeastOnce());
} }