From 74517c01255c7ebd42abccd3828f84cdaa839c32 Mon Sep 17 00:00:00 2001
From: Steven Hardy <shardy@redhat.com>
Date: Tue, 15 Nov 2016 22:49:18 +0000
Subject: [PATCH] Correct AllNodesDeploySteps depends_on

This is wrong atm, it should loop to create a list for the depends_on
not multiple depends_on statements.

Note this was first corrected in https://review.openstack.org/#/c/330659/
but we need it as a standalone patch that can be backported.

Change-Id: I4d1d6346f2147e573fc0900038f1ad1d782e75ee
Closes-Bug: #1642069
(cherry picked from commit 8fe71c088889426f5a1547e96b47c569a0a0643c)
---
 overcloud.j2.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/overcloud.j2.yaml b/overcloud.j2.yaml
index 7d74a8abd2..1ff1fb801d 100644
--- a/overcloud.j2.yaml
+++ b/overcloud.j2.yaml
@@ -542,8 +542,8 @@ resources:
   # Post deployment steps for all roles
   AllNodesDeploySteps:
     type: OS::TripleO::PostDeploySteps
-{% for role in roles %}
     depends_on:
+{% for role in roles %}
       - {{role.name}}AllNodesDeployment
 {% endfor %}
     properties: