Re: [Kimchi-devel] [PATCH V8 0/7] bug fix: get user and group when VM is running

Let me make a diagram about what this patch should cover: 1) Identify system has support for virDomain.metadata and virDomain.setMetadata FeatureTests.has_metadata_support() returns True or False It should not raise any exception Insert it in CapabilitiesModel (src/kimchi/model/config.py) that way we have all feature tests in one place 2) Use FeatureTests.has_metadata_support() result to SET metadata element KIMCHI_META_URL = "https://github.com/kimchi-project/kimchi/" KIMCHI_NAMESPACE = "kimchi" if FeatureTests.has_metadata_support(): dom.setMetadata(libvirt.VIR_DOMAIN_METADATA_ELEMENT, <meta_xml>, KIMCHI_NAMESPACE, KIMCHI_META_URL) The <meta_xml> should be something like, built in _get_metadata_node() <kimchi> <access> <user>...</user> ... <group>...</group> ... </access> <os> .... </os> </kimchi> # FeatureTests.has_metadata_support() is False else: # use ElementMaker (instead of original lxml.builder.E) to generate metadata xml E = lxml.builder.ElementMaker(namespace=KIMCHI_META_URL, nsmap={KIMCHI_NAMESPACE: KIMCHI_META_URL}) # follow the same build process made on _get_metadata_node() That way both xmls will be generated in same way <kimchi:kimchi xmlns:kimchi="http://github.com/kimchi-project/kimchi"> <kimchi:access> <kimchi:user>...</kimchi:user> ... <kimchi:group>...</kimchi:group> ... </kimchi:access> <kimchi:os> .... </kimchi:os> </kimchi> 3) Use FeatureTests.has_metadata_support() result to GET metadata element if FeatureTests.has_metadata_support(): return dom.metadata() else: # use lxml to return the metadata element IMPORTANT: While using setMetadata() you need to ALWAYS provide the whole metadata XML otherwise the metadata info will be override. On 04/24/2014 02:19 PM, shaohef(a)linux.vnet.ibm.com wrote: