From 08750536e766e946e85fa37f659c6ce4368fd35b Mon Sep 17 00:00:00 2001 From: licanwei Date: Fri, 2 Nov 2018 11:58:40 +0800 Subject: [PATCH] optimize get_instances_by_node We can set host filed in search_opts. refer to: https://developer.openstack.org/api-ref/compute/?expanded=list-servers-detail#list-servers Change-Id: I36b27167d7223f3bf6bb05995210af41ad01fc6d --- watcher/common/nova_helper.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/watcher/common/nova_helper.py b/watcher/common/nova_helper.py index 1e6f78974..83b3e1981 100644 --- a/watcher/common/nova_helper.py +++ b/watcher/common/nova_helper.py @@ -872,9 +872,9 @@ class NovaHelper(object): def get_instances_by_node(self, host): return [instance for instance in - self.nova.servers.list(search_opts={"all_tenants": True}, - limit=-1) - if self.get_hostname(instance) == host] + self.nova.servers.list(search_opts={"all_tenants": True, + "host": host}, + limit=-1)] def get_hostname(self, instance): return str(getattr(instance, 'OS-EXT-SRV-ATTR:host'))