Glance

v2 API image upload lacks exception handling

Bug #1037979 reported by IWAMOTO Toshihiro
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
High
Iccha Sethi
Glance 2012.2 "folsom"

Bug Description

ImageDataController.upload catches only exception.Duplicate.
When store API raises other types of exceptions (eg. exception.StorageFull), v1 API returns HTTPRequestEntityTooLarge, but v2 API would result in HTTP error code 500.

I think v2 API code needs better exception handling.

Brian Waldon (bcwaldon)
Changed in glance:
milestone: none → folsom-rc1
status: New → Triaged
importance: Undecided → High
Iccha Sethi (iccha-sethi)
Changed in glance:
assignee: nobody → Iccha Sethi (iccha-sethi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

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

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

Reviewed: https://review.openstack.org/11653
Committed: http://github.com/openstack/glance/commit/c82ae99bcb5f7d413887cf9a0a197a9fba683c73
Submitter: Jenkins
Branch: master

commit c82ae99bcb5f7d413887cf9a0a197a9fba683c73
Author: isethi <email address hidden>
Date: Mon Aug 20 16:10:03 2012 +0000

    Exception Handling for image upload in v2

    Adds logging and exception handling for image data controller
    in glance v2

    Fixes bug 1037979

    Change-Id: Ieba0b9766d7c205e361a3d9a9618300af815beb5

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: folsom-rc1 → 2012.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.