[Kimchi-devel] [PATCH V9 7/7] write the template OS info to vm metadata

[Royce Lv]

Reviewed-by: Royce Lv<lvroyce at linux.vnet.ibm.com>
On 2014年04月25日 22:14, shaohef at linux.vnet.ibm.com wrote:
> From: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
>
> Then kimchi can choose the divice mode depend on these info.
>
> Note, the info is the initial OS when install OS on a blank guest.
>
> OS may change after the install.
>
> So other device should still keep their model info in the vm metadata.
>
> Signed-off-by: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
> Signed-off-by: Royce Lv <lvroyce at linux.vnet.ibm.com>
> ---
>   src/kimchi/model/vms.py | 18 ++++++++++++++++++
>   1 file changed, 18 insertions(+)
>
> diff --git a/src/kimchi/model/vms.py b/src/kimchi/model/vms.py
> index 8def756..e8b56ee 100644
> --- a/src/kimchi/model/vms.py
> +++ b/src/kimchi/model/vms.py
> @@ -223,6 +223,8 @@ class VMsModel(object):
>               raise OperationFailed("KCHVM0007E", {'name': name,
>                                                    'err': e.get_error_message()})
>
> +        VMModel.vm_updata_os_metadata(VMModel.get_vm(name, self.conn), t.info)
> +
>           return name
>
>       def get_list(self):
> @@ -288,6 +290,22 @@ class VMModel(object):
>           node = self._get_metadata_access_node(users, groups)
>           set_metadata_node(dom, ET.tostring(node))
>
> +    @staticmethod
> +    def vm_get_os_metadata(dom):
> +        os_xml = get_metadata_node(dom, "os") or """<os></os>"""
> +        os_elem = ET.fromstring(os_xml)
> +        return (os_elem.attrib.get("version"), os_elem.attrib.get("distro"))
> +
> +    @staticmethod
> +    def vm_updata_os_metadata(dom, params):
> +        distro = params.get("os_distro")
> +        version = params.get("os_version")
> +        if distro is None:
> +            return
> +        node = create_metadata_node("os")
> +        os_elem = node.os({"distro": distro, "version": version})
> +        set_metadata_node(dom, ET.tostring(os_elem))
> +
>       def _static_vm_update(self, dom, params):
>           state = DOM_STATE_MAP[dom.info()[0]]
>           old_xml = new_xml = dom.XMLDesc(0)