-
Notifications
You must be signed in to change notification settings - Fork 32
Add a uw rocoto iterate action #766
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
Add a uw rocoto iterate action #766
Conversation
Co-authored-by: Emily Carpenter <137525341+elcarpenterNOAA@users.noreply.github.com>
… into uw-629-rocoto-monitor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great. It looks really clean and I appreciate that it works directly with the database to find the status, resorting to a full rocotostat
only when reporting to the user.
My only slight concern is the name of the mode. I'm curious if you'd consider iterate
instead of run
since the latter is a bit convoluted by the convention that rocotorun
only does a single iteration of the full workflow.
@christinaholtNOAA Good idea, I'll change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for that tedious change!
Synopsis
Add a
iterate
action to moderocoto
to iterate a Rocoto workflow to completion (successful or unsuccessful).Fixes #629.
Type
Impact
Checklist