nova.conductor.manager should list ComputeHostNotFound as a client exception

Bug #1154597 reported by Hans Lindgren
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Hans Lindgren

Bug Description

service_get_all_by() in nova.conductor.manager should list ComputeHostNotFound as a client exception for rpc. This to prevent the rpc layer from logging a traceback when this exception is raised. This is an expected type of exception that should be left to the client to handle.

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/24322

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

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

commit a8429a9bc943832c5b83fdeae004acc0fb832d37
Author: Hans Lindgren <email address hidden>
Date: Wed Mar 13 14:48:45 2013 +0100

    List ComputeHostNotFound as a client exception

    service_get_all_by() in nova.conductor.manager should list
    ComputeHostNotFound as a client exception for rpc. This to prevent
    the rpc layer from logging a traceback when this exception is raised.
    This is an expected type of exception that should be left to the
    client to handle.

    Resolves bug 1154597.

    Change-Id: I656e264526673fa10e47692903a5f30f813b479b

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → grizzly-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: grizzly-rc1 → 2013.1
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.