diff options
| author |   Emilien Macchi <emilien@redhat.com> | 2017-02-27 13:04:08 -0500 |
|---|---|---|
| committer | Emilien Macchi <emilien@redhat.com> | 2017-03-02 12:16:20 +0000 |
| commit | 53225d57b1c4e3de722f9cdc25eefa9a3dec1e97 (patch) | |
| tree | 3fd6b9776e3ac76b3eea744c1b0f276f6ef3d0e0 /puppet/services/horizon.yaml | |
| parent | 74a99e72828c2bda46c856fc4983ad1ae1a4472f (diff) | |
upgrades/validation: only run validation when services exist
During upgrades, validation test if a service is running before the
upgrade process starts.
In some cases, servies doesn't exist yet so we don't want to run the
validation.
This patch makes sure we check if the service is actually present on the
system before validating it's running correctly.
Also it makes sure that services are enabled before trying to stop them.
It allows use-cases where we want to add new services during an upgrade.
Also install new packages of services added in Ocata, so we can validate
upgrades on scenarios jobs.
Change-Id: Ib48fb6b1557be43956557cbde4cbe26b53a50bd8
(cherry picked from commit 7c84a9b390c469e716e5802eef078d2df3902c6a)
Diffstat (limited to 'puppet/services/horizon.yaml')
| -rw-r--r-- | puppet/services/horizon.yaml | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/puppet/services/horizon.yaml b/puppet/services/horizon.yaml index 9fa72a3b..60b009a8 100644 --- a/puppet/services/horizon.yaml +++ b/puppet/services/horizon.yaml @@ -100,7 +100,7 @@ outputs: upgrade_tasks: - name: Check if httpd is deployed command: systemctl is-enabled httpd - tags: step0,validation + tags: common ignore_errors: True register: httpd_enabled - name: "PreUpgrade step0,validation: Check if httpd is running" @@ -111,4 +111,5 @@ outputs: tags: step0,validation - name: Stop Horizon (under httpd) tags: step1 + when: httpd_enabled.rc == 0 service: name=httpd state=stopped |