Disassociate Floating IP from an instance with 2 NICs doesn't work

[marcos.gm]

An instance with two internal networks configured, and a floating IP associated, cannot release the Floating IP because a message 'No floating IP's to disassociate' pops-up

As a result, the instance keeps its floating IP

Openstack version: Havana installed via Packstack (on Centos6)
[root@compute2 ~]# nova --version
2.15.0
[root@compute2 ~]# neutron --version
2.3.1

Screenshots attached (in them, the floating IP is 192.168.1.59)

[George Peristerakis (george-peristerakis)]

I traced the problem down to the function api.neutron.FloatingIpManager.get_target_id_by_instance. The function returns the first ip in the list. This function should be returning a list of ips associated to the instance.

[OpenStack Infra (hudson-openstack)]

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

[marcos.gm (marcos-gm)]

thank you! I'll see how can I possibly test it again

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/65164
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=86f87944900e6e713d2baaacec18f55cbe81d878
Submitter: Jenkins
Branch: master

commit 86f87944900e6e713d2baaacec18f55cbe81d878
Author: George Peristerakis <email address hidden>
Date: Mon Jan 6 17:23:03 2014 -0500

    Bug fix on neutron's API to return the correct target ID

    In the class FloatingIpManager, the function filters the
    resulting list of ports with all available floating ips.
    The function now returns the target ID of a port with a
    floating IP associated to it. Added a new function to get
    this to pass. In the next phase is to refactor the
    floating ip api to allow a finer granularity. This will
    be useful later for managing floating ips.

    Change-Id: I02286dbbb60318b0157992622ffdeeae45a71bc1
    Closes-Bug: 1265872

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/87734

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/87734
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=6ad29a1f55dadb110b2aacf5f4e5870d4a0c0129
Submitter: Jenkins
Branch: stable/havana

commit 6ad29a1f55dadb110b2aacf5f4e5870d4a0c0129
Author: George Peristerakis <email address hidden>
Date: Mon Jan 6 17:23:03 2014 -0500

    Bug fix on neutron's API to return the correct target ID

    In the class FloatingIpManager, the function filters the
    resulting list of ports with all available floating ips.
    The function now returns the target ID of a port with a
    floating IP associated to it. Added a new function to get
    this to pass. In the next phase is to refactor the
    floating ip api to allow a finer granularity. This will
    be useful later for managing floating ips.

    Conflicts:
     openstack_dashboard/dashboards/project/instances/tables.py

    Change-Id: I02286dbbb60318b0157992622ffdeeae45a71bc1
    Closes-Bug: 1265872
    (cherry picked from commit 86f87944900e6e713d2baaacec18f55cbe81d878)