diff options
author | Michele Baldessari <michele@acksyn.org> | 2017-03-17 15:36:50 +0100 |
---|---|---|
committer | Michele Baldessari <michele@acksyn.org> | 2017-03-18 22:38:25 +0100 |
commit | 557b0214162ac60b65db93deb4af71dee14a28dd (patch) | |
tree | dcef7f4c467608a509d85900a2a951be4eb33fb8 /.gitreview | |
parent | bc8dcd1054f95620d69c4595eebb9157df9c7e7b (diff) |
Make sure PrePuppet runs before any Deployment_Step
We used to have this in mitaka:
https://github.com/openstack/tripleo-heat-templates/blob/stable/mitaka/puppet/controller-post.yaml#L45
but we lost it along the way. The problem without this change is that we
are open to the following race:
1) ControllerDeployment_Step1 is started and manages to do a successful
"systemctl start pacemaker"
2) PrePuppet gets called and in the HA deployment calls
pacemaker_maintenance_mode.sh
3) pacemaker_maintenance_mode.sh will set the maintenance-mode=true
property because the pacemaker service is already up:
https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/tasks/pacemaker_maintenance_mode.sh#L8-L9
4) If the maintenance property is set to true at this stage, the
creation of any resource will take place but they won't really
start.
Note that this is not a straight cherry pick from commit
bae48e60b3cb9b5f21490997ca39c1e0e23fd195 because in ocata only
ControllerPrePuppet exists and not {{role.name}}PrePuppet like in
pike.
Change-Id: Icb7495edd00385b2975dd42f63085d20292ef9a9
Closes-Bug: #1673795
Co-Authored-By: Jiri Stransky <jstransk@redhat.com>
Diffstat (limited to '.gitreview')
0 files changed, 0 insertions, 0 deletions