Adding image with swift backend poorly constructs url

Bug #860010 reported by Brian Waldon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Low
Brian Waldon

Bug Description

I added an image with swift as the default backend. The auth url in the config was 'auth.api.rackspacecloud.com/v1.0/'. The location returned by glance after adding was 'swift+https://rscsbcb:<API_KEY>@auth.api.rackspacecloud.com/v1.0//waldon_testing/7'. It is a minor detail, but there are two slashes after 'v1.0'.

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

Debugging the tests, and I noticed the following assertEquals:

swift+https://user:key@localhost/v1//glance/49
swift+https://user:key@localhost/v1//glance/49

The format_swift_location method in glance.tests.unit.test_swift_store seems to be the culprit

Changed in glance:
importance: Undecided → Low
status: New → In Progress
Revision history for this message
Brian Waldon (bcwaldon) wrote :
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : A change has been merged to openstack/glance

Reviewed: https://review.openstack.org/670
Committed: http://github.com/openstack/glance/commit/2b43f89e2cb7e0d89b9f2a957b4ae25c2cb496d7
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 2b43f89e2cb7e0d89b9f2a957b4ae25c2cb496d7
Author: Brian Waldon <email address hidden>
Date: Mon Sep 26 16:57:59 2011 -0400

    Improving swfit store uri construction

    Fixing bug 860010

    Change-Id: I1b414fafb492f3cafd5a1e857d55e8a8d3fa9a8a

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
milestone: none → essex-1
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: essex-1 → 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.