Baremetal driver reports bogus vm_mode of 'baremetal'

[Daniel Berrange]

The Baremetal driver reports a 'vm_mode' of 'baremetal' for supported instance types. This is bogus because the baremetal driver is running OS using the native machine ABI, which is represented by vm_mode.HVM

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/109593

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/109593
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=77efa337c5b17b1853c6dd0f02009ed60925ff37
Submitter: Jenkins
Branch: master

commit 77efa337c5b17b1853c6dd0f02009ed60925ff37
Author: Daniel P. Berrange <email address hidden>
Date: Fri Jul 25 12:41:53 2014 +0100

    compute: tweaks to vm_mode APIs to align with arch/hvtype

    Rename the 'name' method to 'canonicalize' to make its purpose
    clearer. Also add a 'is_valid' method to check validity without
    doing canonicalization. Make it use a specialized exception
    class instead of the generic 'Invalid' class.

    Related-bug: #1348629
    Change-Id: I4bb972aeafb69be594bff40eded3cfeec22f0bef

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/109594
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8fcd1b2c73dad8e2a7b4d299da270934fd5328cc
Submitter: Jenkins
Branch: master

commit 8fcd1b2c73dad8e2a7b4d299da270934fd5328cc
Author: Daniel P. Berrange <email address hidden>
Date: Fri Jul 25 12:55:40 2014 +0100

    virt: use compute.vm_mode constants and validate vm mode type

    Where we have hardcoded vm modes, use compute.vm_mode constants.
    Where we get vm modes from external systems, validate them against
    the list of acceptable names.

    The baremetal/ironic drivers are reporting a bogus vm mode of
    'baremetal' for supported instances which is confusing the vm
    mode with the Nova driver type. These drivers use the native
    ABI for their architecture, so should be reporting 'hvm' as
    the vm mode, which indicates an unmodified native OS ABI.

    On the other side, the ImagePropertiesFilter will canonicalize
    the hvtype it fetches from image metadata, so that 'baremetal'
    gets remapped to 'hvm' during comparison.

    Closes-bug: #1348629
    Change-Id: Ibda7c5ab0759aee672870d84974ccdea25f14407