Skip to content

Conversation

maddenp-cu
Copy link
Collaborator

@maddenp-cu maddenp-cu commented Jul 21, 2025

Synopsis

Add a iterate action to mode rocoto to iterate a Rocoto workflow to completion (successful or unsuccessful).

Fixes #629.

Type

  • Documentation
  • Enhancement (adds new functionality)

Impact

  • This is a non-breaking change (existing functionality continues to work as expected)

Checklist

  • I have added myself and any co-authors to the PR's Assignees list.
  • I have reviewed the documentation and have made any updates necessitated by this change.

maddenp-cu and others added 3 commits July 23, 2025 11:04
Co-authored-by: Emily Carpenter <137525341+elcarpenterNOAA@users.noreply.github.com>
Copy link
Collaborator

@christinaholtNOAA christinaholtNOAA left a 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.

@maddenp-cu
Copy link
Collaborator Author

@christinaholtNOAA Good idea, I'll change run to iterate.

@maddenp-cu maddenp-cu changed the title Add a uw rocoto run action Add a uw rocoto iterate action Jul 24, 2025
Copy link
Collaborator

@christinaholtNOAA christinaholtNOAA left a 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!

@maddenp-cu maddenp-cu merged commit 72a45fe into ufs-community:main Jul 24, 2025
2 checks passed
@maddenp-cu maddenp-cu deleted the uw-629-rocoto-monitor branch July 24, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider adding a Rocoto action to monitor jobs.
3 participants