dictmodel string representation is port specific

Bug #1512753 reported by Kevin Benton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Kevin Benton

Bug Description

the __str__ representation added in https://review.openstack.org/#/c/219706/ is port specific. It will throw an error if it is used on a non-port object that doesn't have the 'id' or 'network_id' fields.

Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
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/241249

Changed in neutron:
status: New → In Progress
Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → High
milestone: none → mitaka-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/241249
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=c6476838cc7d94982f3d6aea88926679cf40d839
Submitter: Jenkins
Branch: master

commit c6476838cc7d94982f3d6aea88926679cf40d839
Author: Kevin Benton <email address hidden>
Date: Tue Nov 3 07:31:54 2015 -0800

    Make string representation of DictModel generic

    The previous version was specific to a port dict and would
    fail if str() was called on other models. This iterates over
    and logs all values of the dict instead so it works under any
    dictionary.

    Change-Id: I630b39338e4e67080e100a9f32c9b6e341a93472
    Closes-Bug: #1512753

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/neutron 8.0.0.0b1

This issue was fixed in the openstack/neutron 8.0.0.0b1 development milestone.

Changed in neutron:
status: Fix Committed → Fix Released
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.