← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1620841] Re: uuid_list validator regression

 

Reviewed:  https://review.openstack.org/366398
Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=3d043c0ebc0528508a7a7393388c674afd44bae1
Submitter: Jenkins
Branch:    master

commit 3d043c0ebc0528508a7a7393388c674afd44bae1
Author: Henry Gessau <HenryG@xxxxxxxxxx>
Date:   Tue Sep 6 18:22:34 2016 -0400

    Fix regression in validate_uuid_list()
    
    Change https://review.openstack.org/358088 broke validate_uuid_list()
    by not including the valid_values parameter.
    
    Closes-Bug: #1620841
    
    Change-Id: Ic75f4d1319bb88a9e4f9e4699575ce8e75829841


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

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1620841

Title:
  uuid_list validator regression

Status in neutron:
  Fix Released

Bug description:
  Change https://review.openstack.org/358088 broke validate_uuid_list()

  Validators are always called with a valid_values argument [1] even though it may not be used:
  https://github.com/openstack/neutron/blob/9.0.0.0b3/neutron/api/v2/attributes.py#L431

  The flavors extension was broken by this since it uses uuid_list for the service_profiles attribute:
  https://github.com/openstack/neutron/blob/9.0.0.0b3/neutron/extensions/flavors.py#L109-L111

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


References