make flavor 'id' a string in the OSAPI v1.1

Bug #861666 reported by Dan Prince
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Brian Waldon

Bug Description

Flavor ID should be returned as a string in the OSAPI v1.1.

Tags: osapi-v1.1
Dan Prince (dan-prince)
Changed in nova:
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Dan Prince (dan-prince)
Revision history for this message
Dan Prince (dan-prince) wrote :

Apparently flavors are really going to be uuids in the v1.1 OSAPI.

So now instead of just knowing that a 2 Gig instance is flavor ID 2 I get to do something like this:

  FLAVOR_ID=$(nova flavor-list | grep 2048 | cut -f 2 -d "|" | tr -d ' ')

Not really what I'd consider to be a step forward in usability. :(

Changed in nova:
status: In Progress → Invalid
Revision history for this message
Brian Waldon (bcwaldon) wrote :

Actually, just making it a string is fine. The spec doesn't require us to use uuids, just strings. I've got a branch in progress to do this.

Changed in nova:
status: Invalid → In Progress
assignee: Dan Prince (dan-prince) → Brian Waldon (bcwaldon)
Brian Waldon (bcwaldon)
tags: added: osapi-v1.1
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/859
Committed: http://github.com/openstack/nova/commit/0162a42970b833c2d5d0802ff4c55f65fa253ee2
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit 0162a42970b833c2d5d0802ff4c55f65fa253ee2
Author: Brian Waldon <email address hidden>
Date: Wed Oct 12 16:28:24 2011 -0400

    Convert instancetype.flavorid to string

    Fixes bug 861666. This also removes some direct database access
    in favor of using nova.compute.instance_types throughout the code.

    Change-Id: I572cc19454fa76f435f5672d3d6e7ed55c8817da

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