Deepti B Kalakeri wrote:
Kaitlin Rupert wrote:
+def verify_err_fields(cxml, server, conn, keys, classname, + assoc_classname, msg, field): + try: + ret = try_assoc(conn, classname, assoc_classname, keys, + field_name=field, expr_values=expr_values[field], + bug_no="") + if ret != PASS: + if classname == 'Linux_ComputerSystem': + return XFAIL_RC(bug_sblim) + else: + logger.error("--- FAILED: %s---", msg) + cxml.destroy(server)
You also need cxml.undefine() here. I removed the cxml.undefine() since cxml.destroy() is just sufficient to destroy the domain that was created using cxml.create(). I had verified this before removing the cxml.undefine(), let me know if you still think we need to include cxml.undefine().
Ah, good point. My mistake. =) Your patch is fine as is. I'll apply =) -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com