On 05/08/2016 11:16, "Joe Stringer" <[email protected]> wrote:
>On 4 August 2016 at 18:40, Daniele Di Proietto <[email protected]> wrote:
>> If the ethtool command is not available on the system we should fail,
>> since the userspace testsuite cannot work properly without disabling
>> offloads.
>>
>> Also, add ethtool to the list of installed packages on Vagrantfile.
>>
>> Fixes: ddcf96d2dcc1 ("system-tests: Disable offloads in userspace tests.")
>> Reported-by: Joe Stringer <[email protected]>
>> Signed-off-by: Daniele Di Proietto <[email protected]>
>
>Thanks, this should make it more obvious when offloads are causing failures.
>
>The commit message doesn't really explain why the vagrantfile change
>is in the same commit; a simple mention that it's being added here 'to
>ensure that offloads don't cause test failures in the vagrant VM when
>the kernel is updated' would make it more clear why these two changes
>are in the same commit.
Ok
>
>Acked-by: Joe Stringer <[email protected]>
Thanks! I applied this and the next patch to master
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
