Merge "Fix positional arg swallow decorator"

1 files changed, 3 insertions, 6 deletions

diff --git a/nova/api/openstack/compute/contrib/hosts.py b/nova/api/openstack/compute/contrib/hosts.py
index 8982f5724..0d2b68f11 100644
--- a/nova/api/openstack/compute/contrib/hosts.py
+++ b/nova/api/openstack/compute/contrib/hosts.py
@@ -93,7 +93,7 @@ class HostUpdateDeserializer(wsgi.XMLDeserializer):
         return dict(body=updates)
 
 
-def _list_hosts(req, service=None):
+def _list_hosts(req):
     """Returns a summary list of hosts, optionally filtering
     by service type.
     """
@@ -108,16 +108,13 @@ def _list_hosts(req, service=None):
     for host in services:
         hosts.append({"host_name": host['host'], 'service': host['topic'],
                       'zone': host['availability_zone']})
-    if service:
-        hosts = [host for host in hosts
-                 if host["service"] == service]
     return hosts
 
 
 def check_host(fn):
     """Makes sure that the host exists."""
-    def wrapped(self, req, id, service=None, *args, **kwargs):
-        listed_hosts = _list_hosts(req, service)
+    def wrapped(self, req, id, *args, **kwargs):
+        listed_hosts = _list_hosts(req)
         hosts = [h["host_name"] for h in listed_hosts]
         if id in hosts:
             return fn(self, req, id, *args, **kwargs)