Skip to content

Commit

Permalink
Force a restart of all the apache nodes during upgrade
Browse files Browse the repository at this point in the history
During the upgrades, the venv path will change and therefore the
apache configuration file will change too.

However we apply the restart of apache after the keystone_service_setup
for nodes [1:] (the first node gets restarted as first task of the
keystone_service_setup).

So during an upgrade, because apache is up, the configuration file has
changed but apache still serves the old code (because not restarted yet
on the nodes 1 and above) when the keystone_service_setup is applied.

The keystone module can then hit any node in the load balancer, which
could be a different version.

This commit fixes the issue by ensuring apache is restarted and
therefore runs the latest code.

Change-Id: Iac94a8fc337c2139d1876b9753e46815910a0ba0
Signed-off-by: Jean-Philippe Evrard <jean-philippe.evrard@rackspace.co.uk>
(cherry picked from commit 4f30d3a)
  • Loading branch information
Jean-Philippe Evrard authored and Jesse Pretorius (odyssey4me) committed Aug 18, 2016
1 parent 14d1be4 commit b9e799b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
6 changes: 0 additions & 6 deletions tasks/keystone_service_setup.yml
Expand Up @@ -13,12 +13,6 @@
# See the License for the specific language governing permissions and
# limitations under the License.

- name: Restart service
service:
name: "{{ keystone_system_service_name }}"
state: restarted
pattern: "{{ keystone_system_service_name }}"

- name: Wait for keystone admin to come up
wait_for:
host: "{{ ansible_ssh_host }}"
Expand Down
3 changes: 3 additions & 0 deletions tasks/main.yml
Expand Up @@ -87,6 +87,9 @@
- keystone-install
- keystone-config

- name: Flush handlers
meta: flush_handlers

- include: keystone_service_setup.yml
tags:
- keystone-install
Expand Down

0 comments on commit b9e799b

Please sign in to comment.