[ovs-dev] [PATCH 3/3] Vagrantfile: add test_ovs_kmod provision

Andy Zhou azhou at nicira.com
Thu Jan 15 20:26:26 UTC 2015


On Thu, Jan 15, 2015 at 9:58 AM, Thomas Graf <tgraf at noironetworks.com> wrote:
> On 01/13/15 at 05:28pm, Andy Zhou wrote:
>> Add 'test_ovs_kmod' provision.  Command:
>>    # vagrant provision --provision-with=test_ovs_kmod
>>
>> will run "make check-kmod" in the vagrant launched VM.
>>
>> It turns out to be more convenient to test within as root. Switch
>> Vagrantfile provisions to run as root user.  However, vagrant can
>> still be launched as normal user.
>>
>> Signed-off-by: Andy Zhou <azhou at nicira.com>
>
> LGTM
>
> Acked-by: Thomas Graf <tgraf at noironetworks.com>
Thanks for the review.

>
> We might need to move the providers into the individual vm definitions
> when we start adding more distributions. Due to the outer is executed
> first rule that was fixed earlier.
Make sense.  I will make the change.

Do you have thoughts how we can support multiple kernel versions (for
testing compat code)
May be even multiple kernel versions over multiple distributions?



More information about the dev mailing list