From f1ca4b6114036f437859695cbfbd29cd7ae0ef75 Mon Sep 17 00:00:00 2001 From: Daniel Rammer Date: Fri, 7 Apr 2023 16:23:52 -0500 Subject: [PATCH] returning 'Undefined' when recovering a SKIPPED node so that the node is executed Signed-off-by: Daniel Rammer --- pkg/controller/nodes/executor.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/controller/nodes/executor.go b/pkg/controller/nodes/executor.go index 3d0707469..c447b779c 100644 --- a/pkg/controller/nodes/executor.go +++ b/pkg/controller/nodes/executor.go @@ -216,7 +216,7 @@ func (c *nodeExecutor) attemptRecovery(ctx context.Context, nCtx handler.NodeExe // A recoverable node execution should always be in a terminal phase switch recovered.Closure.Phase { case core.NodeExecution_SKIPPED: - return handler.PhaseInfoSkip(nil, "node execution recovery indicated original node was skipped"), nil + return handler.PhaseInfoUndefined, nil case core.NodeExecution_SUCCEEDED: fallthrough case core.NodeExecution_RECOVERED: