Merge branch 'inventory_refactoring' into svg_and_inventory_refactor

lib/ansible/inventory/__init__.py

@@ -146,6 +146,17 @@ class Inventory(object):
         else:
             return fnmatch.fnmatch(str, pattern_str)
 
+    def _match_list(self, items, item_attr, pattern_str):
+        results = []
+        if not pattern_str.startswith('~'):
+            pattern = re.compile(fnmatch.translate(pattern_str))
+        else:
+            pattern = re.compile(pattern_str[1:])
+        for item in items:
+            if pattern.search(getattr(item, item_attr)):
+                results.append(item)
+        return results
+
     def get_hosts(self, pattern="all"):
         """ 
         find all host names matching a pattern string, taking into account any inventory restrictions or
@@ -201,6 +212,10 @@ class Inventory(object):
         hosts = []
 
         for p in patterns:
+            # avoid resolving a pattern that is a plain host
+            if p in self._hosts_cache:
+                hosts.append(self.get_host(p))
+            else:
                 that = self.__get_hosts(p)
                 if p.startswith("!"):
                     hosts = [ h for h in hosts if h not in that ]
@@ -209,7 +224,6 @@ class Inventory(object):
                 else:
                     to_append = [ h for h in that if h.name not in [ y.name for y in hosts ] ]
                     hosts.extend(to_append)
-        
         return hosts
 
     def __get_hosts(self, pattern):
@@ -296,20 +310,31 @@ class Inventory(object):
     def _hosts_in_unenumerated_pattern(self, pattern):
         """ Get all host names matching the pattern """
 
+        results = []
         hosts = []
         hostnames = set()
 
         # ignore any negative checks here, this is handled elsewhere
         pattern = pattern.replace("!","").replace("&", "")
 
-        results = []
+        def __append_host_to_results(host):
+            if host not in results and host.name not in hostnames:
+                hostnames.add(host.name)
+                results.append(host)
+
         groups = self.get_groups()
         for group in groups:
+            if pattern == 'all':
                 for host in group.get_hosts():
-                if pattern == 'all' or self._match(group.name, pattern) or self._match(host.name, pattern):
+                    __append_host_to_results(host)
-                    if host not in results and host.name not in hostnames:
+            else:
-                        results.append(host)
+                if self._match(group.name, pattern):
-                        hostnames.add(host.name)
+                    for host in group.get_hosts():
+                        __append_host_to_results(host)
+                else:
+                    matching_hosts = self._match_list(group.get_hosts(), 'name', pattern)
+                    for host in matching_hosts:
+                        __append_host_to_results(host)
 
         if pattern in ["localhost", "127.0.0.1"] and len(results) == 0:
             new_host = self._create_implicit_localhost(pattern)
@@ -324,10 +349,8 @@ class Inventory(object):
         results = []
         groups = self.get_groups()
         for group in groups:
-            for hostn in group.get_hosts():
+            if host in group.get_hosts():
-                if host == hostn.name:
                 results.append(group)
-                    continue
         return results
 
     def groups_list(self):

lib/ansible/inventory/group.py

@@ -28,7 +28,8 @@ class Group(object):
         self.vars = {}
         self.child_groups = []
         self.parent_groups = []
-        self.clear_hosts_cache()
+        self._hosts_cache = None
+        #self.clear_hosts_cache()
         if self.name is None:
             raise Exception("group name is required")