From 2c187204874670c3ba41b83881efdc5c9c62c209 Mon Sep 17 00:00:00 2001
From: Matt Thompson <mattt@defunct.ca>
Date: Thu, 28 Jan 2016 09:28:07 +0000
Subject: [PATCH] Restart horizon when package/venvs change

Currently, upgrading a horizon package/venv does not explicitly trigger
a restart.  A restart will only occur if an explicit or implicit config
change occurs.

This commit adds several restart notifiers such that a horizon package
upgrade or venv replacement will force a restart.

Note that this issue exists in all OpenStack roles at the moment, but
the remaining roles are being addressed in their re-deploy venv
reviews.  The horizon re-deploy venv change was merged a while back
which is why we need the separate review for this.

Change-Id: I890e46bb7a2973c3582be7a37ee70730d1e3ffef
---
 tasks/horizon_install.yml     | 2 ++
 tasks/horizon_pre_install.yml | 1 +
 2 files changed, 3 insertions(+)

diff --git a/tasks/horizon_install.yml b/tasks/horizon_install.yml
index 74841d15..11446a6e 100644
--- a/tasks/horizon_install.yml
+++ b/tasks/horizon_install.yml
@@ -52,6 +52,7 @@
   when:
     - horizon_venv_enabled | bool
     - horizon_get_venv | failed
+  notify: Restart apache2
   tags:
     - horizon-pip-packages
 
@@ -66,5 +67,6 @@
   delay: 2
   with_items: horizon_pip_packages
   when: not horizon_venv_enabled | bool
+  notify: Restart apache2
   tags:
     - horizon-pip-packages
diff --git a/tasks/horizon_pre_install.yml b/tasks/horizon_pre_install.yml
index cbfd14a2..6c2dda7a 100644
--- a/tasks/horizon_pre_install.yml
+++ b/tasks/horizon_pre_install.yml
@@ -149,6 +149,7 @@
   when:
     - horizon_venv_enabled | bool
     - horizon_get_venv | changed
+  notify: Restart apache2
   tags:
     - horizon-install
     - horizon-pip-packages