mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
fix issue with add_hosts overwriting existing vars
fixes github.com/ansible/ansible-modules-core/issues/2799 and is alternative to #13841
This commit is contained in:
parent
ec95f50cb6
commit
c2d314dec5
1 changed files with 3 additions and 3 deletions
|
@ -40,6 +40,7 @@ from ansible.playbook.included_file import IncludedFile
|
||||||
from ansible.plugins import action_loader, connection_loader, filter_loader, lookup_loader, module_loader, test_loader
|
from ansible.plugins import action_loader, connection_loader, filter_loader, lookup_loader, module_loader, test_loader
|
||||||
from ansible.template import Templar
|
from ansible.template import Templar
|
||||||
from ansible.vars.unsafe_proxy import wrap_var
|
from ansible.vars.unsafe_proxy import wrap_var
|
||||||
|
from ansible.vars import combine_vars
|
||||||
|
|
||||||
try:
|
try:
|
||||||
from __main__ import display
|
from __main__ import display
|
||||||
|
@ -372,9 +373,8 @@ class StrategyBase:
|
||||||
allgroup.add_host(new_host)
|
allgroup.add_host(new_host)
|
||||||
|
|
||||||
# Set/update the vars for this host
|
# Set/update the vars for this host
|
||||||
new_vars = host_info.get('host_vars', dict())
|
new_host.vars = combine_vars(new_host.vars, self._inventory.get_host_vars(new_host))
|
||||||
new_host.vars = self._inventory.get_host_vars(new_host)
|
new_host.vars = combine_vars(new_host.vars, host_info.get('host_vars', dict()))
|
||||||
new_host.vars.update(new_vars)
|
|
||||||
|
|
||||||
new_groups = host_info.get('groups', [])
|
new_groups = host_info.get('groups', [])
|
||||||
for group_name in new_groups:
|
for group_name in new_groups:
|
||||||
|
|
Loading…
Reference in a new issue