[openstack-dev] [ALL] Removing generate_uuid() from uuidutils
andrew.laski at rackspace.com
Wed Nov 13 14:21:07 UTC 2013
On 11/13/13 at 05:48am, Gary Kotton wrote:
>I recall a few cycles ago having str(uuid.uuid4()) replaced by generate_uuid(). There was actually a helper function in neutron (back when it was called quantum) and it was replaced. So now we are going back…
>I am not in favor of this change.
I'm also not really in favor of it. Though it is a trivial method
having it in oslo implies that this is what uuids should look like
across OpenStack projects. And I'm in favor of consistency for uuids
across the projects because the same parsers and checkers can then be
used for input validation or log parsing.
>From: Zhongyue Luo <zhongyue.nah at intel.com<mailto:zhongyue.nah at intel.com>>
>Reply-To: "OpenStack Development Mailing List (not for usage questions)" <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
>Date: Wednesday, November 13, 2013 8:07 AM
>To: OpenStack Development Mailing List <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
>Subject: [openstack-dev] [ALL] Removing generate_uuid() from uuidutils
>We had a discussion of the modules that are incubated in Oslo.
>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/<https://urldefense.proofpoint.com/v1/url?u=https://review.openstack.org/%23/c/56152/&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=eH0pxTUZo8NPZyF6hgoMQu%2BfDtysg45MkPhCZFxPEq8%3D%0A&m=3ns0o3FRyS2%2Fg%2FTFIH7waZX1o%2FHdXvrJ%2FnH9XMCRy08%3D%0A&s=adb860d11d1ad02718e306b9408c603daa00970685a208db375a9ec011f13978>
>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.
>880 Zixing Road, Zizhu Science Park, Minhang District, 200241, Shanghai, China
>OpenStack-dev mailing list
>OpenStack-dev at lists.openstack.org
More information about the OpenStack-dev