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

Remove unnecessary healthcheck for healthy service container

Browse Source
This commit is contained in:
JoannaaKL
2022-09-27 07:59:44 +00:00
parent fec24e8341
commit f39a18dfd2
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -330,11 +330,6 @@ namespace GitHub.Runner.Worker
{ {
if (!container.IsJobContainer && !container.FailedInitialization) if (!container.IsJobContainer && !container.FailedInitialization)
{ {
var healthcheck = await ContainerHealthcheck(executionContext, container);
if (string.Equals(healthcheck, "healthy", StringComparison.OrdinalIgnoreCase))
{
// Print logs for service container jobs (not the "action" job itself b/c that's already logged).
// Print them only if the service was healthy, else they were already logged in the initialize containers section.
executionContext.Output($"Print service container logs: {container.ContainerDisplayName}"); executionContext.Output($"Print service container logs: {container.ContainerDisplayName}");
int logsExitCode = await _dockerManager.DockerLogs(executionContext, container.ContainerId); int logsExitCode = await _dockerManager.DockerLogs(executionContext, container.ContainerId);
@@ -342,7 +337,6 @@ namespace GitHub.Runner.Worker
{ {
executionContext.Warning($"Docker logs fail with exit code {logsExitCode}"); executionContext.Warning($"Docker logs fail with exit code {logsExitCode}");
} }
}
} }
executionContext.Output($"Stop and remove container: {container.ContainerDisplayName}"); executionContext.Output($"Stop and remove container: {container.ContainerDisplayName}");