diff --git a/changelogs/fragments/79945-host_group_vars-improvements.yml b/changelogs/fragments/79945-host_group_vars-improvements.yml new file mode 100644 index 00000000000..684ecdb72dc --- /dev/null +++ b/changelogs/fragments/79945-host_group_vars-improvements.yml @@ -0,0 +1,5 @@ +bugfixes: + - Cache host_group_vars after instantiating it once and limit the amount of repetitive work it needs to do every time it runs. + - Call PluginLoader.all() once for vars plugins, and load vars plugins that run automatically or are enabled specifically by name subsequently. +deprecated_features: + - Old style vars plugins which use the entrypoints `get_host_vars` or `get_group_vars` are deprecated. The plugin should be updated to inherit from `BaseVarsPlugin` and define a `get_vars` method as the entrypoint. diff --git a/lib/ansible/inventory/group.py b/lib/ansible/inventory/group.py index e81cca1b3da..65f1afed5c6 100644 --- a/lib/ansible/inventory/group.py +++ b/lib/ansible/inventory/group.py @@ -18,6 +18,7 @@ from __future__ import (absolute_import, division, print_function) __metaclass__ = type from collections.abc import Mapping, MutableMapping +from enum import Enum from itertools import chain from ansible import constants as C @@ -53,8 +54,14 @@ def to_safe_group_name(name, replacer="_", force=False, silent=False): return name +class InventoryObjectType(Enum): + HOST = 0 + GROUP = 1 + + class Group: ''' a group of ansible hosts ''' + base_type = InventoryObjectType.GROUP # __slots__ = [ 'name', 'hosts', 'vars', 'child_groups', 'parent_groups', 'depth', '_hosts_cache' ] diff --git a/lib/ansible/inventory/host.py b/lib/ansible/inventory/host.py index 18569ce50b6..d8b4c6c694b 100644 --- a/lib/ansible/inventory/host.py +++ b/lib/ansible/inventory/host.py @@ -21,7 +21,7 @@ __metaclass__ = type from collections.abc import Mapping, MutableMapping -from ansible.inventory.group import Group +from ansible.inventory.group import Group, InventoryObjectType from ansible.parsing.utils.addresses import patterns from ansible.utils.vars import combine_vars, get_unique_id @@ -31,6 +31,7 @@ __all__ = ['Host'] class Host: ''' a single ansible host ''' + base_type = InventoryObjectType.HOST # __slots__ = [ 'name', 'vars', 'groups' ] diff --git a/lib/ansible/plugins/loader.py b/lib/ansible/plugins/loader.py index dc0eb891a9a..39b10950635 100644 --- a/lib/ansible/plugins/loader.py +++ b/lib/ansible/plugins/loader.py @@ -238,6 +238,10 @@ class PluginLoader: self._module_cache = MODULE_CACHE[class_name] self._paths = PATH_CACHE[class_name] self._plugin_path_cache = PLUGIN_PATH_CACHE[class_name] + try: + self._plugin_instance_cache = {} if self.type == 'vars' else None + except ValueError: + self._plugin_instance_cache = None self._searched_paths = set() @@ -262,6 +266,7 @@ class PluginLoader: self._module_cache = MODULE_CACHE[self.class_name] self._paths = PATH_CACHE[self.class_name] self._plugin_path_cache = PLUGIN_PATH_CACHE[self.class_name] + self._plugin_instance_cache = {} if self.type == 'vars' else None self._searched_paths = set() def __setstate__(self, data): @@ -866,23 +871,35 @@ class PluginLoader: collection_list = kwargs.pop('collection_list', None) if name in self.aliases: name = self.aliases[name] + + if self._plugin_instance_cache and (cached_load_result := self._plugin_instance_cache.get(name)): + # Resolving the FQCN is slow, even if we've passed in the resolved FQCN. + # Short-circuit here if we've previously resolved this name. + # This will need to be restricted if non-vars plugins start using the cache, since + # some non-fqcn plugin need to be resolved again with the collections list. + return get_with_context_result(*cached_load_result) + plugin_load_context = self.find_plugin_with_context(name, collection_list=collection_list) if not plugin_load_context.resolved or not plugin_load_context.plugin_resolved_path: # FIXME: this is probably an error (eg removed plugin) return get_with_context_result(None, plugin_load_context) fq_name = plugin_load_context.resolved_fqcn - if '.' not in fq_name: + if '.' not in fq_name and plugin_load_context.plugin_resolved_collection: fq_name = '.'.join((plugin_load_context.plugin_resolved_collection, fq_name)) - name = plugin_load_context.plugin_resolved_name + resolved_type_name = plugin_load_context.plugin_resolved_name path = plugin_load_context.plugin_resolved_path + if self._plugin_instance_cache and (cached_load_result := self._plugin_instance_cache.get(fq_name)): + # This is unused by vars plugins, but it's here in case the instance cache expands to other plugin types. + # We get here if we've seen this plugin before, but it wasn't called with the resolved FQCN. + return get_with_context_result(*cached_load_result) redirected_names = plugin_load_context.redirect_list or [] if path not in self._module_cache: - self._module_cache[path] = self._load_module_source(name, path) + self._module_cache[path] = self._load_module_source(resolved_type_name, path) found_in_cache = False - self._load_config_defs(name, self._module_cache[path], path) + self._load_config_defs(resolved_type_name, self._module_cache[path], path) obj = getattr(self._module_cache[path], self.class_name) @@ -899,24 +916,27 @@ class PluginLoader: return get_with_context_result(None, plugin_load_context) # FIXME: update this to use the load context - self._display_plugin_load(self.class_name, name, self._searched_paths, path, found_in_cache=found_in_cache, class_only=class_only) + self._display_plugin_load(self.class_name, resolved_type_name, self._searched_paths, path, found_in_cache=found_in_cache, class_only=class_only) if not class_only: try: # A plugin may need to use its _load_name in __init__ (for example, to set # or get options from config), so update the object before using the constructor instance = object.__new__(obj) - self._update_object(instance, name, path, redirected_names, fq_name) + self._update_object(instance, resolved_type_name, path, redirected_names, fq_name) obj.__init__(instance, *args, **kwargs) # pylint: disable=unnecessary-dunder-call obj = instance except TypeError as e: if "abstract" in e.args[0]: # Abstract Base Class or incomplete plugin, don't load - display.v('Returning not found on "%s" as it has unimplemented abstract methods; %s' % (name, to_native(e))) + display.v('Returning not found on "%s" as it has unimplemented abstract methods; %s' % (resolved_type_name, to_native(e))) return get_with_context_result(None, plugin_load_context) raise - self._update_object(obj, name, path, redirected_names, fq_name) + self._update_object(obj, resolved_type_name, path, redirected_names, fq_name) + if self._plugin_instance_cache is not None and getattr(obj, 'is_stateless', False): + # store under both the originally requested name and the resolved FQ name + self._plugin_instance_cache[name] = self._plugin_instance_cache[fq_name] = (obj, plugin_load_context) return get_with_context_result(obj, plugin_load_context) def _display_plugin_load(self, class_name, name, searched_paths, path, found_in_cache=None, class_only=None): @@ -1008,6 +1028,16 @@ class PluginLoader: yield path continue + if path in legacy_excluding_builtin: + fqcn = basename + else: + fqcn = f"ansible.builtin.{basename}" + + if self._plugin_instance_cache is not None and fqcn in self._plugin_instance_cache: + # Here just in case, but we don't call all() multiple times for vars plugins, so this should not be used. + yield self._plugin_instance_cache[basename][0] + continue + if path not in self._module_cache: if self.type in ('filter', 'test'): # filter and test plugin files can contain multiple plugins @@ -1055,11 +1085,12 @@ class PluginLoader: except TypeError as e: display.warning("Skipping plugin (%s) as it seems to be incomplete: %s" % (path, to_text(e))) - if path in legacy_excluding_builtin: - fqcn = basename - else: - fqcn = f"ansible.builtin.{basename}" self._update_object(obj, basename, path, resolved=fqcn) + + if self._plugin_instance_cache is not None and fqcn not in self._plugin_instance_cache: + # Use get_with_context to cache the plugin the first time we see it. + self.get_with_context(fqcn)[0] + yield obj diff --git a/lib/ansible/plugins/vars/__init__.py b/lib/ansible/plugins/vars/__init__.py index 2a7bafd9ad9..4f9045b067f 100644 --- a/lib/ansible/plugins/vars/__init__.py +++ b/lib/ansible/plugins/vars/__init__.py @@ -30,6 +30,7 @@ class BaseVarsPlugin(AnsiblePlugin): """ Loads variables for groups and/or hosts """ + is_stateless = False def __init__(self): """ constructor """ diff --git a/lib/ansible/plugins/vars/host_group_vars.py b/lib/ansible/plugins/vars/host_group_vars.py index acb1a4afe0c..28b42131ed1 100644 --- a/lib/ansible/plugins/vars/host_group_vars.py +++ b/lib/ansible/plugins/vars/host_group_vars.py @@ -55,18 +55,29 @@ DOCUMENTATION = ''' import os from ansible.errors import AnsibleParserError -from ansible.module_utils.common.text.converters import to_bytes, to_native, to_text +from ansible.module_utils.common.text.converters import to_native from ansible.plugins.vars import BaseVarsPlugin -from ansible.inventory.host import Host -from ansible.inventory.group import Group +from ansible.utils.path import basedir +from ansible.inventory.group import InventoryObjectType from ansible.utils.vars import combine_vars +CANONICAL_PATHS = {} # type: dict[str, str] FOUND = {} # type: dict[str, list[str]] +NAK = set() # type: set[str] +PATH_CACHE = {} # type: dict[tuple[str, str], str] class VarsModule(BaseVarsPlugin): REQUIRES_ENABLED = True + is_stateless = True + + def load_found_files(self, loader, data, found_files): + for found in found_files: + new_data = loader.load_from_file(found, cache=True, unsafe=True) + if new_data: # ignore empty files + data = combine_vars(data, new_data) + return data def get_vars(self, loader, path, entities, cache=True): ''' parses the inventory file ''' @@ -74,41 +85,68 @@ class VarsModule(BaseVarsPlugin): if not isinstance(entities, list): entities = [entities] - super(VarsModule, self).get_vars(loader, path, entities) + # realpath is expensive + try: + realpath_basedir = CANONICAL_PATHS[path] + except KeyError: + CANONICAL_PATHS[path] = realpath_basedir = os.path.realpath(basedir(path)) data = {} for entity in entities: - if isinstance(entity, Host): - subdir = 'host_vars' - elif isinstance(entity, Group): - subdir = 'group_vars' - else: + try: + entity_name = entity.name + except AttributeError: + raise AnsibleParserError("Supplied entity must be Host or Group, got %s instead" % (type(entity))) + + try: + first_char = entity_name[0] + except (TypeError, IndexError, KeyError): raise AnsibleParserError("Supplied entity must be Host or Group, got %s instead" % (type(entity))) # avoid 'chroot' type inventory hostnames /path/to/chroot - if not entity.name.startswith(os.path.sep): + if first_char != os.path.sep: try: found_files = [] # load vars - b_opath = os.path.realpath(to_bytes(os.path.join(self._basedir, subdir))) - opath = to_text(b_opath) - key = '%s.%s' % (entity.name, opath) - if cache and key in FOUND: - found_files = FOUND[key] + try: + entity_type = entity.base_type + except AttributeError: + raise AnsibleParserError("Supplied entity must be Host or Group, got %s instead" % (type(entity))) + + if entity_type is InventoryObjectType.HOST: + subdir = 'host_vars' + elif entity_type is InventoryObjectType.GROUP: + subdir = 'group_vars' else: - # no need to do much if path does not exist for basedir - if os.path.exists(b_opath): - if os.path.isdir(b_opath): - self._display.debug("\tprocessing dir %s" % opath) - found_files = loader.find_vars_files(opath, entity.name) - FOUND[key] = found_files - else: - self._display.warning("Found %s that is not a directory, skipping: %s" % (subdir, opath)) - - for found in found_files: - new_data = loader.load_from_file(found, cache=True, unsafe=True) - if new_data: # ignore empty files - data = combine_vars(data, new_data) + raise AnsibleParserError("Supplied entity must be Host or Group, got %s instead" % (type(entity))) + + if cache: + try: + opath = PATH_CACHE[(realpath_basedir, subdir)] + except KeyError: + opath = PATH_CACHE[(realpath_basedir, subdir)] = os.path.join(realpath_basedir, subdir) + + if opath in NAK: + continue + key = '%s.%s' % (entity_name, opath) + if key in FOUND: + data = self.load_found_files(loader, data, FOUND[key]) + continue + else: + opath = PATH_CACHE[(realpath_basedir, subdir)] = os.path.join(realpath_basedir, subdir) + + if os.path.isdir(opath): + self._display.debug("\tprocessing dir %s" % opath) + FOUND[key] = found_files = loader.find_vars_files(opath, entity_name) + elif not os.path.exists(opath): + # cache missing dirs so we don't have to keep looking for things beneath the + NAK.add(opath) + else: + self._display.warning("Found %s that is not a directory, skipping: %s" % (subdir, opath)) + # cache non-directory matches + NAK.add(opath) + + data = self.load_found_files(loader, data, found_files) except Exception as e: raise AnsibleParserError(to_native(e)) diff --git a/lib/ansible/utils/vars.py b/lib/ansible/utils/vars.py index c8c842938aa..07d635558f8 100644 --- a/lib/ansible/utils/vars.py +++ b/lib/ansible/utils/vars.py @@ -109,6 +109,8 @@ def merge_hash(x, y, recursive=True, list_merge='replace'): # except performance) if x == {} or x == y: return y.copy() + if y == {}: + return x # in the following we will copy elements from y to x, but # we don't want to modify x, so we create a copy of it diff --git a/lib/ansible/vars/manager.py b/lib/ansible/vars/manager.py index 54a0b87753a..ec69f55a028 100644 --- a/lib/ansible/vars/manager.py +++ b/lib/ansible/vars/manager.py @@ -184,6 +184,9 @@ class VariableManager: See notes in the VarsWithSources docstring for caveats and limitations of the source tracking ''' + if new_data == {}: + return data + if C.DEFAULT_DEBUG: # Populate var sources dict for key in new_data: diff --git a/lib/ansible/vars/plugins.py b/lib/ansible/vars/plugins.py index 215f0975381..37624d9de78 100644 --- a/lib/ansible/vars/plugins.py +++ b/lib/ansible/vars/plugins.py @@ -1,23 +1,57 @@ # Copyright (c) 2018 Ansible Project # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt) -# Make coding more python3-ish -from __future__ import (absolute_import, division, print_function) -__metaclass__ = type +from __future__ import annotations import os +from functools import lru_cache + from ansible import constants as C from ansible.errors import AnsibleError -from ansible.inventory.host import Host -from ansible.module_utils.common.text.converters import to_bytes +from ansible.inventory.group import InventoryObjectType from ansible.plugins.loader import vars_loader -from ansible.utils.collection_loader import AnsibleCollectionRef from ansible.utils.display import Display from ansible.utils.vars import combine_vars display = Display() +cached_vars_plugin_order = None + + +def _load_vars_plugins_order(): + # find 3rd party legacy vars plugins once, and look them up by name subsequently + auto = [] + for auto_run_plugin in vars_loader.all(class_only=True): + needs_enabled = False + if hasattr(auto_run_plugin, 'REQUIRES_ENABLED'): + needs_enabled = auto_run_plugin.REQUIRES_ENABLED + elif hasattr(auto_run_plugin, 'REQUIRES_WHITELIST'): + needs_enabled = auto_run_plugin.REQUIRES_WHITELIST + display.deprecated("The VarsModule class variable 'REQUIRES_WHITELIST' is deprecated. " + "Use 'REQUIRES_ENABLED' instead.", version=2.18) + if needs_enabled: + continue + auto.append(auto_run_plugin._load_name) + + # find enabled plugins once so we can look them up by resolved fqcn subsequently + enabled = [] + for plugin_name in C.VARIABLE_PLUGINS_ENABLED: + if (plugin := vars_loader.get(plugin_name)) is None: + enabled.append(plugin_name) + else: + collection = '.' in plugin.ansible_name and not plugin.ansible_name.startswith('ansible.builtin.') + # Warn if a collection plugin has REQUIRES_ENABLED because it has no effect. + if collection and (hasattr(plugin, 'REQUIRES_ENABLED') or hasattr(plugin, 'REQUIRES_WHITELIST')): + display.warning( + "Vars plugins in collections must be enabled to be loaded, REQUIRES_ENABLED is not supported. " + "This should be removed from the plugin %s." % plugin.ansible_name + ) + enabled.append(plugin.ansible_name) + + global cached_vars_plugin_order + cached_vars_plugin_order = auto + enabled + def get_plugin_vars(loader, plugin, path, entities): @@ -25,9 +59,17 @@ def get_plugin_vars(loader, plugin, path, entities): try: data = plugin.get_vars(loader, path, entities) except AttributeError: + if hasattr(plugin, 'get_host_vars') or hasattr(plugin, 'get_group_vars'): + display.deprecated( + f"The vars plugin {plugin.ansible_name} from {plugin._original_path} is relying " + "on the deprecated entrypoints 'get_host_vars' and 'get_group_vars'. " + "This plugin should be updated to inherit from BaseVarsPlugin and define " + "a 'get_vars' method as the main entrypoint instead.", + version="2.20", + ) try: for entity in entities: - if isinstance(entity, Host): + if entity.base_type is InventoryObjectType.HOST: data |= plugin.get_host_vars(entity.name) else: data |= plugin.get_group_vars(entity.name) @@ -39,59 +81,46 @@ def get_plugin_vars(loader, plugin, path, entities): return data +# optimized for stateless plugins; non-stateless plugin instances will fall out quickly +@lru_cache(maxsize=10) +def _plugin_should_run(plugin, stage): + # if a plugin-specific setting has not been provided, use the global setting + # older/non shipped plugins that don't support the plugin-specific setting should also use the global setting + allowed_stages = None + + try: + allowed_stages = plugin.get_option('stage') + except (AttributeError, KeyError): + pass + + if allowed_stages: + return allowed_stages in ('all', stage) + + # plugin didn't declare a preference; consult global config + config_stage_override = C.RUN_VARS_PLUGINS + if config_stage_override == 'demand' and stage == 'inventory': + return False + elif config_stage_override == 'start' and stage == 'task': + return False + return True + + def get_vars_from_path(loader, path, entities, stage): data = {} - vars_plugin_list = list(vars_loader.all()) - for plugin_name in C.VARIABLE_PLUGINS_ENABLED: - if AnsibleCollectionRef.is_valid_fqcr(plugin_name): - vars_plugin = vars_loader.get(plugin_name) - if vars_plugin is None: - # Error if there's no play directory or the name is wrong? - continue - if vars_plugin not in vars_plugin_list: - vars_plugin_list.append(vars_plugin) - - for plugin in vars_plugin_list: - # legacy plugins always run by default, but they can set REQUIRES_ENABLED=True to opt out. - - builtin_or_legacy = plugin.ansible_name.startswith('ansible.builtin.') or '.' not in plugin.ansible_name - - # builtin is supposed to have REQUIRES_ENABLED=True, the following is for legacy plugins... - needs_enabled = not builtin_or_legacy - if hasattr(plugin, 'REQUIRES_ENABLED'): - needs_enabled = plugin.REQUIRES_ENABLED - elif hasattr(plugin, 'REQUIRES_WHITELIST'): - display.deprecated("The VarsModule class variable 'REQUIRES_WHITELIST' is deprecated. " - "Use 'REQUIRES_ENABLED' instead.", version="2.18") - needs_enabled = plugin.REQUIRES_WHITELIST - - # A collection plugin was enabled to get to this point because vars_loader.all() does not include collection plugins. - # Warn if a collection plugin has REQUIRES_ENABLED because it has no effect. - if not builtin_or_legacy and (hasattr(plugin, 'REQUIRES_ENABLED') or hasattr(plugin, 'REQUIRES_WHITELIST')): - display.warning( - "Vars plugins in collections must be enabled to be loaded, REQUIRES_ENABLED is not supported. " - "This should be removed from the plugin %s." % plugin.ansible_name - ) - elif builtin_or_legacy and needs_enabled and not plugin.matches_name(C.VARIABLE_PLUGINS_ENABLED): - continue - - has_stage = hasattr(plugin, 'get_option') and plugin.has_option('stage') + if cached_vars_plugin_order is None: + _load_vars_plugins_order() - # if a plugin-specific setting has not been provided, use the global setting - # older/non shipped plugins that don't support the plugin-specific setting should also use the global setting - use_global = (has_stage and plugin.get_option('stage') is None) or not has_stage + for plugin_name in cached_vars_plugin_order: + if (plugin := vars_loader.get(plugin_name)) is None: + continue - if use_global: - if C.RUN_VARS_PLUGINS == 'demand' and stage == 'inventory': - continue - elif C.RUN_VARS_PLUGINS == 'start' and stage == 'task': - continue - elif has_stage and plugin.get_option('stage') not in ('all', stage): + if not _plugin_should_run(plugin, stage): continue - data = combine_vars(data, get_plugin_vars(loader, plugin, path, entities)) + if (new_vars := get_plugin_vars(loader, plugin, path, entities)) != {}: + data = combine_vars(data, new_vars) return data @@ -105,10 +134,11 @@ def get_vars_from_inventory_sources(loader, sources, entities, stage): continue if ',' in path and not os.path.exists(path): # skip host lists continue - elif not os.path.isdir(to_bytes(path)): + elif not os.path.isdir(path): # always pass the directory of the inventory source file path = os.path.dirname(path) - data = combine_vars(data, get_vars_from_path(loader, path, entities, stage)) + if (new_vars := get_vars_from_path(loader, path, entities, stage)) != {}: + data = combine_vars(data, new_vars) return data diff --git a/test/integration/targets/old_style_vars_plugins/deprecation_warning/v2_vars_plugin.py b/test/integration/targets/old_style_vars_plugins/deprecation_warning/v2_vars_plugin.py new file mode 100644 index 00000000000..f342b698a01 --- /dev/null +++ b/test/integration/targets/old_style_vars_plugins/deprecation_warning/v2_vars_plugin.py @@ -0,0 +1,6 @@ +class VarsModule: + def get_host_vars(self, entity): + return {} + + def get_group_vars(self, entity): + return {} diff --git a/test/integration/targets/old_style_vars_plugins/deprecation_warning/vars.py b/test/integration/targets/old_style_vars_plugins/deprecation_warning/vars.py index d5c9a422dcf..f554be04fbd 100644 --- a/test/integration/targets/old_style_vars_plugins/deprecation_warning/vars.py +++ b/test/integration/targets/old_style_vars_plugins/deprecation_warning/vars.py @@ -2,7 +2,7 @@ from ansible.plugins.vars import BaseVarsPlugin class VarsModule(BaseVarsPlugin): - REQUIRES_WHITELIST = False + REQUIRES_WHITELIST = True def get_vars(self, loader, path, entities): return {} diff --git a/test/integration/targets/old_style_vars_plugins/runme.sh b/test/integration/targets/old_style_vars_plugins/runme.sh index 4cd1916819c..71275b8ac47 100755 --- a/test/integration/targets/old_style_vars_plugins/runme.sh +++ b/test/integration/targets/old_style_vars_plugins/runme.sh @@ -12,9 +12,37 @@ export ANSIBLE_VARS_PLUGINS=./vars_plugins export ANSIBLE_VARS_ENABLED=require_enabled [ "$(ansible-inventory -i localhost, --list --yaml all "$@" | grep -c 'require_enabled')" = "1" ] -# Test the deprecated class attribute +# Test deprecated features export ANSIBLE_VARS_PLUGINS=./deprecation_warning -WARNING="The VarsModule class variable 'REQUIRES_WHITELIST' is deprecated. Use 'REQUIRES_ENABLED' instead." +WARNING_1="The VarsModule class variable 'REQUIRES_WHITELIST' is deprecated. Use 'REQUIRES_ENABLED' instead." +WARNING_2="The vars plugin v2_vars_plugin .* is relying on the deprecated entrypoints 'get_host_vars' and 'get_group_vars'" ANSIBLE_DEPRECATION_WARNINGS=True ANSIBLE_NOCOLOR=True ANSIBLE_FORCE_COLOR=False \ - ansible-inventory -i localhost, --list all 2> err.txt -ansible localhost -m debug -a "msg={{ lookup('file', 'err.txt') | regex_replace('\n', '') }}" | grep "$WARNING" + ansible-inventory -i localhost, --list all "$@" 2> err.txt +for WARNING in "$WARNING_1" "$WARNING_2"; do + ansible localhost -m debug -a "msg={{ lookup('file', 'err.txt') | regex_replace('\n', '') }}" | grep "$WARNING" +done + +# Test how many times vars plugins are loaded for a simple play containing a task +# host_group_vars is stateless, so we can load it once and reuse it, every other vars plugin should be instantiated before it runs +cat << EOF > "test_task_vars.yml" +--- +- hosts: localhost + connection: local + gather_facts: no + tasks: + - debug: +EOF + +# hide the debug noise by dumping to a file +trap 'rm -rf -- "out.txt"' EXIT + +ANSIBLE_DEBUG=True ansible-playbook test_task_vars.yml > out.txt +[ "$(grep -c "Loading VarsModule 'host_group_vars'" out.txt)" -eq 1 ] +[ "$(grep -c "Loading VarsModule 'require_enabled'" out.txt)" -gt 50 ] +[ "$(grep -c "Loading VarsModule 'auto_enabled'" out.txt)" -gt 50 ] + +export ANSIBLE_VARS_ENABLED=ansible.builtin.host_group_vars +ANSIBLE_DEBUG=True ansible-playbook test_task_vars.yml > out.txt +[ "$(grep -c "Loading VarsModule 'host_group_vars'" out.txt)" -eq 1 ] +[ "$(grep -c "Loading VarsModule 'require_enabled'" out.txt)" -lt 3 ] +[ "$(grep -c "Loading VarsModule 'auto_enabled'" out.txt)" -gt 50 ]