]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Move _check_resize to nova_utils
authorSteve Baker <sbaker@redhat.com>
Thu, 15 Aug 2013 23:38:19 +0000 (11:38 +1200)
committerSteve Baker <sbaker@redhat.com>
Mon, 19 Aug 2013 02:03:39 +0000 (14:03 +1200)
Refactoring as part of blueprint native-nova-instance.

Change-Id: Ice9a4541316ad04ac04b24ac877e0fc3fbeae9c8

heat/engine/resources/instance.py
heat/engine/resources/nova_utils.py
heat/engine/resources/rackspace/cloud_server.py

index 6946a60de22be23fa73795f2a4b2e5a9ddfb094f..9972a34c07cc20eb1b8b35e5cb7a71e21fba00ce 100644 (file)
@@ -353,24 +353,7 @@ class Instance(resource.Resource):
             flavor_id = nova_utils.get_flavor_id(self.nova(), flavor)
             server = self.nova().servers.get(self.resource_id)
             server.resize(flavor_id)
-            scheduler.TaskRunner(self._check_resize, server, flavor)()
-
-    def _check_resize(self, server, flavor):
-        """
-        Verify that the server is properly resized. If that's the case, confirm
-        the resize, if not raise an error.
-        """
-        yield
-        server.get()
-        while server.status == 'RESIZE':
-            yield
-            server.get()
-        if server.status == 'VERIFY_RESIZE':
-            server.confirm_resize()
-        else:
-            raise exception.Error(
-                _("Resizing to '%(flavor)s' failed, status '%(status)s'") %
-                dict(flavor=flavor, status=server.status))
+            scheduler.TaskRunner(nova_utils.check_resize, server, flavor)()
 
     def metadata_update(self, new_metadata=None):
         '''
index 05969ebb1bb6e85254920428efd3aae25778c0a6..daf4d770a7f3f7595704c806ffd10d0d6c0dff72 100644 (file)
@@ -201,3 +201,21 @@ def delete_server(server):
             server.get()
         except clients.novaclient.exceptions.NotFound:
             break
+
+
+def check_resize(server, flavor):
+    """
+    Verify that the server is properly resized. If that's the case, confirm
+    the resize, if not raise an error.
+    """
+    yield
+    server.get()
+    while server.status == 'RESIZE':
+        yield
+        server.get()
+    if server.status == 'VERIFY_RESIZE':
+        server.confirm_resize()
+    else:
+        raise exception.Error(
+            _("Resizing to '%(flavor)s' failed, status '%(status)s'") %
+            dict(flavor=flavor, status=server.status))
index 0faf64d4f20118e3f77b1e4440759ff3f68d94f5..16584919a002225f090c1c41dd74e8b69632e563 100644 (file)
@@ -418,7 +418,7 @@ zypper --non-interactive in cloud-init python-boto python-pip gcc python-devel
         if 'flavor' in prop_diff:
             self.flavor = json_snippet['Properties']['flavor']
             self.server.resize(self.flavor)
-            resize = scheduler.TaskRunner(self._check_resize,
+            resize = scheduler.TaskRunner(nova_utils.check_resize,
                                           self.server,
                                           self.flavor)
             resize(wait_time=1.0)