openstack
/
tripleo-common
Code Issues Proposed changes

[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>
This commit is contained in:
Luke Short 2020-05-22 20:48:45 -04:00
parent d9433a5af0
commit f161bfb049
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
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: