cloud-certification-team team mailing list archive

[Jeff Marcom <jeff.marcom@xxxxxxxxxxxxx>]

Jeff Marcom has proposed merging lp:~jeffmarcom/cloud-testing/block_storage_create_exception_catch into lp:cloud-testing/cloudbox.

Requested reviews:
  Cloud Certification (cloud-certification-team)

For more details, see:
https://code.launchpad.net/~jeffmarcom/cloud-testing/block_storage_create_exception_catch/+merge/191643

Trivial fix to catch exception for creating block storage via cinder/nova-volume api.
-- 
https://code.launchpad.net/~jeffmarcom/cloud-testing/block_storage_create_exception_catch/+merge/191643
Your team Cloud Certification is requested to review the proposed merge of lp:~jeffmarcom/cloud-testing/block_storage_create_exception_catch into lp:cloud-testing/cloudbox.

=== modified file 'scripts/storage_api_test'
--- scripts/storage_api_test	2013-07-17 16:48:42 +0000
+++ scripts/storage_api_test	2013-10-17 14:01:39 +0000
@@ -62,7 +62,13 @@
     if args.create:
         volume_info = test.create()
         logging.debug(volume_info)
-        id = volume_info["volume"]["id"]
+
+        try:
+            id = volume_info["volume"]["id"]
+        except TypeError as creation_exception:
+            print("Failed creating volume, no volume id found")
+            print(creation_exception)
+
         for i in range(0, VOLUME_POLLING_REPEAT):
             time.sleep(VOLUME_POLLING_INTERVAL)
             print(test.list_volumes())