#551 Fixing recovering of SKIPPED nodes Pull request opened by hamersaw TL;DR The PR to enable recovering dynamic workflows when they have already compiled the workflow closure updated the node executor to always use the phase returned from

attemptRecovery

. This surfaced an issue where nodes in the

SKIPPED

phase were reported as

SKIPPED

during recovery when they should be executed in the recovered workflow. This PR correctly reports the phase of previously

SKIPPED

nodes as

UNDEFINED

to ensure the nodes are correctly evaluated during workflow recoveries. Type ☑︎ Bug Fix ☐ Feature ☐ Plugin Are all requirements met? ☑︎ Code completed ☑︎ Smoke tested ☐ Unit tests added ☑︎ Code documentation added ☐ Any pending items have an associated Issue Complete description ^^^ Tracking Issue fixes flyteorg/flyte#3578 Follow-up issue NA flyteorg/flytepropeller GitHub Actions: Goreleaser GitHub Actions: Build & Push Flytepropeller Image GitHub Actions: Bump Version ✅ 11 other checks have passed 11/14 successful checks