When one or more of the Actions included in an Action Plan finishes with FAILED status, the Action Plan still reports SUCCEEDED status.

According to the documentation https://docs.openstack.org/watcher/latest/architecture.html#state-machine-diagrams SUCCEDED means that "the Action Plan has been executed successfully (i.e. all Actions that it contains have been executed successfully)" so in those cases de Action Plan should be reported as FAILED.

To Reproduce Steps to reproduce the behavior:

1. Create an audit with a parameter that forces action failure, i.e. unexisting compute node as backup host in the host_maintenance strategy.
2. Run de audit
3. Start the action plan
4. The action fails
5. The action plan is marked as SUCCEEDED

Expected behavior

• The action plan should be marked as FAILED

Bug impact

• User will not see the real action plan result when an action fails.

Known workaround

• User can check the status of each action in an action plan manually using openstack cli or horizon watcher dashboard.