volume transfer url is different from its href in detail body

Bug #1242980 reported by Dong Liu
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Dong Liu
Havana
Fix Released
High
Dong Liu

Bug Description

After creating a volume transfer, I got a detail information of volume transfer like this:
{
    "transfer": {
        "auth_key": "cfcb4552f7c1f2df",
        "links": [
            {
                "href": "http://192.168.83.241:8776/v2/c7d8da28afff47189ff1d651992b593b/transfers/dfedf224-1aba-420a-a387-3b971447a28f",
                "rel": "self"
            },
            {
                "href": "http://192.168.83.241:8776/c7d8da28afff47189ff1d651992b593b/transfers/dfedf224-1aba-420a-a387-3b971447a28f",
                "rel": "bookmark"
            }
        ],
        "created_at": "2013-10-22T01:18:35.627545",
        "volume_id": "f636037c-1aa0-4533-aef2-bcbd7ab3896c",
        "id": "dfedf224-1aba-420a-a387-3b971447a28f",
        "name": null
    }
}

But I cannot describe its detail information with this href of this retrun.
        "links": [
            {
                "href": "http://192.168.83.241:8776/v2/c7d8da28afff47189ff1d651992b593b/transfers/dfedf224-1aba-420a-a387-3b971447a28f",
                "rel": "self"
            },

The correct url is:
"http://192.168.83.241:8776/v2/c7d8da28afff47189ff1d651992b593b/os-volume-transfer/dfedf224-1aba-420a-a387-3b971447a28f"

Then I found that the define of collection_name are defferent between /cinder/api/views/transfers.py and /cinder/api/contrib/volume_transfer.py.

So I think this is a bug.

Dong Liu (liudong78)
Changed in cinder:
assignee: nobody → Dong Liu (liudong78)
wingwj (wingwj)
summary: - volume tranfer url is different from its href in detal body
+ volume transfer url is different from its href in detal body
description: updated
tags: added: volume-transfer
Dong Liu (liudong78)
summary: - volume transfer url is different from its href in detal body
+ volume transfer url is different from its href in detail body
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

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

Reviewed: https://review.openstack.org/53093
Committed: http://github.com/openstack/cinder/commit/b104d1c0c70898b59b6668970e05cebadecd064a
Submitter: Jenkins
Branch: master

commit b104d1c0c70898b59b6668970e05cebadecd064a
Author: liudong <email address hidden>
Date: Tue Oct 22 18:37:50 2013 +0800

    Fix volume transfer href issue

    The volume transfer collection in url defined as "os-volume-tranfer",
    but I got another define "tranfers" in href url. They are different.
    So it should be modify.

    Change-Id: I69a43012f2bb70ce5fc7b0331d2759527a261170
    closes-bug: #1242980

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/64007

Changed in cinder:
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/havana)

Reviewed: https://review.openstack.org/64007
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=018ece5a879433c37c857d7623d67dda2d5af7f4
Submitter: Jenkins
Branch: stable/havana

commit 018ece5a879433c37c857d7623d67dda2d5af7f4
Author: liudong <email address hidden>
Date: Tue Oct 22 18:37:50 2013 +0800

    Fix volume transfer href issue

    The volume transfer collection in url defined as "os-volume-tranfer",
    but I got another define "tranfers" in href url. They are different.
    So it should be modify.

    closes-bug: #1242980
    Change-Id: I69a43012f2bb70ce5fc7b0331d2759527a261170
    (cherry picked from commit b104d1c0c70898b59b6668970e05cebadecd064a)

tags: added: in-stable-havana
Alan Pevec (apevec)
tags: removed: in-stable-havana
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-1 → 2014.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.