Re: [Autotest] [PATCH] Add a client-side test qemu_iotests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2009-07-06 at 13:17 +0800, Yolkfull Chow wrote:

> Hi Lucas, since the qemu_iotests test suite itself has checked qemu,
> qemu-img and qemu-io ELF files before running, I would skip the
> duplicated step.

Fair enough. Making the check on the autotest module would make things
more apparent to the user (we would throw a TestFailure with the
reason), but I see your point.

> I added 'qemu_path' option for users to specify their alternative PATH
> containing these ELF files. 
> 
> > +        if qemu_path: 
> > +            os.environ['PATH'] += ":" + qemu_path 
> > + 

Good.

Martin, any comments? The test module and the qemu-iotests suite are
looking good, can we commit it?

Lucas

--
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

[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux