Commit babf47de by James Cammarata

Clean up some bugs related to --limit on adhoc commands

parent 62157568
...@@ -18,7 +18,7 @@ ...@@ -18,7 +18,7 @@
######################################################## ########################################################
from ansible import constants as C from ansible import constants as C
from ansible.cli import CLI from ansible.cli import CLI
from ansible.errors import AnsibleOptionsError from ansible.errors import AnsibleError, AnsibleOptionsError
from ansible.executor.task_queue_manager import TaskQueueManager from ansible.executor.task_queue_manager import TaskQueueManager
from ansible.inventory import Inventory from ansible.inventory import Inventory
from ansible.parsing import DataLoader from ansible.parsing import DataLoader
...@@ -117,10 +117,11 @@ class AdHocCLI(CLI): ...@@ -117,10 +117,11 @@ class AdHocCLI(CLI):
self.display.warning("provided hosts list is empty, only localhost is available") self.display.warning("provided hosts list is empty, only localhost is available")
no_hosts = True no_hosts = True
inventory.subset(self.options.subset) if self.options.subset:
if len(inventory.list_hosts()) == 0 and not no_hosts: inventory.subset(self.options.subset)
# Invalid limit if len(inventory.list_hosts()) == 0 and not no_hosts:
raise AnsibleError("Specified --limit does not match any hosts") # Invalid limit
raise AnsibleError("Specified --limit does not match any hosts")
if self.options.listhosts: if self.options.listhosts:
self.display.display(' hosts (%d):' % len(hosts)) self.display.display(' hosts (%d):' % len(hosts))
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment