From 300074ee2b5cb454ecb3826a191f83bb45586196 Mon Sep 17 00:00:00 2001
From: Emilien Macchi <emilien@redhat.com>
Date: Fri, 30 Aug 2019 17:00:07 -0400
Subject: [PATCH] config-download: rename pre/post deployment hostvars

We can't have numbers starting an Ansible var, so we need to rename the
pre/post deployments host vars, or we hit this error:
  template error while templating string: expected token 'end of print statement'

https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#creating-valid-variable-names

In this patch we have to duplicate host vars but it'll be cleaned up
once we change to the new file in THT.

Change-Id: Idbdd6a21eb2cd488daa1e3ddc844b4fc5267047c
Related-Bug: #1842141
(cherry picked from commit 21754e9987abd33b5c4f4d79dfbab324186f1fc8)
(cherry picked from commit 44da685cb03d99d9ae34598741b40ed7af54633c)
(cherry picked from commit 74991a643a616be3cfa63d874497a325075714aa)
---
 tripleo_common/templates/group_var_role.j2 | 14 ++++++++++++++
 tripleo_common/tests/utils/data/Compute    |  8 ++++++++
 tripleo_common/tests/utils/data/Controller |  8 ++++++++
 3 files changed, 30 insertions(+)

diff --git a/tripleo_common/templates/group_var_role.j2 b/tripleo_common/templates/group_var_role.j2
index 49c40a00f..ef81ccf78 100644
--- a/tripleo_common/templates/group_var_role.j2
+++ b/tripleo_common/templates/group_var_role.j2
@@ -3,8 +3,22 @@
   - {{ deployment }}
 {% endfor %}
 
+# Temporary duplication
+# https://bugs.launchpad.net/tripleo/+bug/1842141
+pre_deployments_{{ role }}: {% if not pre_deployments %} [] {% endif %}
+
+{% for deployment in pre_deployments %}
+  - {{ deployment }}
+{% endfor %}
+
 {{ role }}_post_deployments: {% if not post_deployments %} [] {% endif %}
 
 {% for deployment in post_deployments %}
   - {{ deployment }}
 {% endfor %}
+
+post_deployments_{{ role }}: {% if not post_deployments %} [] {% endif %}
+
+{% for deployment in post_deployments %}
+  - {{ deployment }}
+{% endfor %}
diff --git a/tripleo_common/tests/utils/data/Compute b/tripleo_common/tests/utils/data/Compute
index b74d0f6f6..f8960f736 100644
--- a/tripleo_common/tests/utils/data/Compute
+++ b/tripleo_common/tests/utils/data/Compute
@@ -5,3 +5,11 @@ Compute_pre_deployments:
   - AnsibleDeployment
 
 Compute_post_deployments:  []
+
+pre_deployments_Compute:
+  - ComputeHostEntryDeployment
+  - NetworkDeployment
+  - MyExtraConfigPost
+  - AnsibleDeployment
+
+post_deployments_Compute:  []
diff --git a/tripleo_common/tests/utils/data/Controller b/tripleo_common/tests/utils/data/Controller
index f0a6bbd8c..bf4fdeabf 100644
--- a/tripleo_common/tests/utils/data/Controller
+++ b/tripleo_common/tests/utils/data/Controller
@@ -5,3 +5,11 @@ Controller_pre_deployments:
   - MyExtraConfigPost
 
 Controller_post_deployments:  []
+
+pre_deployments_Controller:
+  - ControllerHostEntryDeployment
+  - NetworkDeployment
+  - MyPostConfig
+  - MyExtraConfigPost
+
+post_deployments_Controller:  []