diff options
author | Zane Bitter <zbitter@redhat.com> | 2019-01-28 18:54:24 +1300 |
---|---|---|
committer | Zane Bitter <zbitter@redhat.com> | 2020-02-18 09:20:41 -0500 |
commit | 9a4012ba9d99ca30d9f6fc5a226cbd3baa727bc5 (patch) | |
tree | e8e302894016502ec20cd1c9e4d6de6578cfb6bf | |
parent | e311443b6eafec8ed15ab074b371a9ee3d4d1c08 (diff) | |
download | heat-9a4012ba9d99ca30d9f6fc5a226cbd3baa727bc5.tar.gz |
Use set instead of list of server_deferred_statuses
We only use this list to test for membership, so it's more efficient to
use a set.
Change-Id: I520fd87a18f37da3ae2e61c56fe00063348540b4
-rw-r--r-- | heat/engine/clients/os/nova.py | 4 | ||||
-rw-r--r-- | heat/engine/resources/aws/ec2/instance.py | 2 | ||||
-rw-r--r-- | heat/engine/resources/openstack/nova/server.py | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/heat/engine/clients/os/nova.py b/heat/engine/clients/os/nova.py index d8cd22a6f..b070764fc 100644 --- a/heat/engine/clients/os/nova.py +++ b/heat/engine/clients/os/nova.py @@ -48,7 +48,7 @@ CLIENT_NAME = 'nova' class NovaClientPlugin(microversion_mixin.MicroversionMixin, client_plugin.ClientPlugin): - deferred_server_statuses = ['BUILD', + deferred_server_statuses = {'BUILD', 'HARD_REBOOT', 'PASSWORD', 'REBOOT', @@ -57,7 +57,7 @@ class NovaClientPlugin(microversion_mixin.MicroversionMixin, 'REVERT_RESIZE', 'SHUTOFF', 'SUSPENDED', - 'VERIFY_RESIZE'] + 'VERIFY_RESIZE'} exceptions_module = exceptions diff --git a/heat/engine/resources/aws/ec2/instance.py b/heat/engine/resources/aws/ec2/instance.py index 7311eb995..3cdd4b5d1 100644 --- a/heat/engine/resources/aws/ec2/instance.py +++ b/heat/engine/resources/aws/ec2/instance.py @@ -865,7 +865,7 @@ class Instance(resource.Resource, sh.SchedulerHintsMixin): status = cp.get_status(server) LOG.debug('%(name)s check_suspend_complete status = %(status)s', {'name': self.name, 'status': status}) - if status in list(cp.deferred_server_statuses + ['ACTIVE']): + if status in (cp.deferred_server_statuses | {'ACTIVE'}): return status == 'SUSPENDED' else: exc = exception.ResourceUnknownStatus( diff --git a/heat/engine/resources/openstack/nova/server.py b/heat/engine/resources/openstack/nova/server.py index 9d9b2df2f..c1fc2c2bb 100644 --- a/heat/engine/resources/openstack/nova/server.py +++ b/heat/engine/resources/openstack/nova/server.py @@ -1701,7 +1701,7 @@ class Server(server_base.BaseServer, sh.SchedulerHintsMixin, status = cp.get_status(server) LOG.debug('%(name)s check_suspend_complete status = %(status)s', {'name': self.name, 'status': status}) - if status in list(cp.deferred_server_statuses + ['ACTIVE']): + if status in (cp.deferred_server_statuses | {'ACTIVE'}): return status == 'SUSPENDED' else: exc = exception.ResourceUnknownStatus( |