OSAPI 'create image' location header is missing tenant ID

Bug #862672 reported by Dan Prince
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Alex Meade
Diablo
Fix Released
Undecided
Unassigned

Bug Description

Using the v1.1 OSAPI:

When creating an image snapshot the location header contains a URL to the newly created image. The location header is currently missing the tenant ID.

For reference see:

http://docs.openstack.org/api/openstack-compute/1.1/content/Create_Image-d1e4655.html

Dan Prince (dan-prince)
Changed in nova:
importance: Undecided → Medium
status: New → Confirmed
Alex Meade (alex-meade)
Changed in nova:
assignee: nobody → Alex Meade (alex-meade)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : A change has been merged to openstack/nova

Reviewed: https://review.openstack.org/743
Committed: http://github.com/openstack/nova/commit/f3fb16a7935e91f8c9034d1da84a2b17cbe186f8
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit f3fb16a7935e91f8c9034d1da84a2b17cbe186f8
Author: Alex Meade <email address hidden>
Date: Thu Sep 29 15:58:43 2011 -0400

    Adds the tenant id to the create images response Location header
    Fixes bug 862672

    Change-Id: I85fc91c64146b02718dad77686762e26bdfce08c

Changed in nova:
status: Confirmed → Fix Committed
Revision history for this message
Mark McLoughlin (markmc) wrote :

Doesn't the same problem also apply to backup creation?

    def _action_create_backup(self, input_dict, req, instance_id):
        ...
        # build location of newly-created image entity
        image_id = str(image['id'])
        image_ref = os.path.join(req.application_url, 'images', image_id)

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to nova (stable/diablo)

Reviewed: https://review.openstack.org/1078
Committed: http://github.com/openstack/nova/commit/6f96fa65aa73e7803c9ba82d7a4f3b49b310e1b7
Submitter: Jenkins
Branch: stable/diablo

 status fixcommitted
 done

commit 6f96fa65aa73e7803c9ba82d7a4f3b49b310e1b7
Author: Alex Meade <email address hidden>
Date: Thu Sep 29 15:58:43 2011 -0400

    Adds the tenant id to the create images response Location header
    Fixes bug 862672

    (cherry picked from commit f3fb16a7935e91f8c9034d1da84a2b17cbe186f8)

    Change-Id: I97357dc76561db576d354eacf4d7756c664e0580

Thierry Carrez (ttx)
Changed in nova:
milestone: none → essex-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Hello Dan, or anyone else affected,

Accepted nova into oneiric-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Thierry Carrez (ttx)
Changed in nova:
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.