Merge pull request #12114 from ilya-epifanov/devel

fixed hostvars access in conjunction with --limit usage

lib/ansible/inventory/__init__.py

@@ -178,7 +178,7 @@ class Inventory(object):
 
         return [x for x in term.findall(pattern) if x]
 
-    def get_hosts(self, pattern="all"):
+    def get_hosts(self, pattern="all", ignore_limits_and_restrictions=False):
         """ 
         Takes a pattern or list of patterns and returns a list of matching
         inventory host names, taking into account any active restrictions
@@ -196,14 +196,16 @@ class Inventory(object):
         patterns = self._split_pattern(pattern)
         hosts = self._evaluate_patterns(patterns)
 
-        # exclude hosts not in a subset, if defined
-        if self._subset:
-            subset = self._evaluate_patterns(self._subset)
-            hosts = [ h for h in hosts if h in subset ]
+        # mainly useful for hostvars[host] access
+        if not ignore_limits_and_restrictions:
+            # exclude hosts not in a subset, if defined
+            if self._subset:
+                subset = self._evaluate_patterns(self._subset)
+                hosts = [ h for h in hosts if h in subset ]
 
-        # exclude hosts mentioned in any restriction (ex: failed hosts)
-        if self._restriction is not None:
-            hosts = [ h for h in hosts if h in self._restriction ]
+            # exclude hosts mentioned in any restriction (ex: failed hosts)
+            if self._restriction is not None:
+                hosts = [ h for h in hosts if h in self._restriction ]
 
         return hosts
 

lib/ansible/vars/hostvars.py

@@ -43,7 +43,7 @@ class HostVars(collections.Mapping):
         # in inventory
         restriction = inventory._restriction
         inventory.remove_restriction()
-        hosts = inventory.get_hosts()
+        hosts = inventory.get_hosts(ignore_limits_and_restrictions=True)
         inventory.restrict_to_hosts(restriction)
 
         # check to see if localhost is in the hosts list, as we