xenapi: glance upload retries should not occur on 401 errors

Bug #1199454 reported by John Garbutt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
John Garbutt

Bug Description

When the user's token expires, you might get this error during the upload of the glance image from the XenAPI plugin:

glance: ERROR Unexpected response while writing image data to http://X.X.X.X/v1/images/<uuid>: Response Status: 401, Response body: 401 Unauthorized This server could not verify that you are authorized to access the document you requested. Either you supplied the wrong credentials (e.g., bad password), or your browser does not understand how to supply the credentials required. Authentication required

It seems pointless to retry in these cases without making an error to refresh a token, or similar.

Tags: xenserver
Changed in nova:
importance: Undecided → Medium
status: New → Triaged
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/36308

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

Reviewed: https://review.openstack.org/36308
Committed: http://github.com/openstack/nova/commit/8e81e069a18e09562abfc4b18c5c7e57020aee66
Submitter: Jenkins
Branch: master

commit 8e81e069a18e09562abfc4b18c5c7e57020aee66
Author: John Garbutt <email address hidden>
Date: Tue Jul 9 18:49:46 2013 +0100

    xenapi: no glance upload retry on 401 error

    It is possible for a token to expire during a very long image upload.
    If this happens, there is little point in retrying the upload, as it
    is most likely to fail again for the same reason.

    Fixes bug 1199454

    Change-Id: Id699135a8ec4c603671b16f0127d0b2cd2a55855

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