diff options
| author |   Steven Hardy <shardy@redhat.com> | 2017-05-02 11:54:12 +0100 |
|---|---|---|
| committer |   Emilien Macchi <emilien@redhat.com> | 2017-05-02 11:06:03 -0400 |
| commit | f907ee34acdcb756b7c82c208f11fcb8678a54a9 (patch) | |
| tree | 152395b6f761b146545c79f0dfa709ef001e9df7 | |
| parent | cc9695e74e588a8aa35aa3850bb0b336906c78e0 (diff) | |
Ensure AllNodesExtraConfig runs before AllNodesDeploySteps
When implementing custom roles, we lost an implicit dependency that
ensured AllNodesExtraConfig is applied before AllNodesDeploySteps,
which causes problems if you need to write hieradata via the
AllNodesExtraConfig hook (some cisco integrations we have in tree
do this, and are now broken because the ordering is no longer ensured.
Change-Id: Ie78ecbb4e135ab7f196867ef9d8d271049a9cd10
Closes-Bug: #1687597
(cherry picked from commit 4efc067a7e2965fc7a07eb05b019d0e3e8160606)
| -rw-r--r-- | overcloud.j2.yaml | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/overcloud.j2.yaml b/overcloud.j2.yaml index a36e4078..181fb6c7 100644 --- a/overcloud.j2.yaml +++ b/overcloud.j2.yaml @@ -637,6 +637,7 @@ resources: AllNodesDeploySteps: type: OS::TripleO::PostDeploySteps depends_on: + - AllNodesExtraConfig {% for role in roles %} - {{role.name}}AllNodesDeployment {% endfor %} |