Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-04-02 Thread Ofer Ben-Yacov
OK. I will. Ofer. On Fri, Apr 1, 2016 at 6:31 PM Ben Pfaff wrote: > On Mon, Mar 28, 2016 at 06:25:51AM +0000, Ofer Ben-Yacov wrote: > > I understand that you met Ayal for a short talk during ONS. > > From what he told me I think there might be some misunderstanding so I > &g

Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-03-27 Thread Ofer Ben-Yacov
e just need to verify that the schema version hasn't changed. Does this make sense? BR, Ofer On Tue, Mar 8, 2016 at 8:41 AM Ofer Ben-Yacov wrote: > Hi Ben, > > I'm still waiting for you response. > Do you see the reason for the current usage of IDL, where Neutron and my &g

Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-03-07 Thread Ofer Ben-Yacov
t *wants*, not on the >> schema that the database server *has*. That's why it reads the schema >> from a local disk file by default. If Neutron does something else then >> it should probably be changed. >> >> On Mon, Feb 29, 2016 at 08:06:33AM +

Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-03-01 Thread Ofer Ben-Yacov
ething else then > it should probably be changed. > > On Mon, Feb 29, 2016 at 08:06:33AM +, Ofer Ben-Yacov wrote: > > Take a loot at the contractor of IDL: > > > > ... > > def __init__(self, remote, schema): > > ... > > > > > > You

Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-02-29 Thread Ofer Ben-Yacov
faff‬‏ <‪b...@ovn.org‬‏>:‬ > On Wed, Feb 24, 2016 at 08:39:02AM +0000, Ofer Ben-Yacov wrote: > > The current implementation IDL is: > > 1. connects to the OVSDB > > 2. get the schema and > > 3. disconnect > > 4. connect again to work with the OVSDB server. > >

Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-02-24 Thread Ofer Ben-Yacov
k just the same in this regard for active and passive connections? > > Thanks, > > Ben. > > On Wed, Feb 17, 2016 at 05:22:19PM +0200, Ofer Ben Yacov wrote: > > From: Ofer Ben-Yacov > > > > Currently the IDL does not support passive TCP connection, > > i.e.

[ovs-dev] Fwd: [PATCH V6] Add Passive TCP connection to IDL.

2016-02-23 Thread Ofer Ben-Yacov
Hi Russel. Did you have a chance to check V6 patch? This should fix the testing problem. Ofer. -- Forwarded message - From: Ofer Ben Yacov ‪Date: יום ד׳, 17 בפבר׳ 2016 ב-17:21‬ Subject: [PATCH V6] Add Passive TCP connection to IDL. To: From: Ofer Ben-Yacov Currently the

[ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-02-17 Thread Ofer Ben Yacov
From: Ofer Ben-Yacov Currently the IDL does not support passive TCP connection, i.e. when the OVSDB connects to its manager. This patch enables IDL to use an already-open session (the one which was previously used for retrieving the db schema). In addition, it enables IDL to go back to "l

Re: [ovs-dev] [ovs-dev,V5] Add Passive TCP connection to IDL.

2016-02-17 Thread Ofer Ben-Yacov
; > > > ‫בתאריך יום ג׳, 16 בפבר׳ 2016 ב-16:47 מאת ‪Russell Bryant‬‏ <‪ > rbry...@redhat.com‬‏>:‬ > >> On 02/16/2016 09:40 AM, Ofer Ben-Yacov wrote: >> > looking at the tests , I see that the only test that failed is for >> > Kernel 3.18.1. >> > I&#x

Re: [ovs-dev] [ovs-dev,V5] Add Passive TCP connection to IDL.

2016-02-16 Thread Ofer Ben-Yacov
09:40 AM, Ofer Ben-Yacov wrote: > > looking at the tests , I see that the only test that failed is for > > Kernel 3.18.1. > > I'm trying to find vagrant box with this kernel to test it. > > > > I see in the tests that you are not running them on Kernel 3.16.7, which

Re: [ovs-dev] [ovs-dev,V5] Add Passive TCP connection to IDL.

2016-02-16 Thread Ofer Ben-Yacov
me. Ofer. ‫בתאריך יום ב׳, 15 בפבר׳ 2016 ב-22:32 מאת ‪Russell Bryant‬‏ <‪ russ...@ovn.org‬‏>:‬ > On 02/15/2016 09:28 AM, Russell Bryant wrote: > > On 02/14/2016 07:10 AM, Ofer Ben-Yacov wrote: > >> > >> > >> The make check failed after g

Re: [ovs-dev] [ovs-dev,V5] Add Passive TCP connection to IDL.

2016-02-14 Thread Ofer Ben-Yacov
1677 failed Something on your side?! Ofer. ‫בתאריך יום ו׳, 12 בפבר׳ 2016 ב-21:04 מאת ‪Russell Bryant‬‏ <‪ rbry...@redhat.com‬‏>:‬ > On 02/09/2016 04:25 AM, Ofer Ben-Yacov wrote: > > From: Ofer Ben-Yacov > > > > Currently the IDL does not support passi

Re: [ovs-dev] [PATCH V2] Add tunnel key to Physical_Locator

2016-02-06 Thread Ofer Ben-Yacov
It does. Thanks, Ofer. ‫בתאריך יום ו׳, 5 בפבר׳ 2016 ב-22:57 מאת ‪Ben Pfaff‬‏ <‪b...@ovn.org‬‏>:‬ > On Thu, Feb 04, 2016 at 09:49:35AM +0000, Ofer Ben-Yacov wrote: > > The tunnel key in decoupled from the actual implementation of the tunnel. > > The only tunnel proto

Re: [ovs-dev] [PATCH V2] Add tunnel key to Physical_Locator

2016-02-04 Thread Ofer Ben-Yacov
ty to use different keys in multiple tunnels in the same logical switch. Ofer. ‫בתאריך יום ה׳, 4 בפבר׳ 2016 ב-1:42 מאת ‪Ben Pfaff‬‏ <‪b...@ovn.org‬‏>:‬ > On Tue, Dec 29, 2015 at 02:36:56PM +0200, Ofer Ben Yacov wrote: > > Add tunnel key to Physical_Locator to support differe

Re: [ovs-dev] [PATCH ] Add tunnel key to Physical_Locator

2016-02-02 Thread Ofer Ben-Yacov
Hi. I added the key to the index. Can you please review again. BR, Ofer. ‫בתאריך יום ד׳, 23 בדצמ׳ 2015 ב-8:07 מאת ‪Ben Pfaff‬‏ <‪b...@ovn.org‬‏>:‬ > On Sun, Dec 06, 2015 at 06:04:45PM +0200, ofer.benya...@gmail.com wrote: > > From: Ofer Ben Yacov > > > > Curre

Re: [ovs-dev] [PATCH V4] Add Passive TCP connection to IDL.

2016-01-26 Thread Ofer Ben-Yacov
: > On 01/25/2016 04:09 AM, ofer.benya...@gmail.com wrote: > > From: Ofer Ben-Yacov > > > > Currently the IDL does not support passive TCP connection, > > i.e. when the OVSDB connects to its manager. > > > > This patch enables IDL to use an already-open session

[ovs-dev] [PATCH V3] Add Passive TCP connection to IDL

2015-12-31 Thread Ofer Ben Yacov
test for passive mode. See ovsdb-idl.at file. Signed-off-by: "Ofer Ben-Yacov" Tested-by: "Ofer Ben-Yacov" Requested-by: Ben Pfaff , "D M, Vikas" , "Kamat, Maruti Haridas" , "Sukhdev Kapur" , "Migliac

[ovs-dev] [PATCH V2] Add tunnel key to Physical_Locator

2015-12-29 Thread Ofer Ben Yacov
Add tunnel key to Physical_Locator to support different tunnel ID on different locators on the same logical switch. --- vtep/vtep.ovsschema | 7 --- vtep/vtep.xml | 22 -- 2 files changed, 16 insertions(+), 13 deletions(-) diff --git a/vtep/vtep.ovsschema b/vtep

[ovs-dev] [PATCH V3] Add Passive TCP connection to IDL

2015-12-28 Thread Ofer Ben Yacov
Add unit test for passive mode. --- python/ovs/db/idl.py | 18 +++--- python/ovs/jsonrpc.py | 19 +++ python/ovs/stream.py | 47 +++ tests/ovsdb-idl.at| 31 +++ tests/test-ovsdb.py | 49 +++

[ovs-dev] [PATCH v2] Add Passive TCP connection to IDL

2015-12-20 Thread Ofer Ben Yacov
From: Ofer Ben-Yacov Currently the IDL does not support passive TCP connection, i.e. when the OVSDB connects to its manager. This patch enables IDL to use an already-open session (the one which was previously used for retrieving the db schema). In addition, it enables IDL to go back to "l

Re: [ovs-dev] [PATCH] Add Passive TCP connection to IDL

2015-12-20 Thread Ofer Ben-Yacov
it the new code with the modifications again. Ofer. On Thu, Dec 17, 2015 at 11:48 PM, Russell Bryant wrote: > On 12/17/2015 04:37 AM, Ofer Ben Yacov wrote: > > From: Ofer Ben-Yacov > > > > Currently the IDL does not support passive TCP connection, > > i.e. w

[ovs-dev] [PATCH] Add Passive TCP connection to IDL

2015-12-17 Thread Ofer Ben Yacov
From: Ofer Ben-Yacov Currently the IDL does not support passive TCP connection, i.e. when the OVSDB connects to its manager. This patch enables IDL to use an already-open session (the one which was previously used for retrieving the db schema). In addition, it enables IDL to go back to "l

Re: [ovs-dev] [PATCH] python/ovs: Add passive TCP connection to IDL

2015-12-02 Thread Ofer Ben-Yacov
Thanks Russell. I did use Gmail while the instruction in the link you sent me says that gmail does not work for patches. I will resend using git send-mail command. Regards, Ofer. On Wed, Dec 2, 2015 at 6:50 PM, Russell Bryant wrote: > On 12/02/2015 09:49 AM, Ofer Ben-Yacov wrote: > >

[ovs-dev] [PATCH] python/ovs: Add passive TCP connection to IDL

2015-12-02 Thread Ofer Ben-Yacov
TCP to the OVSDB, a helper program was used. The program flow is: 1. Open TCP connection to OVSDB 2. Open TCP connection to the agent application that uses IDL 3. Forward data between connections (Proxy) 4. Simulate a connection problem by stopping and restarting the program Signed-off-by: Ofer Be

[ovs-dev] python/ovs: Add passive TCP connection to IDL

2015-12-01 Thread Ofer Ben-Yacov
and then re-started. Signed-off-by: Ofer Ben-Yacov Tested-by: Ofer Ben-Yacov Requested-by: Ben Pfaff , "D M, Vikas" , "Kamat, Maruti Haridas" , Sukhdev Kapur mailto:sukh...@arista.com>>,"Migliaccio, Armando" ___