id() builtin used instead of subnet id in error messages

Bug #1213930 reported by Salvatore Orlando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
ZhiQiang Fan
ZhiQiang Fan (aji-zqfan)
Changed in neutron:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
status: New → In Progress
Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

Actually, _validate_subnet is called in create_subnet() and update_subnet(), and the param s[ubnet] is user's input, which will not ontain an id

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/42679

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/42679
Committed: http://github.com/openstack/neutron/commit/6b0c899d36d5a98963342e9cffb351fc3a1f4fd2
Submitter: Jenkins
Branch: master

commit 6b0c899d36d5a98963342e9cffb351fc3a1f4fd2
Author: ZhiQiang Fan <email address hidden>
Date: Tue Aug 20 00:05:15 2013 +0800

    Use subnet id instead of wrong built-in id()

    In _validate_subnet(), built-in id() is used as param for exceptions,
    this patch fixes it via using proper subnet id.

    Closes-Bug: #1213930

    Change-Id: I9a88f4dc7b771047f4061fb94ba65f0515afa745

Changed in neutron:
status: In Progress → Fix Committed
Changed in neutron:
milestone: none → havana-3
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.