From 746c9d9ec0583a213c9ec3e3dda0ebb94f91aed5 Mon Sep 17 00:00:00 2001 From: David Kale Date: Tue, 21 Jan 2020 11:23:30 -0500 Subject: [PATCH] Trace javascript action exit code instead of user logs (#290) * Trace javascript action exit code instead of user logs * Debug instead of trace --- src/Runner.Worker/Handlers/ContainerActionHandler.cs | 2 +- src/Runner.Worker/Handlers/NodeScriptActionHandler.cs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Runner.Worker/Handlers/ContainerActionHandler.cs b/src/Runner.Worker/Handlers/ContainerActionHandler.cs index bcd539469..a623da968 100644 --- a/src/Runner.Worker/Handlers/ContainerActionHandler.cs +++ b/src/Runner.Worker/Handlers/ContainerActionHandler.cs @@ -193,9 +193,9 @@ namespace GitHub.Runner.Worker.Handlers using (var stderrManager = new OutputManager(ExecutionContext, ActionCommandManager, container)) { var runExitCode = await dockerManger.DockerRun(ExecutionContext, container, stdoutManager.OnDataReceived, stderrManager.OnDataReceived); + ExecutionContext.Debug($"Docker Action run completed with exit code {runExitCode}"); if (runExitCode != 0) { - ExecutionContext.Error($"Docker run failed with exit code {runExitCode}"); ExecutionContext.Result = TaskResult.Failed; } } diff --git a/src/Runner.Worker/Handlers/NodeScriptActionHandler.cs b/src/Runner.Worker/Handlers/NodeScriptActionHandler.cs index 911265e3c..fb3b15448 100644 --- a/src/Runner.Worker/Handlers/NodeScriptActionHandler.cs +++ b/src/Runner.Worker/Handlers/NodeScriptActionHandler.cs @@ -122,9 +122,9 @@ namespace GitHub.Runner.Worker.Handlers else { var exitCode = await step; + ExecutionContext.Debug($"Node Action run completed with exit code {exitCode}"); if (exitCode != 0) { - ExecutionContext.Error($"Node run failed with exit code {exitCode}"); ExecutionContext.Result = TaskResult.Failed; } }