From 9ec8c8f8afa0a9cec7af76f6a256ae4b6007eba3 Mon Sep 17 00:00:00 2001
From: Jiri Stransky <jistr@redhat.com>
Date: Tue, 30 Jan 2018 18:52:03 +0100
Subject: [PATCH] Make sure deploy_steps_max is defined for update playbook

Not having deploy_steps_max failed the update. Also adding
non-repetitive facts gathering.

Change-Id: I1848dc47266a35a0ba383e55787c4aea986bd7a9
Closes-Bug: #1746306
---
 common/deploy-steps.j2 | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/common/deploy-steps.j2 b/common/deploy-steps.j2
index 5260eb5272..c2a55883ef 100644
--- a/common/deploy-steps.j2
+++ b/common/deploy-steps.j2
@@ -504,8 +504,22 @@ outputs:
               when: role_name == '{{role.name}}'
 {%- endfor %}
       update_steps_playbook: |
+        - hosts: undercloud
+          name: Gather facts undercloud
+          gather_facts: yes
+          become: false
         - hosts: overcloud
+          name: Gather facts overcloud
+          gather_facts: yes
+        - hosts: all
+          name: Load global variables
+          gather_facts: no
+          tasks:
+            - include_vars: global_vars.yaml
+        - hosts: overcloud
+          name: Run update
           serial: 1
+          gather_facts: no
           tasks:
             - include: update_steps_tasks.yaml
               with_sequence: start=0 end={{update_steps_max-1}}