]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove the use of common.uuidutils.gen_uuid
authorJohn Griffith <john.griffith@solidfire.com>
Wed, 13 Nov 2013 22:29:32 +0000 (15:29 -0700)
committerJohn Griffith <john.griffith@solidfire.com>
Wed, 13 Nov 2013 22:30:28 +0000 (15:30 -0700)
There's a movement to remove gen_uuid from the
openstack common.  This change removes it's use
from Cinder and just puts the call inline.

Change-Id: I07329bab9553e15e498ddbda3848d50b8f48fd53

cinder/taskflow/patterns/base.py
cinder/taskflow/utils.py
cinder/tests/keymgr/mock_key_mgr.py

index 7f47debd914d930099d7d84065556f4010a2445c..20ab9d1ef99a5269d898307e02dc711e9a47c691 100644 (file)
 
 import abc
 import threading
+import uuid as uuidlib
 
 import six
 
-from cinder.openstack.common import uuidutils
 
 from cinder.taskflow import decorators
 from cinder.taskflow import exceptions as exc
@@ -94,7 +94,7 @@ class Flow(object):
         if uuid:
             self._id = str(uuid)
         else:
-            self._id = uuidutils.generate_uuid()
+            self._id = str(uuidlib.uuid4())
 
     @property
     def name(self):
index 81d53d8d036b8b603a0773d0afc503184bd910e8..44a526c6df6fe1c9813ec17dc02c961c4bd0ba8f 100644 (file)
@@ -25,8 +25,8 @@ import sys
 import threading
 import time
 import types
+import uuid as uuidlib
 
-from cinder.openstack.common import uuidutils
 
 from cinder.taskflow import decorators
 
@@ -202,7 +202,7 @@ class Runner(object):
         self.runs_before = []
         self.result = None
         if not uuid:
-            self._id = uuidutils.generate_uuid()
+            self._id = str(uuidlib.uuid4())
         else:
             self._id = str(uuid)
 
index ac8fd7d53999f49d678778a9cffa0fda62c35b65..beeb376c267bfd56c8057000723593726ad0edb7 100644 (file)
@@ -28,11 +28,11 @@ this class.
 """
 
 import array
+import uuid
 
 from cinder import exception
 from cinder.keymgr import key
 from cinder.keymgr import key_mgr
-from cinder.openstack.common import uuidutils
 from cinder import utils
 
 
@@ -77,9 +77,9 @@ class MockKeyManager(key_mgr.KeyManager):
         return self.store_key(ctxt, key)
 
     def _generate_key_id(self):
-        key_id = uuidutils.generate_uuid()
+        key_id = str(uuid.uuid4())
         while key_id in self.keys:
-            key_id = uuidutils.generate_uuid()
+            key_id = str(uuid.uuid4())
 
         return key_id