Re: [git pull request] kvm-autotest: sync with upstream Autotest SVN

[Lucas Meneghel Rodrigues <lmr@redhat.com>]

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