Skip to content
This repository has been archived by the owner on Dec 5, 2023. It is now read-only.

Support ABORTED state #44

Closed
jkeifer opened this issue Feb 16, 2022 · 0 comments · Fixed by #45
Closed

Support ABORTED state #44

jkeifer opened this issue Feb 16, 2022 · 0 comments · Fixed by #45
Assignees
@jkeifer
Milestone
0.7.0

Comments

@jkeifer
Copy link
Member

jkeifer commented Feb 16, 2022 •
edited
Loading

If a workflow is aborted (which we currently don't handle but want to) the closest state we have to that condition is FAILED. But it seems like we would want to differentiate between the two, and moreover FAILED requires an error message we don't have when aborting.

So let's add ABORTED as a state so we can support aborting workflows.

See cirrus-geo/cirrus-geo#85 for additional context.
@jkeifer jkeifer added this to the 0.7.0 milestone Feb 16, 2022
@jkeifer jkeifer self-assigned this Feb 16, 2022
jkeifer added a commit that referenced this issue Feb 16, 2022
@jkeifer
add aborted state closing #44
d7e80fc
@jkeifer jkeifer mentioned this issue Feb 16, 2022
Merged
jkeifer added a commit that referenced this issue Feb 16, 2022
@jkeifer
add aborted state closing #44
f7992cd
jkeifer added a commit that referenced this issue Feb 17, 2022
@jkeifer
add aborted state closing #44 (#45)
64f7cd1
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@jkeifer jkeifer

Labels
None yet
Projects
None yet
Milestone
0.7.0
Development

Successfully merging a pull request may close this issue.

Add ABORTED state cirrus-geo/cirrus-lib
1 participant
@jkeifer