← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1253497] Re: Replace uuidutils.generate_uuid() with str(uuid.uuid4())

 

** Also affects: billingstack
   Importance: Undecided
       Status: New

** Also affects: designate
   Importance: Undecided
       Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1253497

Title:
  Replace uuidutils.generate_uuid() with str(uuid.uuid4())

Status in Project Barbican:
  Confirmed
Status in BillingStack:
  New
Status in Cinder:
  Confirmed
Status in Climate:
  In Progress
Status in Designate:
  New
Status in OpenStack Image Registry and Delivery Service (Glance):
  Fix Released
Status in Orchestration API (Heat):
  Fix Released
Status in OpenStack Dashboard (Horizon):
  In Progress
Status in Ironic (Bare Metal Provisioning):
  Fix Released
Status in Manila:
  New
Status in Murano Project:
  Fix Committed
Status in OpenStack Neutron (virtual network service):
  In Progress
Status in OpenStack Compute (Nova):
  In Progress
Status in Oslo - a Library of Common OpenStack Code:
  Fix Released
Status in OpenStack Data Processing (Savanna):
  Fix Released
Status in Staccato VM Image And Data Transfer Service:
  New
Status in Taskflow for task-oriented systems.:
  In Progress
Status in Trove - Database as a Service:
  Fix Released
Status in Tuskar:
  Fix Committed

Bug description:
  http://lists.openstack.org/pipermail/openstack-
  dev/2013-November/018980.html

  
  > Hi all,
  >
  > We had a discussion of the modules that are incubated in Oslo.
  >
  > https://etherpad.openstack.org/p/icehouse-oslo-status
  >
  > One of the conclusions we came to was to deprecate/remove uuidutils in
  > this cycle.
  >
  > The first step into this change should be to remove generate_uuid() from
  > uuidutils.
  >
  > The reason is that 1) generating the UUID string seems trivial enough to
  > not need a function and 2) string representation of uuid4 is not what we
  > want in all projects.
  >
  > To address this, a patch is now on gerrit.
  > https://review.openstack.org/#/c/56152/
  >
  > Each project should directly use the standard uuid module or implement its
  > own helper function to generate uuids if this patch gets in.
  >
  > Any thoughts on this change? Thanks.
  >

  Unfortunately it looks like that change went through before I caught up on
  email. Shouldn't we have removed its use in the downstream projects (at
  least integrated projects) before removing it from Oslo?

  Doug

To manage notifications about this bug go to:
https://bugs.launchpad.net/barbican/+bug/1253497/+subscriptions