[libvirt] [test-API PATCH] Added screenshot test

Guannan Ren gren at redhat.com
Tue Apr 3 15:18:21 UTC 2012


On 04/03/2012 09:34 PM, Peter Krempa wrote:
> On 04/02/2012 08:18 PM, Martin Kletzander wrote:
>> This patch adds a test that obtains a screenshot of a domain and saves
>> it in a file.
>
> Maybe we shoud later on modify this or add another test to allow 
> comparing the returned image file against a pre-defined pattern, so we 
> could actualy check if the complete stack including the OS, qemu, 
> video drivers and libvirt's streams work.
>
> Looks good, but I'm not a very experienced Pythonist, so if somebody 
> else (Guannan?) could have a quick look and confirm my ACK before 
> pushing.
>
> Peter
>
> -- 
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

          Thanks for the review.
          pushed.

          Guannan Ren




More information about the libvir-list mailing list