Change to use Glance V1 API to upload image

Change-Id: I9c18eb5ea9a3bbca76079ffffe09ce7d434f69bc

vmtp/compute.py

@@ -16,9 +16,9 @@
 '''Module for Openstack compute operations'''
 
 import os
-import subprocess
 import time
 
+import glanceclient.exc as glance_exception
 import novaclient
 import novaclient.exceptions as exceptions
 
@@ -35,75 +35,38 @@ class Compute(object):
         except novaclient.exceptions.NotFound:
             return None
 
-    def upload_image_via_url(self, creds, glance_client,
+    def upload_image_via_url(self, glance_client, final_image_name,
-                             final_image_name, image_url, retry_count=60):
+                             image_url, retry_count=60):
         '''
         Directly uploads image to Nova via URL if image is not present
         '''
+        retry = 0
+        try:
+            # Upload the image
+            img = glance_client.images.create(
+                name=str(final_image_name), disk_format="qcow2", container_format="bare",
+                is_public=True, copy_from=image_url)
 
-        # Here is the deal:
+            # Check for the image in glance
-        # Idealy, we should better to use the python library glanceclient to perform the
+            while img.status in ['queued', 'saving'] and retry < retry_count:
-        # image uploades. However, due to a limitation of the v2.0 API right now, it is
+                img = glance_client.images.find(name=img.name)
-        # impossible to tell Glance to download the image from a URL directly.
+                retry = retry + 1
-        #
+                if self.config.debug:
-        # There are two steps to create the image:
+                    print "Image not yet active, retrying %s of %s..." \
-        # (1) Store the binary image data into Glance;
+                        % (retry, retry_count)
-        # (2) Store the metadata about the image into Glance;
+                time.sleep(2)
-        # PS: The order does not matter.
+            if img.status != 'active':
-        #
+                raise Exception
-        # The REST API allows to do two steps in one if a Location header is provided with
+        except glance_exception.HTTPForbidden:
-        # the POST request. (REF: http://developer.openstack.org/api-ref-image-v2.html)
+            print "Cannot upload image without admin access. Please make sure the "\
-        #
+                  "image is uploaded and is either public or owned by you."
-        # However the python API doesn't support a customized header in POST request.
+            return False
-        # So we have to do two steps in two calls.
+        except Exception:
-        #
+            print "Failed while uploading the image, please make sure the cloud "\
-        # The good thing is: the API does support (2) perfectly, but for (1) it is only
+                  "under test has the access to URL: %s." % image_url
-        # accepting the data from local, not remote URL. So... Ur... Let's keep the CLI
+            return False
-        # version as the workaround for now.
 
-        # # upload in glance
+        return True
-        # image = glance_client.images.create(
-        #     name=str(final_image_name), disk_format="qcow2", container_format="bare",
-        #     Location=image_url)
-        # glance_client.images.add_location(image.id, image_url, image)
-
-        # sys.exit(0)
-        # for retry_attempt in range(retry_count):
-        #     if image.status == "active":
-        #         print 'Image: %s successfully uploaded to Nova' % (final_image_name)
-        #         return 1
-        #     # Sleep between retries
-        #     if self.config.debug:
-        #         print "Image is not yet active, retrying %s of %s... [%s]" \
-        #             % ((retry_attempt + 1), retry_count, image.status)
-        #     time.sleep(5)
-
-        # upload in glance
-        cred = "--os-username %s --os-password %s --os-tenant-name %s --os-auth-url %s" %\
-            (creds['username'], creds['password'], creds['tenant_name'], creds['auth_url'])
-        glance_cmd = "glance %s image-create --name=\"%s\" --disk-format=qcow2 "\
-            "--container-format=bare --is-public True --copy-from %s" %\
-            (cred, str(final_image_name), image_url)
-        if self.config.debug:
-            print "Will update image to glance via CLI: %s" % (glance_cmd)
-        subprocess.check_output(glance_cmd, shell=True)
-
-        # check for the image in glance
-        glance_check_cmd = "glance %s image-list --name \"%s\"" %\
-            (cred, str(final_image_name))
-        for retry_attempt in range(retry_count):
-            result = subprocess.check_output(glance_check_cmd, shell=True)
-            if "active" in result:
-                print 'Image: %s successfully uploaded to Nova' % (final_image_name)
-                return 1
-            # Sleep between retries
-            if self.config.debug:
-                print "Image not yet active, retrying %s of %s..." \
-                    % ((retry_attempt + 1), retry_count)
-            time.sleep(2)
-
-        print 'ERROR: Cannot upload image %s from URL: %s' % (final_image_name, image_url)
-        return 0
 
     # Remove keypair name from openstack if exists
     def remove_public_key(self, name):

vmtp/vmtp.py

@@ -30,7 +30,7 @@ import compute
 from config import config_load
 from config import config_loads
 import credentials
-from glanceclient.v2 import client as glanceclient
+from glanceclient.v1 import client as glanceclient
 import iperf_tool
 from keystoneclient.v2_0 import client as keystoneclient
 import network
@@ -207,7 +207,7 @@ class VmtpTest(object):
             self.image_instance = self.comp.find_image(self.config.image_name)
             if self.image_instance is None:
                 if self.config.vm_image_url != "":
-                    print '%s: image for VM not found, uploading it ...' \
+                    print '%s: image for VM not found, trying to upload it ...' \
                         % (self.config.image_name)
                     keystone = keystoneclient.Client(**creds)
                     glance_endpoint = keystone.service_catalog.url_for(
@@ -215,7 +215,7 @@ class VmtpTest(object):
                     glance_client = glanceclient.Client(
                         glance_endpoint, token=keystone.auth_token)
                     self.comp.upload_image_via_url(
-                        creds, glance_client,
+                        glance_client,
                         self.config.image_name,
                         self.config.vm_image_url)
                     self.image_instance = self.comp.find_image(self.config.image_name)