← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1659053] Re: use uuids with pycadf

 

Reviewed:  https://review.openstack.org/426411
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=2be615ea9303109f11033e293c229df586e3b1e1
Submitter: Jenkins
Branch:    master

commit 2be615ea9303109f11033e293c229df586e3b1e1
Author: Gage Hugo <gagehugo@xxxxxxxxx>
Date:   Fri Jan 27 15:52:02 2017 -0600

    Fix multiple uuid warnings with pycadf
    
    There are multiple pycadf warnings about invalid uuids when
    running keystone tests:
    
    To ensure interoperability, identifiersshould be a valid uuid.
      warnings.warn('Invalid uuid. To ensure interoperability, identifiers'
    
    This changes multiple fixtures within default_fixtures to use valid
    uuids for their 'id' values. Also changed load_fixtures to build
    the fixtures based on the default_fixtures' 'name' value, rather
    than 'id'. Replaced many instances of invalid hard-coded ids to use
    random uuids, default_fixture ids, or 'default'.
    
    Co-Authored-By: Tin Lam <tinlam@xxxxxxxxx>
    
    Change-Id: Ic4fff30c306561b71288712480f073aba1fccbde
    Closes-Bug: #1659053
    Depends-On: I58bba04c21c2d24fd37850c9ecc6fac99deb3fc4


** Changed in: keystone
       Status: In Progress => Fix Released

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

Title:
  use uuids with pycadf

Status in OpenStack Identity (keystone):
  Fix Released
Status in pycadf:
  In Progress

Bug description:
  pycadf warnings are plentiful in keystone tests: UserWarning: Invalid uuid. To ensure interoperability, identifiersshould be a valid uuid.
    warnings.warn('Invalid uuid. To ensure interoperability, identifiers'

  
  Be sure keystone is providing uuids appropriately.

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


References