In the nova-volume api a GET to /types returns extra db fields for json response

Bug #1027281 reported by Chuck Thier
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Eoghan Glynn
OpenStack Compute (nova)
Fix Released
Low
Eoghan Glynn

Bug Description

Right now a GET to /types will return like the following:

{
            "created_at": "2012-07-11 22:20:36",
            "deleted": false,
            "deleted_at": null,
            "extra_specs": {},
            "id": 1,
            "name": "SATA",
            "updated_at": null
}

It should instead be:

{
            "id": 1,
            "name": "SATA",
            "extra_specs": {}
}

Eoghan Glynn (eglynn)
Changed in nova:
status: New → Confirmed
assignee: nobody → Eoghan Glynn (eglynn)
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/10334

Changed in nova:
status: Confirmed → In Progress
Eoghan Glynn (eglynn)
Changed in cinder:
assignee: nobody → Eoghan Glynn (eglynn)
status: New → In Progress
Changed in cinder:
importance: Undecided → Low
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/10357

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

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

commit eb1e9ae114eccae373157c5391d06c97eec1e0a2
Author: Eoghan Glynn <email address hidden>
Date: Thu Jul 26 11:02:05 2012 +0100

    Trim volume type representation.

    Fixes nova-volume aspect of LP 1027281.

    Use view builder pattern to only include non-extraneous
    volume type attributes in the representation.

    Change-Id: I33f1b579fd6ed4ed98fa04e9148e91eca125213e

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

Reviewed: https://review.openstack.org/10357
Committed: http://github.com/openstack/cinder/commit/95262bb692746ca89dc01337357c4329766df5e5
Submitter: Jenkins
Branch: master

commit 95262bb692746ca89dc01337357c4329766df5e5
Author: Eoghan Glynn <email address hidden>
Date: Thu Jul 26 16:26:55 2012 +0000

    Trim volume type representation.

    Fixes cinder aspect of LP 1027281.

    Use view builder pattern to only include non-extraneous
    volume type attributes in the representation.

    Change-Id: I33f1b579fd6ed4ed98fa04e9148e91eca125213e

Changed in cinder:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in nova:
importance: Undecided → Low
milestone: none → folsom-3
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → folsom-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: folsom-3 → 2012.2
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-3 → 2012.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.