Glance Add Image API returns 400 response if storage is inaccessible

Bug #919257 reported by Unmesh Gurjar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Medium
Eoghan Glynn

Bug Description

Glance Add Image API returns 400 response if storage is inaccessible or insufficient permissions in accessing the image store. The API should return an Internal Server Error(500) response instead.

Revision history for this message
Jay Pipes (jaypipes) wrote :

I tend to think that 500 Internal Server Error should be returned only when a server is entirely broken. I'm thinking that 503 Service Unavailable may be a better code to return...

Revision history for this message
Brian Waldon (bcwaldon) wrote :

Yep, I'm with Jay.

Changed in glance:
status: New → Triaged
importance: Undecided → Medium
milestone: none → essex-4
Eoghan Glynn (eglynn)
Changed in glance:
assignee: nobody → Eoghan Glynn (eglynn)
Eoghan Glynn (eglynn)
Changed in glance:
status: Triaged → In Progress
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/4251

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to glance (master)

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

commit 45f9e05572303b5bd8ea8ee3f179bb2269bb091c
Author: Eoghan Glynn <email address hidden>
Date: Thu Feb 16 20:24:39 2012 +0000

    Return 503 if insufficient permission on filestore

    Fixes lp 919257

    Return 503 "Service Unavailable" status from POST /images request
    where there's insufficient permission to write image content to
    the backend filestore.

    Change-Id: I0c4e8bb1e70d6542852a86903a4a2a200dd23f01

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: essex-4 → 2012.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.