Don't process the same host multiple times

Fixes #14259
pull/15155/head
Toshio Kuratomi 9 years ago
parent 3de08e1bfc
commit 17e8500202

@ -159,7 +159,7 @@ class InventoryDirectory(object):
if 'ungrouped' in self.groups: if 'ungrouped' in self.groups:
ungrouped = self.groups['ungrouped'] ungrouped = self.groups['ungrouped']
# loop on a copy of ungrouped hosts, as we want to change that list # loop on a copy of ungrouped hosts, as we want to change that list
for host in ungrouped.hosts[:]: for host in frozenset(ungrouped.hosts):
if len(host.groups) > 1: if len(host.groups) > 1:
host.groups.remove(ungrouped) host.groups.remove(ungrouped)
ungrouped.hosts.remove(host) ungrouped.hosts.remove(host)

Loading…
Cancel
Save