DUNE-DAQ / nanorc

2 stars 2 forks source link

check FSM error first, THEN for error/force flag #163

Closed plasorak closed 1 year ago

plasorak commented 1 year ago

This fixes the problem that happens when the top node is in error, and shutdown --force is issued, it will try to drain-dataflow first, which shouldn't happen. The fix in this PR is that sequential command function firstly checks that the command it's trying to execute is valid under the rules of the FSM (irrespective of its error state), and AFTER THAT, checks if the node is in error state and force execute the command if needed.