On 29/09/2015 18:51, "Ben Pfaff" <b...@nicira.com> wrote:

>On Tue, Sep 29, 2015 at 10:47:56AM -0700, Joe Stringer wrote:
>> On 28 September 2015 at 20:27, Ben Pfaff <b...@nicira.com> wrote:
>> > On Mon, Sep 28, 2015 at 07:36:37PM +0100, Daniele Di Proietto wrote:
>> >> Recently some testcases have been failing in travis because of a
>>warning
>> >> related to the use of an L3 device (OpenVZ specific) inside the
>>workers.
>> >>
>> >> To get travis tests working again we can move to the newer container
>> >> infrastructure: this commit does that.
>> >>
>> >> The disadvantage is that there's no sudo access anymore, but we can
>> >> install packages with the apt plugin, and we shouldn't use root for
>> >> anything else
>> >>
>> >> Tested-at: 
>>https://urldefense.proofpoint.com/v2/url?u=https-3A__travis-2Dci.org_ddip
>>roietto_ovs_builds_81764972&d=BQIBAg&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihV
>>MNtXt-uEs&r=SmB5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=kX6-y5h7zFnf0oq
>>XM0FwXDLDIyXS2_aY3aYfGXIGabE&s=Nv-s5jWvXQ4qwG47DtljWH_eRLqLgbDH5K-AFdf_f9
>>A&e= 
>> >> Signed-off-by: Daniele Di Proietto <diproiet...@vmware.com>
>> >> CC: Joe Stringer <joestrin...@nicira.com>
>> >
>> > Thanks for doing this!
>> 
>> +1 Appreciate you looking into this Daniele, keen to get back to stable
>>CI :-)
>> 
>> There's a few failures remaining, mostly in backports at the moment.

Yep, seems like one more function is needed.

>> This one looks different though:
>> 
>> 
>>https://urldefense.proofpoint.com/v2/url?u=https-3A__travis-2Dci.org_open
>>vswitch_ovs_jobs_82728444&d=BQIBAg&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMN
>>tXt-uEs&r=SmB5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=kX6-y5h7zFnf0oqXM
>>0FwXDLDIyXS2_aY3aYfGXIGabE&s=fGe217kY6orUy8l-mAH7drH5BoJ7TqHjGWI5kbZSl_U&
>>e= 
>> 
>> Seems that we're using 'bc' somewhere, but this isn't installed by
>> default in the travis environment. Perhaps should either figure out an
>> alternative way to do whatever we're using bc for, or install bc as
>> well.
>
>Based on the log, my guess is that that is happening somewhere in the
>kernel build system, so it's probably easiest to add bc to the packages
>we install.
>
>I don't see use of "bc" inside the OVS build system or tests.

I agree, we can install bc. I've sent a patch:

http://openvswitch.org/pipermail/dev/2015-September/060563.html

Sorry for all the noise.

Daniele

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to