[libvirt] [PATCH v2] test: fix nwfilter tests following changes in virfirewall.c

Martin Kletzander mkletzan at redhat.com
Thu Nov 27 08:37:27 UTC 2014

On Wed, Nov 26, 2014 at 01:18:53PM -0500, Stefan Berger wrote:
>On 11/26/2014 11:53 AM, Eric Blake wrote:
>> On 11/26/2014 09:26 AM, Stefan Berger wrote:
>>> Some of the nwfilter tests are now failing since --concurrent shows
>>> up in the ebtables command. To avoid this, implement a function
>>> preventing the probing for lock support in the eb/iptables tools
>>> and use it in the tests.
>> Now that I've read Martin and Prerna's exchange, I'm wondering if we
>> should instead make this override force the locking flags ON, and adjust
>> the expected test output to expect the -w/--concurrent.
>Not sure... this patch is definitely 'simpler' and that one call we add
>to virsocket.h would only to be called by test cases. What we would test
>if we considered the --concurrent and -w flag is basically not much,
>just that some probing happened.

I must say I agree with Stefan (or at least for now).  I started
working on the second variant (forcing the flag "on" for tests), and
not only I've found out that the tests themselves (not talking about
the outputs) are not adapted to that, but even though it's still not
fully fixed, I already have a diff of 1051(+), 1014(-).

I can finish it or at least share somewhere if someone else wants to

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20141127/6604d39d/attachment-0001.sig>

More information about the libvir-list mailing list