driver flavor ids need to accept non-int values

Bug #1515000 reported by Adam Gandelman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Astara
Fix Released
High
Adam Gandelman
Liberty
Fix Committed
High
Unassigned

Bug Description

We use instance_flavor to define the nova flavor id the appliance should be booted with. This is an IntOpt but needs to be a StrOpt to accept both integer IDs and UUIDs/strings.

.:stupidnic:. So in the [router] section you can specify the flavor
.:stupidnic:. and the default is 1, I don't have a 1 since I edited the m1.tiny, and when you do that, it drops the int and replaces it with an uuid
.:stupidnic:. however you can't use the uuid in the instance_flavor value
.:stupidnic:. it complains that it isn't an int() base 10

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

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

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

Reviewed: https://review.openstack.org/243820
Committed: https://git.openstack.org/cgit/openstack/astara/commit/?id=a355c4d73d52222b58c3cd36791af60fade92cff
Submitter: Jenkins
Branch: master

commit a355c4d73d52222b58c3cd36791af60fade92cff
Author: Adam Gandelman <email address hidden>
Date: Tue Nov 10 12:49:45 2015 -0800

    Convert instance_flavor configs from IntOpt to StrOpt

    Nova instances are not limited to integer IDs and this is required
    to support instance flavors with UUIDs or str IDs.

    Closes-bug: #1515000
    Change-Id: I1abf665f43cb33c1d1d2bd612aed750cae3455a6

Changed in astara:
status: In Progress → Fix Released
Changed in astara:
assignee: nobody → Adam Gandelman (gandelman-a)
importance: Undecided → Medium
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to astara (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/273778

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to astara (stable/liberty)

Reviewed: https://review.openstack.org/273778
Committed: https://git.openstack.org/cgit/openstack/astara/commit/?id=a8336091a8c420ae168c1a380275f0c5654451cc
Submitter: Jenkins
Branch: stable/liberty

commit a8336091a8c420ae168c1a380275f0c5654451cc
Author: Adam Gandelman <email address hidden>
Date: Tue Nov 10 12:49:45 2015 -0800

    Convert instance_flavor configs from IntOpt to StrOpt

    Nova instances are not limited to integer IDs and this is required
    to support instance flavors with UUIDs or str IDs.

    Closes-bug: #1515000
    Change-Id: I1abf665f43cb33c1d1d2bd612aed750cae3455a6
    (cherry picked from commit a355c4d73d52222b58c3cd36791af60fade92cff)

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.