Re: [libvirt] [PATCH] tests: Test for nwfilter
Eric Blake wrote:
On 05/07/2010 03:39 PM, Stefan Berger wrote:
> This is a repost of a previously posted patch.
>
> Attached is a test for automatic testing of of the nwfilter rules as the
> are instantiated in form of ebtables, iptables and ip6tables rules on
> running VMs.
>
> The test automatically starts libvirtd from the build directory unless
> it finds libvirtd running.
This is an implied race.
All it takes is a parallel "make check" and two libvirtd-starting
tests started at approximately the same time.
But that's minor.
> My hope is that one won't notice this. It
> uses virsh from the build directory to create two dummy VMs with random
> name suffixes. The VMs don't boot any OS but just stop in the BIOS. This
> is enough to run the nwfilter tests. Afterwards the nwfilter of the one
> VM are continuously modified and the instantiation is checked. The
> instantiation of rules of the 2nd VM are also continously checked to
> verify that the modifications on the 1st VM has had no effect on the
> instantiated rules of the 2nd VM.
I'm still a bit wary of this patch. Is this something that can be done
with 'virsh -c test:///default' can do? Or can we at least copy how
daemon-conf runs an instance of libvirtd pointing to an independent pid
and config file, whether or not the system libvirtd is running?
Or should we be trying to do this as part of libvirt-tck instead?
I really like the idea of being able to test functionality like this
via a quick "make check", and using code in the same repository.
However, a test that's skipped whenever it detects a running
libvirtd is less valuable than one that runs unconditionally,
so libvirt-tck is probably the ideal destination.
It is critically important to have an easily accessible "make check"-
like process that can be used to exercise more of libvirt.
I hope libvirt-tck will soon fit the bill.