Comment 4 for bug 1460673

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

Reviewed: https://review.openstack.org/187206
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=240fb9c5392f71e59e6ef2f8f917b098f1cf9960
Submitter: Jenkins
Branch: master

commit 240fb9c5392f71e59e6ef2f8f917b098f1cf9960
Author: Dan Smith <email address hidden>
Date: Mon Jun 1 07:31:26 2015 -0700

    Make nova-manage handle completely missing flavor information

    If you have a really (really) old instance without any flavor information
    stashed in sysmeta, then nova-manage will not be able to convert it to
    a proper flavor object in extra. This patch makes it handle that case
    by looking up the flavor by id instead. Not only will this facilitate
    the transition, but will also ensure that any such legacy instances are
    properly brought up to date so that going forward we can just assume that
    they have all been converted (and remove some other places where we
    handle the has-no-flavor-info case).

    This involves changing the flavor cache to contain flavor objects
    instead of DB objects so that the two methods that use the cache can
    coexist. It doesn't much affect the _augment_flavors_to_migrate() path,
    other than some mechanics and test changes.

    Change-Id: I54a056d339d98bc4092af8cf9f4f5d24b882506b
    Closes-Bug: #1460673