Hi Uri! Thank you very much, I've identified the problems you mentioned and made patches to fix them. They will follow shortly. On Sun, 2009-06-07 at 12:06 +0300, Uri Lublin wrote: > Hello, > > I ran some tests using the newly merged kvm-autotest. > Results summary: 148 tests passed (out of 250) > > Following are some things that got broken in the merge, which are easy to fix: > 1. kvm_config.py as a stand-alone program. > - please do not throw autotest-exceptions (such as error.TestError) in this > file. > 2. make_html_report.py > - missing \ at the end of quoted line. An example is: > The code: > a = "first value %s second value %d" % (val1 ,val2) > Was replace with: > a = "first value %s" > "second value %d" % (val1, val2) > The second line is broken. > To fix just add \ at the end of the first line (+ indentation). > > 3. Running autotest tests on KVM guests: > Same problem as in make_html_report above. > > 4. Change to python logging: (at least in kvm_utils.py) > need to replace logging.debug(msg, additional_string) with logging.debug(msg > + additional_string) > > Regards, > Uri > -- Lucas Meneghel Rodrigues Software Engineer (QE) Red Hat - Emerging Technologies -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
