argo-workflows icon indicating copy to clipboard operation
argo-workflows copied to clipboard

Unique phases for suspended and stopped

Open Colstuwjx opened this issue 3 years ago • 2 comments

Summary

What change needs making?

For the workflow phase and node phase, we didn't implement a full workflow logic, e.g.

  1. if we resume or stop a completed workflow, it should panic since workflow state machine wouldn't allow transform state from ``, see issue #8173
  2. if we suspend a workflow, it will set suspend=true in the workflow status, but the phase is still be Running, I think it doesn't make sense, how about design a set of workflow primitives and use a single field to show the workflow state?

Use Cases

When would you use this?

When we're planning to design a workflow system based on argo-workflow, we need to read the workflow state and take actions to run the workflow, therefore, we need to improve our workflow design. For now, it's too fragmented and not friendly to user.


Message from the maintainers:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Colstuwjx avatar Mar 18 '22 04:03 Colstuwjx

Is this a duplicate?

alexec avatar Mar 18 '22 16:03 alexec

I don't think it's duplicated to #8173 , instead, I want to confirm if we plan to merge these workflow state into a single phase or not.

From a point view of user, I can't understand that I need to check both the workflow phase and the suspend flag to confirm whether the workflow is running or not. If I suspended a workflow, it should turn to SUSPENDING or PAUSING, rather than RUNNING with another flag suspend: true.

Colstuwjx avatar Mar 19 '22 00:03 Colstuwjx