Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs doesn't tell user differences between workflow Stop and Terminate behaviors #4454

Closed
bklaubos opened this issue Nov 3, 2020 · 6 comments
Labels

Comments

@bklaubos
Copy link

bklaubos commented Nov 3, 2020

Summary

I expect CLI docs to tell end user differences between argo terminate and argo stop intentions and its expected behavior. I found none.

Diagnostics

What Kubernetes provider are you using?
AWS
What version of Argo Workflows are you running?
2.16

Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@simster7
Copy link
Member

simster7 commented Nov 3, 2020

Feel free to submit a PR

@bklaubos
Copy link
Author

bklaubos commented Nov 4, 2020

I logged this bug because as end user, I don't know the difference between stop vs terminate workflow. Are they the same?. If not, what are the expected behaviours.

@alexec
Copy link
Contributor

alexec commented Nov 4, 2020

They seem to be related to "shutdown strategy".

  • Terminate do not run any more node nodes in the workflow.
  • Stop do not run any more nodes - except exit handlers.

I'm not clear what happens to running nodes.

@simster7
Copy link
Member

simster7 commented Nov 4, 2020

I'm not clear what happens to running nodes.

In both cases running nodes are stopped.

Terminate stops the workflow immediately, while Stop stops the workflow but allows all exit handlers to run.

@knabben
Copy link
Contributor

knabben commented Nov 5, 2020

What about adding these notes to the Cobra helper? I can help with this ticket.

https://github.com/argoproj/argo/blob/master/cmd/argo/commands/terminate.go#L16
https://github.com/argoproj/argo/blob/master/cmd/argo/commands/stop.go#L27

@simster7
Copy link
Member

simster7 commented Nov 5, 2020

Sure, that would be great. I would also find a place in the docs to add the same information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants