Browse Source

[Train-Only] Fix mixing nodes and limit_hosts.

This corrects a behavior where in some cases the --limit is not
passed to the ansible-playbook command during config_download_deploy.

Change-Id: Ie5b8d41abacb071f528d4f9220bff3532ed0ab2f
Closes-Bug: #1880977
Signed-off-by: Luke Short <ekultails@gmail.com>
changes/26/730426/5
Luke Short 2 months ago
parent
commit
f161bfb049
1 changed files with 3 additions and 3 deletions
  1. +3
    -3
      workbooks/deployment.yaml

+ 3
- 3
workbooks/deployment.yaml View File

@@ -444,9 +444,9 @@ workflows:
fields: ['instance_info']
with-items: node in <% $.nodes %>
publish-on-error:
limit_hosts: <% $.nodes.join(',') %>
nodes_names: <% $.nodes.join(',') %>
publish:
limit_hosts: <% list(task().result.instance_info.display_name).join(',') %>
nodes_names: <% list(task().result.instance_info.display_name).join(',') %>

get_private_key:
action: tripleo.validations.get_privkey
@@ -525,7 +525,7 @@ workflows:
timeout: <% $.connection_timeout %>
command_timeout: <% $.config_download_timeout %>
execution_id: <% execution().id %>
limit_hosts: <% $.nodes and $.get(limit_hosts) or false %>
limit_hosts: <% ($.nodes_names or $.limit_hosts) or false %>
skip_tags: <% $.skip_tags %>
tags: <% $.tags %>
publish:


Loading…
Cancel
Save