nova-network: inconsistent parameters in deallocate_for_instance()

Bug #1350355 reported by Oleg Bondarev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Oleg Bondarev

Bug Description

Client side of the network RPC API sets 'requested_networks' parameter in deallocate_for_instance() call [1]
while server side expects 'fixed_ips' parameter [2]

[1] https://github.com/openstack/nova/blob/master/nova/network/rpcapi.py#L183
[2] https://github.com/openstack/nova/blob/master/nova/network/manager.py#L555

Tags: network
Tracy Jones (tjones-i)
tags: added: api
Changed in nova:
status: New → Confirmed
importance: Undecided → High
tags: added: network
removed: api
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/119745

Changed in nova:
status: Confirmed → In Progress
Changed in nova:
assignee: Oleg Bondarev (obondarev) → Matt Riedemann (mriedem)
Changed in nova:
assignee: Matt Riedemann (mriedem) → Oleg Bondarev (obondarev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit 03d34c975586788dc25249b5e0b962fc0634008c
Author: Oleg Bondarev <email address hidden>
Date: Mon Sep 8 16:45:59 2014 +0400

    Nova-net: fix server side deallocate_for_instance()

    Client side of the network RPC API sets 'requested_networks'
    parameter in deallocate_for_instance() call,
    while server side expected 'fixed_ips' parameter.
    This patch fixes server side so it handles 'requested_networks'

    Closes-Bug: #1350355

    Change-Id: If9554a2e98397c5e63e929e1606d99dd2328262c

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → juno-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: juno-rc1 → 2014.2
Revision history for this message
Calvin Walton (kepstin) wrote :

This fix appears to have caused a regression when the "update_dns_entries=True" is set in nova.conf; I've opened a bug at https://bugs.launchpad.net/nova/+bug/1427295 to track that.

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.