Terminate a workflow dynamically

21 views
Skip to first unread message

Henrik Rusche

unread,
Dec 17, 2015, 9:09:56 AM12/17/15
to fireworkflows
Hi,

is it possible to terminate the exeution of the (complete) workflow by returning an FWAction?

I am aware of FWAction(defuse_children=True), but I am looking for something stronger that stops other branches as well.

Best Regards,

Henrik

Anubhav Jain

unread,
Dec 18, 2015, 12:17:43 PM12/18/15
to Henrik Rusche, fireworkflows
Hi Henrik

There is currently no action that defuses other branches of the workflow. It can be added in. One question - how do you want to handle already COMPLETED FireWorks? Should those remain in the COMPLETED state or do you want to revert those to DEFUSED?

Best,
Anubhav

--
You received this message because you are subscribed to the Google Groups "fireworkflows" group.
To unsubscribe from this group and stop receiving emails from it, send an email to fireworkflow...@googlegroups.com.
To post to this group, send email to firewo...@googlegroups.com.
Visit this group at https://groups.google.com/group/fireworkflows.
To view this discussion on the web visit https://groups.google.com/d/msgid/fireworkflows/cb935e42-55bd-4396-aca5-f4c9490de246%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Henrik Rusche

unread,
Dec 22, 2015, 11:12:05 AM12/22/15
to Anubhav Jain, fireworkflows
Hi Anubhav,

thanks for you swift reply.

I got that impression that there is currently no feature to do this, but I though that it's better to ask.

I am happy to help with this if you give an idea where to start.

In my case the status of the other FireWorks should be marked COMPLETED once they are finished.

Henrik

Anubhav Jain

unread,
Dec 22, 2015, 7:10:00 PM12/22/15
to Henrik Rusche, fireworkflows
Hi Henrik,

I have pushed some code in the latest Github master branch that allows for a "defuse_workflow" argument to FWAction. If set true, it will try to defuse any un-completed and un-fizzled jobs in the rest of the workflow.

It would be very helpful if you could:
i) test this functionality and/or
ii) add a simple unit test to make sure that it continues to work in the future

Best,
Anubhav
Reply all
Reply to author
Forward
0 new messages