<div dir="ltr">There are two functions which read a VM metadata on Kimchi: libvirt_get_kimchi_metadata_node (the one you changed) and _kimchi_get_metadata_node. Both functions do the same thing but in different ways: libvirt_get_kimchi_metadata_node uses the libvirt metadata API to fetch the metadata info from a VM (= easier to use), and _kimchi_get_metadata_node reads the metadata info from the VM's XML (= harder to use). As you can see in get_metadata_node, the first function is called when the host system uses a newer version of Kimchi which supports the metadata API, and the second function is used otherwise.<div><br></div><div>Your patch is changing libvirt_get_kimchi_metadata_node so it parses the XML _and_ uses the metadata API to get the metadata info.<span style="line-height:1.5;font-size:13.1999998092651px"> If we want to avoid using the metadata API (because it always prints messages when a VM has no metadata, which is what this issue is about) we should then use the function _kimchi_get_metadata_node every time.</span></div><div><br></div><div>Also, we always use the Python lxml API to read XML data. We used to have mixed APIs in the past and it was hard to maintain, so we decided to use the same one in order to keep consistency.<br><br></div></div><br><div class="gmail_quote">On Fri, Apr 24, 2015 at 3:04 PM Ramon Medeiros <<a href="mailto:ramonn@linux.vnet.ibm.com">ramonn@linux.vnet.ibm.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Ramon Medeiros <<a href="mailto:ramonn@jarvis.br.ibm.com" target="_blank">ramonn@jarvis.br.ibm.com</a>><br>
<br>
The function FeatureTests.disable_libvirt_error_logging only removes the<br>
error message that will be displayed on the console. Libvirt also logs<br>
errors on messages log. To avoid flood messages log with metadata<br>
errors, it will be verified manually if the xml has the tag.<br>
<br>
Signed-off-by: Ramon Medeiros <<a href="mailto:ramonn@linux.vnet.ibm.com" target="_blank">ramonn@linux.vnet.ibm.com</a>><br>
---<br>
src/kimchi/model/utils.py | 8 ++++++++<br>
1 file changed, 8 insertions(+)<br>
<br>
diff --git a/src/kimchi/model/utils.py b/src/kimchi/model/utils.py<br>
index b2739b2..749be93 100644<br>
--- a/src/kimchi/model/utils.py<br>
+++ b/src/kimchi/model/utils.py<br>
@@ -22,6 +22,7 @@ import socket<br>
import urlparse<br>
from lxml import etree, objectify<br>
from lxml.builder import E, ElementMaker<br>
+from xml.dom import minidom<br>
<br>
from kimchi.exception import OperationFailed<br>
from kimchi.model.featuretests import FeatureTests<br>
@@ -95,6 +96,13 @@ def _kimchi_set_metadata_node(dom, node):<br>
<br>
<br>
def libvirt_get_kimchi_metadata_node(dom, mode="current"):<br>
+ # check xml for metadata tag<br>
+ xml = minidom.parseString(dom.XMLDesc(0))<br>
+<br>
+ # no metadata: return<br>
+ if not xml.getElementsByTagName('metadata'):<br>
+ return None<br>
+<br>
FeatureTests.disable_libvirt_error_logging()<br>
try:<br>
xml = dom.metadata(libvirt.VIR_DOMAIN_METADATA_ELEMENT,<br>
--<br>
2.1.0<br>
<br>
_______________________________________________<br>
Kimchi-devel mailing list<br>
<a href="mailto:Kimchi-devel@ovirt.org" target="_blank">Kimchi-devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/kimchi-devel" target="_blank">http://lists.ovirt.org/mailman/listinfo/kimchi-devel</a><br>
</blockquote></div>