OpenStack Compute (Nova)

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

Reported by Dan Prince on 2011-09-28
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Low
Brian Waldon

Bug Description

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

Dan Prince (dan-prince) on 2011-09-28
Changed in nova:
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Dan Prince (dan-prince)
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
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) on 2011-10-12
tags: added: osapi-v1.1

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) on 2011-11-09
Changed in nova:
milestone: none → essex-1
Thierry Carrez (ttx) on 2011-11-17
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in nova:
milestone: essex-1 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers