Describe addresses in ec2 api broken with neutron

Bug #1273837 reported by Vish Ishaya
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
wingwj

Bug Description

Describe addresses using the ec2 api is broken when using neutron. It attempts to retrieve the fixed ip directly which by id:

https://github.com/openstack/nova/blob/bc10b3c2b222b5f5c6ee6ffb79c12a8d3e2931bf/nova/api/ec2/cloud.py#L1209

which is not supported by neutron:

https://github.com/openstack/nova/blob/bc10b3c2b222b5f5c6ee6ffb79c12a8d3e2931bf/nova/network/neutronv2/api.py#L693

It should be pulling the instance uuid from the floating list directly like we do in the v2 api:

https://github.com/openstack/nova/blob/bc10b3c2b222b5f5c6ee6ffb79c12a8d3e2931bf/nova/api/openstack/compute/contrib/floating_ips.py#L74

Changed in nova:
importance: Undecided → Medium
status: New → Triaged
tags: added: ec2 havana-backport-potential
wingwj (wingwj)
Changed in nova:
assignee: nobody → wingwj (wingwj)
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/72628

Changed in nova:
status: Triaged → In Progress
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to nova (master)

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

commit 3d0d6b7f22d0f3880d65143493d8a07a47a6f519
Author: wingwj <email address hidden>
Date: Tue Feb 11 19:46:53 2014 +0800

    Describe addresses in ec2 api broken with neutron

    It attempts to retrieve the fixed ip directly,
    but it's not supported by neutron.

    After the modification,
    the instance uuid will be pulled without using get_fixed_ip().

    Change-Id: Ia1b95d4c18aeb5012919af89148947694d80e2e7
    Closes-Bug: #1273837

Changed in nova:
status: In Progress → Fix Committed
tags: added: icehouse-backport-potential
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-1 → 2014.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.