network_model tests mix IP() and FixedIP

Bug #1260891 reported by Aaron Rosen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Aaron Rosen

Bug Description

network_model tests mix IP() and FixedIP

Tags: network
Aaron Rosen (arosen)
Changed in nova:
assignee: nobody → Aaron Rosen (arosen)
importance: Undecided → Medium
tags: added: network
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/62104
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a86136f8a198933d6099b1dabc007e36c4806e3d
Submitter: Jenkins
Branch: master

commit a86136f8a198933d6099b1dabc007e36c4806e3d
Author: Aaron Rosen <email address hidden>
Date: Fri Dec 13 14:53:15 2013 -0800

    Correct network_model tests and __eq__ operator

    The current network_model tests mix IP() with FixedIP()
    in several places. This patch corrects this issue. This was discovered
    while adding logic to determine if the network_cache needs to be updated.
    While updating the __eq__ operator to check all of the objects fields
    rather than just id there tests started failing.

    Closes-bug: # 1260891
    Related to blueprint: smarter-network-cache-update

    Change-Id: I6d144aecf95725146aa0fc4f4fe5a3f16c434d14

Changed in nova:
status: In Progress → Fix Committed
Changed in nova:
milestone: none → icehouse-rc1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: icehouse-rc1 → 2014.1
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.