Thank you for this, it should be incredibly useful :)

Ed

On Fri, Oct 28, 2016 at 8:16 AM, Pierre Pfister (ppfister) <
[email protected]> wrote:

> Nailed it.
>
> The bug was only happening in setups with single working thread, which was
> not the case in my test setup.
> Test succeeded. The patch is ready to be merged.
>
> I am going to create a new vpp-sandbox directory called vhost-test.
> This sandbox effort intends to provide reference scripts for testing VPP
> vhost user, in the most user-friendly fashion (hopefully).
> Goal being to give pointers to people willing to test vhost user, but also
> to give reference parameters and configuration for people willing to use it.
>
> Thanks everyone,
>
> - Pierre
>
> Le 28 oct. 2016 à 10:55, Pierre Pfister (ppfister) <[email protected]> a
> écrit :
>
> So I tested in my setup with qemu from git with tags: v2.5.0-rc0
> v2.5.0-rc1 v2.5.0 v2.5.1
> They all work.
>
> Could you please have a look at the patch and give me some more logs ?
> VPP with VHOST_USER_DEBUG_SOCKET set to 1 usually helps a lot, as it logs
> all vhost messages received by VPP.
>
> Thanks,
>
> - Pierre
>
>
> Le 26 oct. 2016 à 23:45, Maciek Konstantynowicz (mkonstan) <
> [email protected]> a écrit :
>
> Excellent, thanks Ed! And let’s see …
>
> -Maciek
>
> On 26 Oct 2016, at 22:40, Edward Warnicke <[email protected]> wrote:
>
> Maciek,
>
> Looks like the patch was based at a point in the past that was pointing to
> the old CSIT branch.
>
> I rebased the patch, so hopefully it will now pick up the correct CSIT
> branch.
>
> Ed
>
> On Wed, Oct 26, 2016 at 2:15 PM, Maciek Konstantynowicz (mkonstan) <
> [email protected]> wrote:
>
>> //reduced list, updated subject
>>
>> Peter is right: https://gerrit.fd.io/r/#/c/2922/16
>>
>> looking at the last failed job - https://jenkins.fd.io/job/vp
>> p-csit-verify-virl-master/1940/consoleFull - it's using old csit branch
>> - "CSIT_BRANCH=oper-161017", without ubuntu1604, so pointless. you should
>> rebase to pick up this commit https://gerrit.fd.io/r/#/c/3553/ that's
>> updating to latest csit branch oper-161024.
>>
>> -Maciek
>>
>> On 26 Oct 2016, at 10:06, Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES
>> at Cisco) <[email protected]> wrote:
>>
>> Looks like it is still testing with oper-161017 not oper-161024.
>>
>> + CSIT_BRANCH=oper-161017
>> + git clone https://gerrit.fd.io/r/csit --branch oper-161017
>>
>> Can you please use the latest one (code rebase)?
>>
>> Thanks.
>>
>> *Peter Mikus*
>> Engineer – Software
>> *Cisco Systems Limited*
>>
>> Planned absence: 28.10., 1.11., 17.11., 9.12., 19.-31.12.
>>
>> *From:* Pierre Pfister (ppfister)
>> *Sent:* Wednesday, October 26, 2016 10:52 AM
>> *To:* Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) <
>> [email protected]>
>> *Cc:* Andrew Theurer <[email protected]>; Douglas Shakshober <
>> [email protected]>; Damjan Marion (damarion) <[email protected]>; Bill
>> Michalowski <[email protected]>; vpp-dev <[email protected]>; Rashid
>> Khan <[email protected]>; [email protected]; [email protected];
>> Edward Warnicke <[email protected]>; Thomas F Herbert <
>> [email protected]>
>> *Subject:* Re: [vpp-dev] updated ovs vs. vpp results for 0.002% and 0%
>> loss
>>
>> The recheck failed.
>> https://jenkins.fd.io/job/vpp-csit-verify-virl-master/1937/console
>>
>> Are you sure you updated qemu ?
>>
>> Thanks,
>>
>> - Pierre
>>
>>
>>
>>
>> _______________________________________________
>> csit-dev mailing list
>> [email protected]
>> https://lists.fd.io/mailman/listinfo/csit-dev
>>
>
>
>
> _______________________________________________
> vpp-dev mailing list
> [email protected]
> https://lists.fd.io/mailman/listinfo/vpp-dev
>
>
>
> _______________________________________________
> csit-dev mailing list
> [email protected]
> https://lists.fd.io/mailman/listinfo/csit-dev
>
_______________________________________________
vpp-dev mailing list
[email protected]
https://lists.fd.io/mailman/listinfo/vpp-dev

Reply via email to