OpenStack Image Registry and Delivery Service (Glance)

glance-manage tries to catch exception.Error which doesn't exist

Reported by Chris Behrens on 2011-10-31
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Low
Brian Waldon

Bug Description

in glance-manage, there's a "except exception.Error, e:" but there's no 'Error' class in glance/common/exception.py

Jay Pipes (jaypipes) wrote :

LOL. :)

Changed in glance:
status: New → Confirmed
importance: Undecided → Low
tags: added: low-hanging-fruit

Yeah. :) It's covering up a SQL migration error that would normally fall through and print a useful TRACE. Instead, we get a TRACE on this exception.Error issue.

Brian Waldon (bcwaldon) on 2011-11-01
Changed in glance:
assignee: nobody → Brian Waldon (bcwaldon)
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/1224
Committed: http://github.com/openstack/glance/commit/45c72807835fe884901c49652b70701ecfc0e475
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 45c72807835fe884901c49652b70701ecfc0e475
Author: Brian Waldon <email address hidden>
Date: Tue Nov 1 10:14:36 2011 -0400

    Correcting exception handling in glance-manage

    Fixes bug 884432

    Change-Id: I76c0a97e502dd05b4c56c84fa640d1f0e0cd48ae

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2011-11-09
Changed in glance:
milestone: none → essex-1
Thierry Carrez (ttx) on 2011-11-17
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in glance:
milestone: essex-1 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers