OSAPI: v1.1 links containers do not match spec
Bug #803505 reported by
Brian Waldon
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Compute (nova) |
Fix Released
|
Low
|
Brian Waldon | ||
Bug Description
The links container for a flavor with id 3 is currently formatted like this:
[
{u'href': u'http://
{u'href': u'http://
{u'href': u'http://
]
According to the latest v1.1 spec, it should be:
[
{'href': 'http://
{'href': 'http://
]
This can be applied to servers and images as well.
Related branches
lp:~rackspace-titan/nova/osapi-flavors-links-lp803505
- Brian Waldon (community): Approve
- Dan Prince (community): Approve
-
Diff: 390 lines (+56/-130)7 files modifiednova/api/openstack/common.py (+10/-0)
nova/api/openstack/views/flavors.py (+9/-7)
nova/api/openstack/views/images.py (+9/-7)
nova/api/openstack/views/servers.py (+7/-7)
nova/tests/api/openstack/test_flavors.py (+5/-35)
nova/tests/api/openstack/test_images.py (+14/-60)
nova/tests/api/openstack/test_servers.py (+2/-14)
lp:~rackspace-titan/nova/images-links-lp803505
- Brian Lamar (community): Approve
- William Wolf (community): Approve
- Devin Carlen (community): Approve
-
Diff: 55 lines (+21/-11)2 files modifiednova/api/openstack/views/images.py (+11/-7)
nova/tests/api/openstack/test_images.py (+10/-4)
| Changed in nova: | |
| assignee: | nobody → Alex Meade (alex-meade) |
| Changed in nova: | |
| status: | New → In Progress |
Revision history for this message
| Alex Meade (alex-meade) wrote | #1 |
Revision history for this message
| Brian Waldon (bcwaldon) wrote | #2 |
Revision history for this message
| Alex Meade (alex-meade) wrote | #3 |
| summary: |
- OSAPI: v1.1 flavors links container does not match spec + OSAPI: v1.1 links containers do not match spec |
| description: | updated |
| Changed in nova: | |
| importance: | Undecided → Low |
| Changed in nova: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Brian Waldon (bcwaldon) wrote | #4 |
| Changed in nova: | |
| status: | Fix Committed → Triaged |
| Changed in nova: | |
| assignee: | Alex Meade (alex-meade) → Brian Waldon (bcwaldon) |
| Changed in nova: | |
| status: | Triaged → Fix Committed |
| Changed in nova: | |
| milestone: | none → diablo-3 |
| Changed in nova: | |
| milestone: | diablo-3 → 2011.3 |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This also seems to be the case for images and servers....and are we certain the spec is correct regarding the removal of the version in the 'bookmark' link?