1 Jul
2008
1 Jul
'08
4:43 p.m.
+ ret = cxml.define(options.ip) + if not ret: + logger.error("Failed to define the dom: %s", default_dom) + return FAIL + ret = cxml.start(options.ip) + if not ret: + logger.error("Failed to define the dom: %s", default_dom)
It's better to log the error as "logger.error("Failed to create(or start) the dom: %s", default_dom)", which more reflect the actual failure.
Yes, good catch - thanks! I had a copy/paste error here. I'll change it to be "Failed to start the dom: %s" since we're starting it (not creating it). -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com