diff --git a/docsite/rst/playbooks_variables.rst b/docsite/rst/playbooks_variables.rst index 5c5cad0ca93..ded843fb942 100644 --- a/docsite/rst/playbooks_variables.rst +++ b/docsite/rst/playbooks_variables.rst @@ -801,11 +801,11 @@ In 2.x, we have made the order of precedence more specific (with the last listed * playbook group_vars * playbook host_vars * host facts - * registered vars - * set_facts * play vars * play vars_prompt * play vars_files + * registered vars + * set_facts * role and include vars * block vars (only for tasks in block) * task vars (only for the task) diff --git a/lib/ansible/vars/__init__.py b/lib/ansible/vars/__init__.py index 25dd9989aa2..f64764a890e 100644 --- a/lib/ansible/vars/__init__.py +++ b/lib/ansible/vars/__init__.py @@ -327,20 +327,19 @@ class VariableManager: for role in play.get_roles(): all_vars = combine_vars(all_vars, role.get_vars(include_params=False)) + if host: + all_vars = combine_vars(all_vars, self._vars_cache.get(host.get_name(), dict())) + all_vars = combine_vars(all_vars, self._nonpersistent_fact_cache.get(host.name, dict())) + if task: if task._role: all_vars = combine_vars(all_vars, task._role.get_vars(include_params=False)) all_vars = combine_vars(all_vars, task._role.get_role_params(task._block.get_dep_chain())) all_vars = combine_vars(all_vars, task.get_vars()) - if host: - all_vars = combine_vars(all_vars, self._vars_cache.get(host.get_name(), dict())) - all_vars = combine_vars(all_vars, self._nonpersistent_fact_cache.get(host.name, dict())) - - # special case for include tasks, where the include params - # may be specified in the vars field for the task, which should - # have higher precedence than the vars/np facts above - if task: + # special case for include tasks, where the include params + # may be specified in the vars field for the task, which should + # have higher precedence than the vars/np facts above all_vars = combine_vars(all_vars, task.get_include_params()) all_vars = combine_vars(all_vars, self._extra_vars)