> +
> + except Exception, detail:
> + logger.error("Exception: %s", detail)
> + status = FAIL
Instead of calling undefine before each call to raise Exception(), why
not put the undefine call in the except block? This means you'll only
need to call undefine() once for any exception. And if this test case
is extended in the future, it'll safe guard against an accidental
exclusion of the undefine() call when an exception occurs.
There is an exception raised if define fails. We don't want to call
undefine if the VM wasn't defined.
-Sharad