[Libvirt-cim] [PATCH] [TEST] Update 32_start_reboot.py to use cim_() functions
Kaitlin Rupert
kaitlin at linux.vnet.ibm.com
Wed Jan 7 18:08:18 UTC 2009
> I get your point here, but still it left me confused as to why I am
> getting the Error Messages.
> It would be good to change the Error messages to Info messages whenever
> appropriate.
> We can use the timeout period to decide whether it is time to print an
> error message in case the guest info is not found or the states are not
> yet properly.
> Any thoughts ??
>
Using the timeout won't work because it is check_guest_state() that is
printing the error messages, not cim_state_change(). I'm going to
submit a patch that changes the log message from error to info.
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.
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com
More information about the Libvirt-cim
mailing list