openstack/zun
Code Issues Proposed changes

Use uuidutils instead of uuid.uuid4()

Browse Source 
Openstack common has a wrapper for generating uuids.We should
use that function when generating uuids for consistency.
Replaces uuid.uuid4 with uuidutils.generate_uuid()

Change-Id: I7687b9ce8ef76ea9e670dca10006476735fca475
This commit is contained in:
bhavani 2017-03-11 15:09:57 +05:30 committed by bhavani
parent b9e21fdd77
commit 86eb9e8dce
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
nova/virt/zun/driver.py
View File

@ -22,7 +22,6 @@ import os
import shutil
import socket
import time
import uuid
from docker import errors
import eventlet
@ -31,6 +30,7 @@ from oslo_log import log
from oslo_utils import fileutils
from oslo_utils import importutils
from oslo_utils import units
from oslo_utils import uuidutils
from oslo_utils import versionutils
from nova.compute import flavors
@ -393,7 +393,8 @@ class DockerDriver(driver.ComputeDriver):
fileutils.ensure_tree(snapshot_directory)
with utils.tempdir(dir=snapshot_directory) as tmpdir:
try:
out_path = os.path.join(tmpdir, uuid.uuid4().hex)
out_path = os.path.join(tmpdir,
uuidutils.generate_uuid(dashed=False))
LOG.debug('Fetching image with id %s from glance',
image_meta.id)

3
zun/common/short_id.py
View File

@ -16,6 +16,7 @@ The IDs each comprise 12 (lower-case) alphanumeric characters.
"""
import base64
from oslo_utils import uuidutils
import uuid
import six
@ -60,4 +61,4 @@ def get_id(source_uuid):
def generate_id():
"""Generate a short (12 character), random id."""
return get_id(uuid.uuid4())
return uuidutils.generate_uuid()