refactor create or get image process to eliminate reduplicate
[functest.git] / utils / openstack_utils.py
old mode 100644 (file)
new mode 100755 (executable)
index b463bf4..a7bc899
@@ -14,11 +14,15 @@ import subprocess
 import sys
 import time
 
+from cinderclient import client as cinderclient
+import functest.utils.functest_logger as ft_logger
 from glanceclient import client as glanceclient
 from keystoneclient.v2_0 import client as keystoneclient
 from neutronclient.v2_0 import client as neutronclient
 from novaclient import client as novaclient
 
+logger = ft_logger.Logger("openstack_utils").getLogger()
+
 
 # *********************************************
 #   CREDENTIALS
@@ -46,7 +50,7 @@ def get_credentials(service):
     envvars = ('OS_USERNAME', 'OS_PASSWORD', 'OS_AUTH_URL', 'OS_TENANT_NAME')
     for envvar in envvars:
         if os.getenv(envvar) is None:
-            print("'%s' is not exported as an env variable." % envvar)
+            logger.error("'%s' is not exported as an env variable." % envvar)
             exit(-1)
 
     # Unfortunately, each of the OpenStack client will request slightly
@@ -66,6 +70,14 @@ def get_credentials(service):
         "auth_url": os.environ.get("OS_AUTH_URL"),
         tenant: os.environ.get("OS_TENANT_NAME")
     })
+    if os.getenv('OS_ENDPOINT_TYPE') is not None:
+        creds.update({
+            "endpoint_type": os.environ.get("OS_ENDPOINT_TYPE")
+        })
+    if os.getenv('OS_REGION_NAME') is not None:
+        creds.update({
+            "region_name": os.environ.get("OS_REGION_NAME")
+        })
     cacert = os.environ.get("OS_CACERT")
     if cacert is not None:
         # each openstack client uses differnt kwargs for this
@@ -76,8 +88,8 @@ def get_credentials(service):
                       "ca_file": cacert})
         creds.update({"insecure": "True", "https_insecure": "True"})
         if not os.path.isfile(cacert):
-            print ("WARNING: The 'OS_CACERT' environment variable is " +
-                   "set to %s but the file does not exist." % cacert)
+            logger.info("WARNING: The 'OS_CACERT' environment variable is "
+                        "set to %s but the file does not exist." % cacert)
     return creds
 
 
@@ -90,6 +102,24 @@ def source_credentials(rc_file):
     return env
 
 
+def get_credentials_for_rally():
+    creds = get_credentials("keystone")
+    admin_keys = ['username', 'tenant_name', 'password']
+    endpoint_types = [('internalURL', 'internal'),
+                      ('publicURL', 'public'), ('adminURL', 'admin')]
+    if 'endpoint_type' in creds.keys():
+        for k, v in endpoint_types:
+            if creds['endpoint_type'] == k:
+                creds['endpoint_type'] = v
+    rally_conf = {"type": "ExistingCloud", "admin": {}}
+    for key in creds:
+        if key in admin_keys:
+            rally_conf['admin'][key] = creds[key]
+        else:
+            rally_conf[key] = creds[key]
+    return rally_conf
+
+
 # *********************************************
 #   CLIENTS
 # *********************************************
@@ -103,6 +133,14 @@ def get_nova_client():
     return novaclient.Client('2', **creds_nova)
 
 
+def get_cinder_client():
+    creds_cinder = get_credentials("cinder")
+    creds_cinder.update({
+        "service_type": "volume"
+    })
+    return cinderclient.Client('2', **creds_cinder)
+
+
 def get_neutron_client():
     creds_neutron = get_credentials("neutron")
     return neutronclient.Client(**creds_neutron)
@@ -110,8 +148,12 @@ def get_neutron_client():
 
 def get_glance_client():
     keystone_client = get_keystone_client()
+    glance_endpoint_type = 'publicURL'
+    os_endpoint_type = os.getenv('OS_ENDPOINT_TYPE')
+    if os_endpoint_type is not None:
+        glance_endpoint_type = os_endpoint_type
     glance_endpoint = keystone_client.service_catalog.url_for(
-        service_type='image', endpoint_type='publicURL')
+        service_type='image', endpoint_type=glance_endpoint_type)
     return glanceclient.Client(1, glance_endpoint,
                                token=keystone_client.auth_token)
 
@@ -125,7 +167,7 @@ def get_instances(nova_client):
         instances = nova_client.servers.list(search_opts={'all_tenants': 1})
         return instances
     except Exception, e:
-        print "Error [get_instances(nova_client)]:", e
+        logger.error("Error [get_instances(nova_client)]: %s" % e)
         return None
 
 
@@ -133,9 +175,8 @@ def get_instance_status(nova_client, instance):
     try:
         instance = nova_client.servers.get(instance.id)
         return instance.status
-    except:
-        # print ("Error [get_instance_status(nova_client, '%s')]:" %
-        #        str(instance)), e
+    except Exception, e:
+        logger.error("Error [get_instance_status(nova_client)]: %s" % e)
         return None
 
 
@@ -144,8 +185,8 @@ def get_instance_by_name(nova_client, instance_name):
         instance = nova_client.servers.find(name=instance_name)
         return instance
     except Exception, e:
-        print ("Error [get_instance_by_name(nova_client, '%s')]:" %
-               instance_name), e
+        logger.error("Error [get_instance_by_name(nova_client, '%s')]: %s"
+                     % (instance_name, e))
         return None
 
 
@@ -174,7 +215,7 @@ def get_floating_ips(nova_client):
         floating_ips = nova_client.floating_ips.list()
         return floating_ips
     except Exception, e:
-        print "Error [get_floating_ips(nova_client)]:", e
+        logger.error("Error [get_floating_ips(nova_client)]: %s" % e)
         return None
 
 
@@ -187,7 +228,7 @@ def get_hypervisors(nova_client):
                 nodes.append(hypervisor.hypervisor_hostname)
         return nodes
     except Exception, e:
-        print "Error [get_hypervisors(nova_client)]:", e
+        logger.error("Error [get_hypervisors(nova_client)]: %s" % e)
         return None
 
 
@@ -195,8 +236,8 @@ def create_flavor(nova_client, flavor_name, ram, disk, vcpus):
     try:
         flavor = nova_client.flavors.create(flavor_name, ram, vcpus, disk)
     except Exception, e:
-        print ("Error [create_flavor(nova_client, '%s', '%s', '%s', "
-               "'%s')]:" % (flavor_name, ram, disk, vcpus)), e
+        logger.error("Error [create_flavor(nova_client, '%s', '%s', '%s', "
+                     "'%s')]: %s" % (flavor_name, ram, disk, vcpus, e))
         return None
     return flavor.id
 
@@ -207,32 +248,40 @@ def create_instance(flavor_name,
                     instance_name="functest-vm",
                     confdrive=True,
                     userdata=None,
-                    av_zone=''):
+                    av_zone='',
+                    fixed_ip=None,
+                    files=None):
     nova_client = get_nova_client()
     try:
         flavor = nova_client.flavors.find(name=flavor_name)
     except:
-        print("Error: Flavor '%s' not found. Available flavors are:" %
-              flavor_name)
-        print(nova_client.flavor.list())
-        return -1
+        flavors = nova_client.flavors.list()
+        logger.error("Error: Flavor '%s' not found. Available flavors are: "
+                     "\n%s" % (flavor_name, flavors))
+        return None
+    if fixed_ip is not None:
+        nics = {"net-id": network_id, "v4-fixed-ip": fixed_ip}
+    else:
+        nics = {"net-id": network_id}
     if userdata is None:
         instance = nova_client.servers.create(
             name=instance_name,
             flavor=flavor,
             image=image_id,
-            nics=[{"net-id": network_id}],
-            availability_zone=av_zone
+            nics=[nics],
+            availability_zone=av_zone,
+            files=files
         )
     else:
         instance = nova_client.servers.create(
             name=instance_name,
             flavor=flavor,
             image=image_id,
-            nics=[{"net-id": network_id}],
+            nics=[nics],
             config_drive=confdrive,
             userdata=userdata,
-            availability_zone=av_zone
+            availability_zone=av_zone,
+            files=files
         )
     return instance
 
@@ -243,7 +292,9 @@ def create_instance_and_wait_for_active(flavor_name,
                                         instance_name="",
                                         config_drive=False,
                                         userdata="",
-                                        av_zone=''):
+                                        av_zone='',
+                                        fixed_ip=None,
+                                        files=None):
     SLEEP = 3
     VM_BOOT_TIMEOUT = 180
     nova_client = get_nova_client()
@@ -253,18 +304,20 @@ def create_instance_and_wait_for_active(flavor_name,
                                instance_name,
                                config_drive,
                                userdata,
-                               av_zone)
-
+                               av_zone=av_zone,
+                               fixed_ip=fixed_ip,
+                               files=files)
     count = VM_BOOT_TIMEOUT / SLEEP
     for n in range(count, -1, -1):
         status = get_instance_status(nova_client, instance)
         if status.lower() == "active":
             return instance
         elif status.lower() == "error":
-            print("The instance %s went to ERROR status." % instance_name)
+            logger.error("The instance %s went to ERROR status."
+                         % instance_name)
             return None
         time.sleep(SLEEP)
-    print("Timeout booting the instance %s." % instance_name)
+    logger.error("Timeout booting the instance %s." % instance_name)
     return None
 
 
@@ -276,7 +329,7 @@ def create_floating_ip(neutron_client):
         fip_addr = ip_json['floatingip']['floating_ip_address']
         fip_id = ip_json['floatingip']['id']
     except Exception, e:
-        print "Error [create_floating_ip(neutron_client)]:", e
+        logger.error("Error [create_floating_ip(neutron_client)]: %s" % e)
         return None
     return {'fip_addr': fip_addr, 'fip_id': fip_id}
 
@@ -286,8 +339,8 @@ def add_floating_ip(nova_client, server_id, floatingip_id):
         nova_client.servers.add_floating_ip(server_id, floatingip_id)
         return True
     except Exception, e:
-        print ("Error [add_floating_ip(nova_client, '%s', '%s')]:" %
-               (server_id, floatingip_id)), e
+        logger.error("Error [add_floating_ip(nova_client, '%s', '%s')]: %s"
+                     % (server_id, floatingip_id, e))
         return False
 
 
@@ -296,7 +349,8 @@ def delete_instance(nova_client, instance_id):
         nova_client.servers.force_delete(instance_id)
         return True
     except Exception, e:
-        print "Error [delete_instance(nova_client, '%s')]:" % instance_id, e
+        logger.error("Error [delete_instance(nova_client, '%s')]: %s"
+                     % (instance_id, e))
         return False
 
 
@@ -305,8 +359,8 @@ def delete_floating_ip(nova_client, floatingip_id):
         nova_client.floating_ips.delete(floatingip_id)
         return True
     except Exception, e:
-        print ("Error [delete_floating_ip(nova_client, '%s')]:" %
-               floatingip_id), e
+        logger.error("Error [delete_floating_ip(nova_client, '%s')]:"
+                     % (floatingip_id, e))
         return False
 
 
@@ -382,14 +436,14 @@ def get_external_net(neutron_client):
     for network in neutron_client.list_networks()['networks']:
         if network['router:external']:
             return network['name']
-    return False
+    return None
 
 
 def get_external_net_id(neutron_client):
     for network in neutron_client.list_networks()['networks']:
         if network['router:external']:
             return network['id']
-    return False
+    return None
 
 
 def check_neutron_net(neutron_client, net_name):
@@ -408,8 +462,9 @@ def create_neutron_net(neutron_client, name):
         network_dict = network['network']
         return network_dict['id']
     except Exception, e:
-        print "Error [create_neutron_net(neutron_client, '%s')]:" % name, e
-        return False
+        logger.error("Error [create_neutron_net(neutron_client, '%s')]: %s"
+                     % (name, e))
+        return None
 
 
 def create_neutron_subnet(neutron_client, name, cidr, net_id):
@@ -419,9 +474,9 @@ def create_neutron_subnet(neutron_client, name, cidr, net_id):
         subnet = neutron_client.create_subnet(body=json_body)
         return subnet['subnets'][0]['id']
     except Exception, e:
-        print ("Error [create_neutron_subnet(neutron_client, '%s', '%s', "
-               "'%s')]:" % (name, cidr, net_id)), e
-        return False
+        logger.error("Error [create_neutron_subnet(neutron_client, '%s', "
+                     "'%s', '%s')]: %s" % (name, cidr, net_id, e))
+        return None
 
 
 def create_neutron_router(neutron_client, name):
@@ -430,8 +485,9 @@ def create_neutron_router(neutron_client, name):
         router = neutron_client.create_router(json_body)
         return router['router']['id']
     except Exception, e:
-        print "Error [create_neutron_router(neutron_client, '%s')]:" % name, e
-        return False
+        logger.error("Error [create_neutron_router(neutron_client, '%s')]: %s"
+                     % (name, e))
+        return None
 
 
 def create_neutron_port(neutron_client, name, network_id, ip):
@@ -445,9 +501,9 @@ def create_neutron_port(neutron_client, name, network_id, ip):
         port = neutron_client.create_port(body=json_body)
         return port['port']['id']
     except Exception, e:
-        print ("Error [create_neutron_port(neutron_client, '%s', '%s', "
-               "'%s')]:" % (name, network_id, ip)), e
-        return False
+        logger.error("Error [create_neutron_port(neutron_client, '%s', '%s', "
+                     "'%s')]: %s" % (name, network_id, ip, e))
+        return None
 
 
 def update_neutron_net(neutron_client, network_id, shared=False):
@@ -456,8 +512,8 @@ def update_neutron_net(neutron_client, network_id, shared=False):
         neutron_client.update_network(network_id, body=json_body)
         return True
     except Exception, e:
-        print ("Error [update_neutron_net(neutron_client, '%s', '%s')]:" %
-               (network_id, str(shared))), e
+        logger.error("Error [update_neutron_net(neutron_client, '%s', '%s')]: "
+                     "%s" % (network_id, str(shared), e))
         return False
 
 
@@ -470,9 +526,9 @@ def update_neutron_port(neutron_client, port_id, device_owner):
                                           body=json_body)
         return port['port']['id']
     except Exception, e:
-        print ("Error [update_neutron_port(neutron_client, '%s', '%s')]:" %
-               (port_id, device_owner)), e
-        return False
+        logger.error("Error [update_neutron_port(neutron_client, '%s', '%s')]:"
+                     " %s" % (port_id, device_owner, e))
+        return None
 
 
 def add_interface_router(neutron_client, router_id, subnet_id):
@@ -481,8 +537,8 @@ def add_interface_router(neutron_client, router_id, subnet_id):
         neutron_client.add_interface_router(router=router_id, body=json_body)
         return True
     except Exception, e:
-        print ("Error [add_interface_router(neutron_client, '%s', '%s')]:" %
-               (router_id, subnet_id)), e
+        logger.error("Error [add_interface_router(neutron_client, '%s', "
+                     "'%s')]: %s" % (router_id, subnet_id, e))
         return False
 
 
@@ -493,8 +549,8 @@ def add_gateway_router(neutron_client, router_id):
         neutron_client.add_gateway_router(router_id, router_dict)
         return True
     except Exception, e:
-        print ("Error [add_gateway_router(neutron_client, '%s')]:" %
-               router_id), e
+        logger.error("Error [add_gateway_router(neutron_client, '%s')]: %s"
+                     % (router_id, e))
         return False
 
 
@@ -503,8 +559,8 @@ def delete_neutron_net(neutron_client, network_id):
         neutron_client.delete_network(network_id)
         return True
     except Exception, e:
-        print ("Error [delete_neutron_net(neutron_client, '%s')]:" %
-               network_id), e
+        logger.error("Error [delete_neutron_net(neutron_client, '%s')]: %s"
+                     % (network_id, e))
         return False
 
 
@@ -513,8 +569,8 @@ def delete_neutron_subnet(neutron_client, subnet_id):
         neutron_client.delete_subnet(subnet_id)
         return True
     except Exception, e:
-        print ("Error [delete_neutron_subnet(neutron_client, '%s')]:" %
-               subnet_id), e
+        logger.error("Error [delete_neutron_subnet(neutron_client, '%s')]: %s"
+                     % (subnet_id, e))
         return False
 
 
@@ -523,8 +579,8 @@ def delete_neutron_router(neutron_client, router_id):
         neutron_client.delete_router(router=router_id)
         return True
     except Exception, e:
-        print ("Error [delete_neutron_router(neutron_client, '%s')]:" %
-               router_id), e
+        logger.error("Error [delete_neutron_router(neutron_client, '%s')]: %s"
+                     % (router_id, e))
         return False
 
 
@@ -533,7 +589,8 @@ def delete_neutron_port(neutron_client, port_id):
         neutron_client.delete_port(port_id)
         return True
     except Exception, e:
-        print "Error [delete_neutron_port(neutron_client, '%s')]:" % port_id, e
+        logger.error("Error [delete_neutron_port(neutron_client, '%s')]: %s"
+                     % (port_id, e))
         return False
 
 
@@ -544,8 +601,8 @@ def remove_interface_router(neutron_client, router_id, subnet_id):
                                                body=json_body)
         return True
     except Exception, e:
-        print ("Error [remove_interface_router(neutron_client, '%s', '%s')]:" %
-               (router_id, subnet_id)), e
+        logger.error("Error [remove_interface_router(neutron_client, '%s', "
+                     "'%s')]: %s" % (router_id, subnet_id, e))
         return False
 
 
@@ -554,13 +611,12 @@ def remove_gateway_router(neutron_client, router_id):
         neutron_client.remove_gateway_router(router_id)
         return True
     except Exception, e:
-        print ("Error [remove_gateway_router(neutron_client, '%s')]:" %
-               router_id), e
+        logger.error("Error [remove_gateway_router(neutron_client, '%s')]: %s"
+                     % (router_id, e))
         return False
 
 
-def create_network_full(logger,
-                        neutron_client,
+def create_network_full(neutron_client,
                         net_name,
                         subnet_name,
                         router_name,
@@ -586,26 +642,26 @@ def create_network_full(logger,
         subnet_id = create_neutron_subnet(neutron_client, subnet_name,
                                           cidr, network_id)
         if not subnet_id:
-            return False
+            return None
 
         logger.debug("Subnet '%s' created successfully" % subnet_id)
         logger.debug('Creating Router...')
         router_id = create_neutron_router(neutron_client, router_name)
 
         if not router_id:
-            return False
+            return None
 
         logger.debug("Router '%s' created successfully" % router_id)
         logger.debug('Adding router to subnet...')
 
         if not add_interface_router(neutron_client, router_id, subnet_id):
-            return False
+            return None
 
         logger.debug("Interface added successfully.")
 
         logger.debug('Adding gateway to router...')
         if not add_gateway_router(neutron_client, router_id):
-            return False
+            return None
 
         logger.debug("Gateway added successfully.")
 
@@ -631,6 +687,10 @@ def update_bgpvpn(neutron_client, bgpvpn_id, **kwargs):
     json_body = {"bgpvpn": kwargs}
     return neutron_client.update_bgpvpn(bgpvpn_id, json_body)
 
+
+def delete_bgpvpn(neutron_client, bgpvpn_id):
+    return neutron_client.delete_bgpvpn(bgpvpn_id)
+
 # *********************************************
 #   SEC GROUPS
 # *********************************************
@@ -642,7 +702,7 @@ def get_security_groups(neutron_client):
             'security_groups']
         return security_groups
     except Exception, e:
-        print "Error [get_security_groups(neutron_client)]:", e
+        logger.error("Error [get_security_groups(neutron_client)]: %s" % e)
         return None
 
 
@@ -663,9 +723,9 @@ def create_security_group(neutron_client, sg_name, sg_description):
         secgroup = neutron_client.create_security_group(json_body)
         return secgroup['security_group']
     except Exception, e:
-        print ("Error [create_security_group(neutron_client, '%s', '%s')]:" %
-               (sg_name, sg_description)), e
-        return False
+        logger.error("Error [create_security_group(neutron_client, '%s', "
+                     "'%s')]: %s" % (sg_name, sg_description, e))
+        return None
 
 
 def create_secgroup_rule(neutron_client, sg_id, direction, protocol,
@@ -681,24 +741,29 @@ def create_secgroup_rule(neutron_client, sg_id, direction, protocol,
                                              'port_range_max': port_range_max,
                                              'protocol': protocol}}
     else:
-        print ("Error [create_secgroup_rule(neutron_client, '%s', '%s', "
-               "'%s', '%s', '%s', '%s')]:" % (neutron_client, sg_id, direction,
-                                              port_range_min, port_range_max,
-                                              protocol),
-               " Invalid values for port_range_min, port_range_max")
+        logger.error("Error [create_secgroup_rule(neutron_client, '%s', '%s', "
+                     "'%s', '%s', '%s', '%s')]:" % (neutron_client,
+                                                    sg_id, direction,
+                                                    port_range_min,
+                                                    port_range_max,
+                                                    protocol),
+                     " Invalid values for port_range_min, port_range_max")
         return False
     try:
         neutron_client.create_security_group_rule(json_body)
         return True
     except Exception, e:
-        print ("Error [create_secgroup_rule(neutron_client, '%s', '%s', "
-               "'%s', '%s', '%s', '%s')]:" % (neutron_client, sg_id, direction,
-                                              port_range_min, port_range_max,
-                                              protocol)), e
+        logger.error("Error [create_secgroup_rule(neutron_client, '%s', '%s', "
+                     "'%s', '%s', '%s', '%s')]: %s" % (neutron_client,
+                                                       sg_id,
+                                                       direction,
+                                                       port_range_min,
+                                                       port_range_max,
+                                                       protocol, e))
         return False
 
 
-def create_security_group_full(logger, neutron_client,
+def create_security_group_full(neutron_client,
                                sg_name, sg_description):
     sg_id = get_security_group_id(neutron_client, sg_name)
     if sg_id != '':
@@ -710,7 +775,7 @@ def create_security_group_full(logger, neutron_client,
                                          sg_description)
         if not SECGROUP:
             logger.error("Failed to create the security group...")
-            return False
+            return None
 
         sg_id = SECGROUP['id']
 
@@ -722,19 +787,19 @@ def create_security_group_full(logger, neutron_client,
         if not create_secgroup_rule(neutron_client, sg_id,
                                     'ingress', 'icmp'):
             logger.error("Failed to create the security group rule...")
-            return False
+            return None
 
         logger.debug("Adding SSH rules in security group '%s'..."
                      % sg_name)
         if not create_secgroup_rule(
                 neutron_client, sg_id, 'ingress', 'tcp', '22', '22'):
             logger.error("Failed to create the security group rule...")
-            return False
+            return None
 
         if not create_secgroup_rule(
                 neutron_client, sg_id, 'egress', 'tcp', '22', '22'):
             logger.error("Failed to create the security group rule...")
-            return False
+            return None
     return sg_id
 
 
@@ -743,8 +808,8 @@ def add_secgroup_to_instance(nova_client, instance_id, secgroup_id):
         nova_client.servers.add_security_group(instance_id, secgroup_id)
         return True
     except Exception, e:
-        print ("Error [add_secgroup_to_instance(nova_client, '%s', '%s')]: " %
-               (instance_id, secgroup_id)), e
+        logger.error("Error [add_secgroup_to_instance(nova_client, '%s', "
+                     "'%s')]: %s" % (instance_id, secgroup_id, e))
         return False
 
 
@@ -759,8 +824,8 @@ def update_sg_quota(neutron_client, tenant_id, sg_quota, sg_rule_quota):
                                     body=json_body)
         return True
     except Exception, e:
-        print ("Error [update_sg_quota(neutron_client, '%s', '%s', "
-               "'%s')]:" % (tenant_id, sg_quota, sg_rule_quota)), e
+        logger.error("Error [update_sg_quota(neutron_client, '%s', '%s', "
+                     "'%s')]: %s" % (tenant_id, sg_quota, sg_rule_quota, e))
         return False
 
 
@@ -769,8 +834,8 @@ def delete_security_group(neutron_client, secgroup_id):
         neutron_client.delete_security_group(secgroup_id)
         return True
     except Exception, e:
-        print ("Error [delete_security_group(neutron_client, '%s')]:" %
-               secgroup_id), e
+        logger.error("Error [delete_security_group(neutron_client, '%s')]: %s"
+                     % (secgroup_id, e))
         return False
 
 
@@ -782,7 +847,7 @@ def get_images(nova_client):
         images = nova_client.images.list()
         return images
     except Exception, e:
-        print "Error [get_images]:", e
+        logger.error("Error [get_images]: %s" % e)
         return None
 
 
@@ -797,10 +862,10 @@ def get_image_id(glance_client, image_name):
 
 
 def create_glance_image(glance_client, image_name, file_path, disk="qcow2",
-                        container="bare", public=True, logger=None):
+                        container="bare", public=True):
     if not os.path.isfile(file_path):
-        print "Error: file " + file_path + " does not exist."
-        return False
+        logger.error("Error: file %s does not exist." % file_path)
+        return None
     try:
         image_id = get_image_id(glance_client, image_name)
         if image_id != '':
@@ -819,9 +884,29 @@ def create_glance_image(glance_client, image_name, file_path, disk="qcow2",
             image_id = image.id
         return image_id
     except Exception, e:
-        print ("Error [create_glance_image(glance_client, '%s', '%s', "
-               "'%s')]:" % (image_name, file_path, str(public))), e
-        return False
+        logger.error("Error [create_glance_image(glance_client, '%s', '%s', "
+                     "'%s')]: %s" % (image_name, file_path, str(public), e))
+        return None
+
+
+def get_or_create_image(name, path, format):
+    image_exists = False
+    glance_client = get_glance_client()
+
+    image_id = get_image_id(glance_client, name)
+    if image_id != '':
+        logger.info("Using existing image '%s'..." % name)
+        image_exists = True
+    else:
+        logger.info("Creating image '%s' from '%s'..." % (name, path))
+        image_id = create_glance_image(glance_client, name, path, format)
+        if not image_id:
+            logger.error("Failed to create a Glance image...")
+        else:
+            logger.debug("Image '%s' with ID=%s created successfully."
+                         % (name, image_id))
+
+    return image_exists, image_id
 
 
 def delete_glance_image(nova_client, image_id):
@@ -829,7 +914,8 @@ def delete_glance_image(nova_client, image_id):
         nova_client.images.delete(image_id)
         return True
     except Exception, e:
-        print ("Error [delete_glance_image(nova_client, '%s')]:" % image_id), e
+        logger.error("Error [delete_glance_image(nova_client, '%s')]: %s"
+                     % (image_id, e))
         return False
 
 
@@ -841,7 +927,7 @@ def get_volumes(cinder_client):
         volumes = cinder_client.volumes.list(search_opts={'all_tenants': 1})
         return volumes
     except Exception, e:
-        print "Error [get_volumes(cinder_client)]:", e
+        logger.error("Error [get_volumes(cinder_client)]: %s" % e)
         return None
 
 
@@ -854,7 +940,7 @@ def list_volume_types(cinder_client, public=True, private=True):
             volume_types = [vt for vt in volume_types if vt.is_public]
         return volume_types
     except Exception, e:
-        print "Error [list_volume_types(cinder_client)]:", e
+        logger.error("Error [list_volume_types(cinder_client)]: %s" % e)
         return None
 
 
@@ -863,7 +949,8 @@ def create_volume_type(cinder_client, name):
         volume_type = cinder_client.volume_types.create(name)
         return volume_type
     except Exception, e:
-        print "Error [create_volume_type(cinder_client, '%s')]:" % name, e
+        logger.error("Error [create_volume_type(cinder_client, '%s')]: %s"
+                     % (name, e))
         return None
 
 
@@ -877,9 +964,9 @@ def update_cinder_quota(cinder_client, tenant_id, vols_quota,
         cinder_client.quotas.update(tenant_id, **quotas_values)
         return True
     except Exception, e:
-        print ("Error [update_cinder_quota(cinder_client, '%s', '%s', '%s'"
-               "'%s')]:" % (tenant_id, vols_quota,
-                            snapshots_quota, gigabytes_quota)), e
+        logger.error("Error [update_cinder_quota(cinder_client, '%s', '%s', "
+                     "'%s' '%s')]: %s" % (tenant_id, vols_quota,
+                                          snapshots_quota, gigabytes_quota, e))
         return False
 
 
@@ -889,14 +976,14 @@ def delete_volume(cinder_client, volume_id, forced=False):
             try:
                 cinder_client.volumes.detach(volume_id)
             except:
-                print "Error:", sys.exc_info()[0]
+                logger.error(sys.exc_info()[0])
             cinder_client.volumes.force_delete(volume_id)
         else:
             cinder_client.volumes.delete(volume_id)
         return True
     except Exception, e:
-        print ("Error [delete_volume(cinder_client, '%s', '%s')]:" %
-               (volume_id, str(forced))), e
+        logger.error("Error [delete_volume(cinder_client, '%s', '%s')]: %s"
+                     % (volume_id, str(forced), e))
         return False
 
 
@@ -905,8 +992,8 @@ def delete_volume_type(cinder_client, volume_type):
         cinder_client.volume_types.delete(volume_type)
         return True
     except Exception, e:
-        print ("Error [delete_volume_type(cinder_client, '%s')]:" %
-               volume_type), e
+        logger.error("Error [delete_volume_type(cinder_client, '%s')]: %s"
+                     % (volume_type, e))
         return False
 
 
@@ -918,7 +1005,7 @@ def get_tenants(keystone_client):
         tenants = keystone_client.tenants.list()
         return tenants
     except Exception, e:
-        print "Error [get_tenants(keystone_client)]:", e
+        logger.error("Error [get_tenants(keystone_client)]: %s" % e)
         return None
 
 
@@ -927,7 +1014,7 @@ def get_users(keystone_client):
         users = keystone_client.users.list()
         return users
     except Exception, e:
-        print "Error [get_users(keystone_client)]:", e
+        logger.error("Error [get_users(keystone_client)]: %s" % e)
         return None
 
 
@@ -968,9 +1055,9 @@ def create_tenant(keystone_client, tenant_name, tenant_description):
                                                 enabled=True)
         return tenant.id
     except Exception, e:
-        print ("Error [create_tenant(cinder_client, '%s', '%s')]:" %
-               (tenant_name, tenant_description)), e
-        return False
+        logger.error("Error [create_tenant(cinder_client, '%s', '%s')]: %s"
+                     % (tenant_name, tenant_description, e))
+        return None
 
 
 def create_user(keystone_client, user_name, user_password,
@@ -981,10 +1068,10 @@ def create_user(keystone_client, user_name, user_password,
                                             enabled=True)
         return user.id
     except Exception, e:
-        print ("Error [create_user(keystone_client, '%s', '%s', '%s'"
-               "'%s')]:" % (user_name, user_password, user_email, tenant_id),
-               e)
-        return False
+        logger.error("Error [create_user(keystone_client, '%s', '%s', '%s'"
+                     "'%s')]: %s" % (user_name, user_password,
+                                     user_email, tenant_id, e))
+        return None
 
 
 def add_role_user(keystone_client, user_id, role_id, tenant_id):
@@ -992,8 +1079,8 @@ def add_role_user(keystone_client, user_id, role_id, tenant_id):
         keystone_client.roles.add_user_role(user_id, role_id, tenant_id)
         return True
     except Exception, e:
-        print ("Error [add_role_user(keystone_client, '%s', '%s'"
-               "'%s')]:" % (user_id, role_id, tenant_id)), e
+        logger.error("Error [add_role_user(keystone_client, '%s', '%s'"
+                     "'%s')]: %s " % (user_id, role_id, tenant_id, e))
         return False
 
 
@@ -1002,7 +1089,8 @@ def delete_tenant(keystone_client, tenant_id):
         keystone_client.tenants.delete(tenant_id)
         return True
     except Exception, e:
-        print "Error [delete_tenant(keystone_client, '%s')]:" % tenant_id, e
+        logger.error("Error [delete_tenant(keystone_client, '%s')]: %s"
+                     % (tenant_id, e))
         return False
 
 
@@ -1011,5 +1099,6 @@ def delete_user(keystone_client, user_id):
         keystone_client.users.delete(user_id)
         return True
     except Exception, e:
-        print "Error [delete_user(keystone_client, '%s')]:" % user_id, e
+        logger.error("Error [delete_user(keystone_client, '%s')]: %s"
+                     % (user_id, e))
         return False