search

lanl / BEE

Other
13 stars 3 forks source link

Allow removal of a workflow with any Archive status #828

Closed pagrubel closed 1 month ago

pagrubel commented 2 months ago

When attempting to remove a workflow with the new Archive/Failed status, it was not allowed because beeflow wasn't aware of this status and thought the workflow might be running. This PR now allows any workflow with a status containing Archive to be removed.

rstyd commented 1 month ago

This looks good to me! Approved.

pagrubel commented 1 month ago

I'd like to understand this a bit better in light of the work I'm doing concerning slurm, so putting the WIP label back on so we don't merge it yet.

rstyd commented 1 month ago

Gotcha thanks!

pagrubel commented 1 month ago

Sorry this was not the PR I wanted to hold off on. So I removed the WIP lablel

rstyd commented 1 month ago

Oh okay in that case I'll merge it.