What I was thinking is to supply the timeout param and maintain a
static
like variable( that is present in C) and use this static like variable
to compare in check_guest_state().
We can achieve the static behaviour in python by declaring a global
variable in the check_guest_state() function. But, I think using globals
is highly discouraged ?
Yes, I think it is best to avoid global variables. It makes the code
execution more confusing, and they are generally considered less safe.
> The test case should be responsible for printing its own error
message
> in the case of a failure, so changing this to info messages shouldn't
> be too problematic.
>
> The only problem it causes is that when you run with -debug, the info
> messages won't print to the terminal, so you'll have to check the
> individual log to see why the test failed.
I think it is better to leave this as it is for now. Since someone might
forget to ON the --debug option to see the log messages.
Using the debug option doesn't print info messages to the console.
Error messages are printed, info messages are not.
I'll continue to do some more brainstorming on this issue.
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com