legacy_net_info guard incompatible with new-style NetworkInfo

Bug #1164152 reported by Rick Harris
30
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Rick Harris
Grizzly
Fix Released
Low
Unassigned

Bug Description

In order to fix https://bugs.launchpad.net/nova/+bug/1152426

a guard was added to _init_instance in compute manger.

The problem is that the guard assumed that `legacy_net_info` would always be a tuple which is not true. Depending on the config, it may be a list of tuples, or a NetworkInfo object.

Tags: vmware
Changed in nova:
assignee: nobody → Rick Harris (rconradharris)
status: New → In Progress
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/26042

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

Reviewed: https://review.openstack.org/26042
Committed: http://github.com/openstack/nova/commit/948d1fefe6cb1e17c5567d3cf3313e13c16a3d57
Submitter: Jenkins
Branch: master

commit 948d1fefe6cb1e17c5567d3cf3313e13c16a3d57
Author: Rick Harris <email address hidden>
Date: Wed Apr 3 21:09:50 2013 +0000

    Fix legacy_net_info guard

    The existing code assumes that `legacy_net_info` is always in legacy
    mode, meaning a list of tuples which causes it to break when passed a
    new-style NetworkInfo object.

    Fixes bug 1164152

    Change-Id: I2131d9b24045cd7531454b65d97776b11ec3ab02

Changed in nova:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/grizzly)

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/26613

Changed in nova:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/grizzly)

Reviewed: https://review.openstack.org/26613
Committed: http://github.com/openstack/nova/commit/b2ec66823baf2a5094a9b398c2554bdcd2aba928
Submitter: Jenkins
Branch: stable/grizzly

commit b2ec66823baf2a5094a9b398c2554bdcd2aba928
Author: Rick Harris <email address hidden>
Date: Wed Apr 3 21:09:50 2013 +0000

    Fix legacy_net_info guard

    The existing code assumes that `legacy_net_info` is always in legacy
    mode, meaning a list of tuples which causes it to break when passed a
    new-style NetworkInfo object.

    Fixes bug 1164152

    Change-Id: I2131d9b24045cd7531454b65d97776b11ec3ab02
    (cherry picked from commit 948d1fefe6cb1e17c5567d3cf3313e13c16a3d57)

Thierry Carrez (ttx)
Changed in nova:
milestone: none → havana-1
status: Fix Committed → Fix Released
Tracy Jones (tjones-i)
tags: added: vmware-co-preferred
tags: added: vmware
removed: vmware-co-preferred
Tracy Jones (tjones-i)
Changed in openstack-vmwareapi-team:
status: New → Fix Released
Tracy Jones (tjones-i)
Changed in openstack-vmwareapi-team:
importance: Undecided → Low
no longer affects: openstack-vmwareapi-team
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-1 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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