From f39a18dfd20b93e247b2fa2a14f2db492a904730 Mon Sep 17 00:00:00 2001
From: JoannaaKL <joannaakl@github.com>
Date: Tue, 27 Sep 2022 07:59:44 +0000
Subject: [PATCH] Remove unnecessary healthcheck for healthy service container

---
 src/Runner.Worker/ContainerOperationProvider.cs | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/Runner.Worker/ContainerOperationProvider.cs b/src/Runner.Worker/ContainerOperationProvider.cs
index d532f9d87..ab6cbcad5 100644
--- a/src/Runner.Worker/ContainerOperationProvider.cs
+++ b/src/Runner.Worker/ContainerOperationProvider.cs
@@ -330,11 +330,6 @@ namespace GitHub.Runner.Worker
             {
                 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}");
 
                         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.Output($"Stop and remove container: {container.ContainerDisplayName}");