
I would try to leverage the NetXML class here. You can have a function similar to dumpxml. You can also have functions similar to xml_get_net_mac(), xml_get_vcpus() etc. This will allow you to verify all the values of the XML, not just the IP.
I discussed this in detail with Deepti. We'd like to define several functions for parsing net pool values in NetXML class. But on the tc, we can only get NetXML string instead of NetXML class, so we can not call the parsing functions. Do you know how to fix this? Now, the only options is to use dump function outside of NetXML, and then parse the values in the str...
I would make the init() function of NetXML take a flag that allows you to specify whether the network is a new one or not. If the network isn't a new one, then you can dump the XML directly (instead of building the XML, like what's currently being done in the init() function). Having a flag like this isn't really ideal, but there's not an easy way to tell NetXML whether you want to init a completely new NetXML object or if you want to just get the XML of an existing object. -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com