[libvirt] [PATCH] Make int-overflow test case pass

Jim Meyering jim at meyering.net
Mon Mar 29 19:21:43 UTC 2010


Stefan Berger wrote:
> Jim Meyering <jim at meyering.net> wrote on 03/29/2010 02:34:06 PM:
>> Stefan Berger wrote:
>> > This fix makes the int-overflow test case pass.
>> >
>> > Signed-off-by: Stefan Berger <stefanb at us.ibm.com>
>> >
>> > Index: libvirt/tests/int-overflow
>> > ===================================================================
>> > --- libvirt.orig/tests/int-overflow
>> > +++ libvirt/tests/int-overflow
>> > @@ -14,6 +14,7 @@ fi
>> >  . "$srcdir/test-lib.sh"
>> >
>> >  echo "error: failed to get domain '4294967298'" > exp || fail=1
>> > +echo "error: Failed to start the nwfilter driver: Is the daemon
>> running ?" >> exp || fail=1
>> >  echo domname 4294967298 | $abs_top_builddir/tools/virsh --quiet \
>> >      --connect test://$abs_top_srcdir/examples/xml/test/testnode.xml \
>> >      > /dev/null 2> err || fail=1
>>
>> If we've already failed to get a domain, does it even
>> make sense to attempt to start the nwfilter driver?
>
> This would be a temporary fix. The root problem I think stems from there not
> being a test driver with name "nwfilter'.

As a temporary fix, that's fine.  Thanks.
It'd be nice to indicate that by adding a "FIXME" comment.




More information about the libvir-list mailing list