Relax pre-upgrade check for failed actions
authorMichele Baldessari <michele@acksyn.org>
Wed, 28 Sep 2016 20:55:25 +0000 (22:55 +0200)
committerMichele Baldessari <michele@acksyn.org>
Thu, 29 Sep 2016 07:02:24 +0000 (09:02 +0200)
commit32c54304f489405ea2e3ab67f5de236ab6f2e5ec
treed187c2ccdb7c938d859ee214ce84606b11d20240
parent1b634c21c118a54071abd2bc5181453f3aa8d7a7
Relax pre-upgrade check for failed actions

Before this change we checked the cluster for any failed actions and
we stopped the upgrade process if there were any.
This is likely eccessive as a failed action could have happened in the
past and the cluster is now fully functional.

Better to check if any of the resources are in Stopped state and break
the upgrade process if any of them are.

We also need to restrict this check to the bootstrap node because
otherwise the following might happen:
1) Bootstrap node does the check, it is successful and it starts
   the full HA -> HA NG migration which *will* create failed actions
   and will start stopping resources
2) If the check now starts on a non-bootstrap node while 1) is ongoing,
   it will find either failed actions or stopped resources so it will
   fail.

Change-Id: Ib091f6dd8884025d2e23bf2fa700169e2dec778f
Closes-Bug: #1628653
extraconfig/tasks/major_upgrade_check.sh
extraconfig/tasks/major_upgrade_controller_pacemaker_1.sh