diff --git a/src/Runner.Worker/ContainerOperationProvider.cs b/src/Runner.Worker/ContainerOperationProvider.cs index 79d2190a4..45017133c 100644 --- a/src/Runner.Worker/ContainerOperationProvider.cs +++ b/src/Runner.Worker/ContainerOperationProvider.cs @@ -126,7 +126,9 @@ namespace GitHub.Runner.Worker foreach (var container in unhealthyContainers) { executionContext.Output($"##[group]Service container {container.ContainerNetworkAlias} failed."); - await ContainerErrorLogs(executionContext, container); + await _dockerManager.DockerLogs(context: executionContext, containerId: container.ContainerId); + executionContext.Error($"Failed to initialize container {container.ContainerImage}"); + container.FailedInitialization = true; executionContext.Output("##[endgroup]"); } throw new InvalidOperationException("One or more containers failed to start."); @@ -448,13 +450,6 @@ namespace GitHub.Runner.Worker return serviceHealth; } - private async Task ContainerErrorLogs(IExecutionContext executionContext, ContainerInfo container) - { - await _dockerManager.DockerLogs(context: executionContext, containerId: container.ContainerId); - executionContext.Error($"Failed to initialize container {container.ContainerImage}"); - container.FailedInitialization = true; - } - private async Task ContainerRegistryLogin(IExecutionContext executionContext, ContainerInfo container) { if (string.IsNullOrEmpty(container.RegistryAuthUsername) || string.IsNullOrEmpty(container.RegistryAuthPassword))