From 05daa35ccc7c90d4655f20bc1ac9e3c777c98e16 Mon Sep 17 00:00:00 2001
From: Jesse Pretorius <jesse.pretorius@rackspace.co.uk>
Date: Tue, 2 Aug 2016 14:19:35 +0100
Subject: [PATCH] Add ability to change apt/yum package state

The current method of installing the distribution packages required is
set in the tasks and cannot be changed by a deployer.

Currently the apt task always installs the latest package. This results
in unexpected binary changes when a deployer may simply be trying to
execute a configuration change.

This patch adds the ability for a deployer to change the desired state
so that the results are predictable.

Change-Id: Iebd947fc93738e6ccb62460061b00306b96437d6
---
 defaults/main.yml                                   |  4 ++++
 .../notes/package-state-fda322f5e667bbec.yaml       | 13 +++++++++++++
 tasks/install-apt.yml                               |  2 +-
 3 files changed, 18 insertions(+), 1 deletion(-)
 create mode 100644 releasenotes/notes/package-state-fda322f5e667bbec.yaml

diff --git a/defaults/main.yml b/defaults/main.yml
index 04b30625..f59f78f4 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -16,6 +16,10 @@
 ## Verbosity Options
 debug: False
 
+# Set the package install state for distribution packages
+# Options are 'present' and 'latest'
+horizon_package_state: "latest"
+
 horizon_git_repo: https://git.openstack.org/openstack/horizon
 horizon_git_install_branch: master
 horizon_developer_mode: false
diff --git a/releasenotes/notes/package-state-fda322f5e667bbec.yaml b/releasenotes/notes/package-state-fda322f5e667bbec.yaml
new file mode 100644
index 00000000..74a8489b
--- /dev/null
+++ b/releasenotes/notes/package-state-fda322f5e667bbec.yaml
@@ -0,0 +1,13 @@
+---
+features:
+  - The os_horizon role now supports the ability to configure whether
+    apt/yum tasks install the latest available package, or just ensure
+    that the package is present. The default action is to ensure that
+    the latest package is present. The action taken may be changed to
+    only ensure that the package is present by setting
+    ``horizon_package_state`` to ``present``.
+upgrade:
+  - The os_horizon role always checks whether the latest package is
+    installed when executed. If a deployer wishes to change the check to
+    only validate the presence of the package, the option
+    ``horizon_package_state`` should be set to ``present``.
diff --git a/tasks/install-apt.yml b/tasks/install-apt.yml
index e62fba95..25102c07 100644
--- a/tasks/install-apt.yml
+++ b/tasks/install-apt.yml
@@ -29,7 +29,7 @@
 - name: Install apt packages
   apt:
     pkg: "{{ item }}"
-    state: latest
+    state: "{{ horizon_package_state }}"
   register: install_packages
   until: install_packages|success
   retries: 5