Melange_ipam_lib needs to catch the right exceptions

Bug #967261 reported by Jason Kölker
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Jason Kölker

Bug Description

in melange_ip_lib get_tenatn_id_by_net_id we search for the right ipam_teant_id by calling out to get_allocated_ips. It triggers going to the next tenant_id in the list by catching KeyError, but melange_connection is raising a bare Exception if the block is not found for that tenant.

Changed in nova:
assignee: nobody → Jason Kölker (jason-koelker)
status: New → Confirmed
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/5912

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

Reviewed: https://review.openstack.org/5912
Committed: http://github.com/openstack/nova/commit/dc5a29d8bd5f041c3864aed915f8ceb70ce7978a
Submitter: Jenkins
Branch: master

commit dc5a29d8bd5f041c3864aed915f8ceb70ce7978a
Author: Jason Kölker <email address hidden>
Date: Wed Mar 28 11:15:00 2012 -0500

    Continue on the the next tenant_id on 400 codes

    * Melange connection raises Exception if the response is > 400
    * Fixes lp967261

    Change-Id: I4592e5907276d290f4a54aa27dd4e91a62cc83e8

Changed in nova:
status: In Progress → Fix Committed
Devin Carlen (devcamcar)
Changed in nova:
milestone: none → folsom-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-1 → 2012.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.