Re: [ovs-dev] OVN - L3 Gap between NB schema and Neutron

Hi, I think Gal is saying the following - ml2 allows the following: VM (10.0.0.2) --- Logical_Switch (10.0.0.2) LogicalRouter | | | | VM (10.0.1.2) ++-(10.0.1.2) LogicalRouter

Re: [ovs-dev] [PATCH v2 1/3] ovn: Update TODO, ovn-northd flow table design, ovn-architecture for L3.

Hi, I have some initial questions in this design. > On Sep 30, 2015, at 1:56 PM, Ben Pfaff wrote: > > This is a proposed plan for logical L3 in OVN. It is not entirely > complete but it includes many important details and I believe that it moves > planning forward. > > +* L3

[ovs-dev] OVS/OVN: Interface with NAT

Hi Jarno, As part of OVN integration in Openstack, I’ve been trying to setup OpenFlow NAT rules and run the system-test-suite 33: system-traffic.at:1396 conntrack - simple SNAT. Using the latest (as of 12/15/2015) ovs master branch on Ubuntu Linux 4.3 kernels (from kernel.ubuntu.com

Re: [ovs-dev] OVS/OVN: Interface with NAT

> On Dec 16, 2015, at 4:42 PM, Jarno Rajahalme <ja...@ovn.org> wrote: > > >> On Dec 16, 2015, at 2:15 PM, Amitabha Biswas <azbis...@gmail.com >> <mailto:azbis...@gmail.com>> wrote: >> >> Hi Jarno, >> >> As part of OVN integration

Re: [ovs-dev] OVS/OVN: Interface with NAT

halme <ja...@ovn.org> wrote: > >> >> On Dec 16, 2015, at 2:15 PM, Amitabha Biswas <azbis...@gmail.com >> <mailto:azbis...@gmail.com>> wrote: >> >> Hi Jarno, >> >> As part of OVN integration in Openstack, I’ve been trying to setup Op

Re: [ovs-dev] OpenStack Proposal: Integration with OVN L3 Gateway

On Jun 8, 2016, at 7:18 AM, Brian Haley <brian.ha...@hpe.com> wrote: > > On 06/07/2016 04:50 PM, Amitabha Biswas wrote: >> This proposal outlines the modifications needed in networking-ovn (addresses >> https://bugs.launchpad.net/networking-ovn/+bug/1551717 >> <

Re: [ovs-dev] OpenStack Proposal: Integration with OVN L3 Gateway

y.com> wrote: > > On Tue, Jun 7, 2016 at 3:50 PM, Amitabha Biswas <azbis...@gmail.com> wrote: >> This proposal outlines the modifications needed in networking-ovn (addresses >> https://bugs.launchpad.net/networking-ovn/+bug/1551717 >> <https://bugs.launchpad.net/netwo

[ovs-dev] OpenStack Proposal: Integration with OVN L3 Gateway

This proposal outlines the modifications needed in networking-ovn (addresses https://bugs.launchpad.net/networking-ovn/+bug/1551717 ) to provide Floating IP (FIP) and SNAT using the L3 gateway router patches listed below:

Re: [ovs-dev] [PATCH] Function tracer to trace all function calls

Hi Nirapada, This patch doesn’t apply, please rebase on top of the master branch. The ovn-northd diff indicates deletion of gateway related code at the minimum. Thanks Amitabha > On Jun 9, 2016, at 11:12 AM, ngh...@us.ibm.com wrote: > > In some circumstances, we might need to figure out where

Re: [ovs-dev] [PATCH v8 1/2] ovn-controller: Add 'put_dhcp_opts' action in ovn-controller

One minor nit below - put_dchp_opts > > +/* > + * Parses the "put_dhcp_opts" action. > + * Format of the action is > + * - put_dchp_opts(R, offer_ip = , D1 = V1,..., Dn = Vn) > + * > + * Example: > + *put_dhcp_opts(reg0, offer_ip = 10.0.0.4, router = 10.0.0.1, > + *

[ovs-dev] [v1 1/1] Add new column compute_types to OVN_Southbound

he ports it is hosting matches the its preference. Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> --- ovn/controller/chassis.c| 62 - ovn/controller/ovn-controller.8.xml | 19 ovn/controller/ovn-controller.c | 12

[ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

he ports it is hosting matches the its preference. Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> --- ovn/controller/chassis.c| 62 - ovn/controller/ovn-controller.8.xml | 19 ovn/controller/ovn-controller.c | 12

Re: [ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

> On Jun 22, 2016, at 2:30 PM, Guru Shetty <g...@ovn.org> wrote: > > > > On 21 June 2016 at 12:20, Amitabha Biswas <azbis...@gmail.com > <mailto:azbis...@gmail.com>> wrote: > This patch allows a OVN hypervisor administator to specify the &g

Re: [ovs-dev] [PATCH v1] Add new column compute_types to OVN_Southbound

> On Jun 24, 2016, at 6:41 AM, Russell Bryant <russ...@ovn.org> wrote: > > > On Wed, Jun 22, 2016 at 5:55 PM, Amitabha Biswas <azbis...@gmail.com > <mailto:azbis...@gmail.com>> wrote: > > On Jun 22, 2016, at 2:30 PM, Guru Shetty <g...

Re: [ovs-dev] [PATCH] ovn-sbctl: Change lport-(un)bind to lsp-(un)bind.

/setup.sh > @@ -61,7 +61,7 @@ ovs-vsctl add-port br-int lport2 -- set Interface lport2 > external_ids:iface-id=p > ovs-vsctl add-port br-int lport5 -- set Interface lport5 > external_ids:iface-id=provnet1-5-101-port1 > ovs-vsctl add-port br-int lport6 -- set Interface lport

Re: [ovs-dev] OVN: Floating IP Support - Proposal

assumption. In such cases, there should not be a need for a L3 gateway (resulting in one less hop). The modified proposal can state this use case at the beginning of the proposal. Does this address your concern? Thanks Amitabha From: Han Zhou <zhou...@gmail.com> To: Amitabha Biswas/San J

[ovs-dev] OVN: Floating IP Support - Proposal

This is a proposal regarding how Floating IP will be supported in OpenStack using OVN. The complete proposal can also be found at https://review.openstack.org/#/c/274274/. The additions proposed to the OVN Northbound Schema and Southbound Flows are outlined here as well. The proposal assumes

Re: [ovs-dev] [RFC] OVN northbound address sets

It’s a great idea. From an OpenStack point of view, this will make NB ACL transaction sets O(1) instead of O(N) when combined with the delta ACL computation patch that is currently under review. I agree with Ryan’s SB extension. Since Address_Set will be a new table, the conditional_monitor

Re: [ovs-dev] weekly OVN report

I see a couple of distinct discussions occurring on this thread, maybe it’s time to deal with them independently. The Security/ACL aspect of the protocol - Is there some reason why existing SSL authentication and encryption mechanisms is not adequate enough for security. I am not opposed to

[ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

be used under such circumstances. If a logical datapath doesn't exist on a particular hypervisor, the ovn-controller will not apply any OpenFlow rules associated with that datapath and not update the cur_cfg. Use caution with the "--wait" option. Signed-off-by: Amitabha Biswas <

Re: [ovs-dev] [v1 1/1] Fix "raceful" E2E ovn tests

Thanks Jeff, 2035 : ovn -- 1 HVs, 2 LSs, 1 lport/LS, 1 LR I did have this test fail even without -j option (every 1 out around 25 runs) and the other tests involving Routers. Ben mentioned that this may be an artifact of the ARP tables not being populated on time, so I’d like his opinion on

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.6

Typo in the previous ack Acked-by: Amitabha Biswas <abis...@us.ibm.com <mailto:abis...@us.ibm.com>> > On Jul 26, 2016, at 4:22 PM, Amitabha Biswas <azbis...@gmail.com> wrote: > > I was able to compile the openvswitch modules on Linux 4.6 kernel and stacked >

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.6

I was able to compile the openvswitch modules on Linux 4.6 kernel and stacked using OpenStack networking-ovn. The basic NAT system tests passed and the OVN test suite passed. Asked-by: Amitabha Biswas <abis...@us.ibm.com <mailto:abis...@us.ibm.com>> > On Jul 26, 2016, at 2:07

Re: [ovs-dev] [PATCH v1] Enable manager_option configuation for NB and SB

that sounds reasonable? Thanks Amitabha > On Jul 27, 2016, at 11:45 AM, Ben Pfaff <b...@ovn.org> wrote: > > On Mon, Jul 18, 2016 at 10:21:09PM -0700, Amitabha Biswas wrote: >> NB and SB dbs are handled by separate ovsdb-server processes. The >> ovsdb-server processes mana

[ovs-dev] [PATCH v1] Python IDL Partial Set and Map Updates

multiple map updates on the same column in a transaction. 2. Partial map Py3 IDL test can now support multiple elements. 3. SetAttr overrides pre-existing insert and remove updates. 4. addvalue/delvalue contains unique elements Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> --- python/ovs/db/

[ovs-dev] [PATCH v1] Enable manager_option configuation for NB and SB

anager $manager_uuid inactivity_probe=0 ovs-vsctl --db=unix:/usr/local/var/run/openvswitch/ovnnb_db.sock list Manager ovs-vsctl --db=unix:/usr/local/var/run/openvswitch/ovnnb_db.sock --no-wait set Manager $manager_uuid inactivity_probe=5 Signed-off-by: Amitabha Biswas <abis...@us.ibm.com>

Re: [ovs-dev] [PATCH v1] Python IDL Partial Set and Map Updates

I must have overlooked removing that routine. It’s not needed. I’ll push v2. Thanks Amitabha > On Aug 15, 2016, at 9:22 AM, Ben Pfaff <b...@ovn.org> wrote: > > On Mon, Aug 15, 2016 at 02:21:55AM -0700, Amitabha Biswas wrote: >> This patch fixes a couple of bugs in commi

[ovs-dev] [PATCH v2] Python IDL Partial Set and Map Updates

multiple map updates on the same column in a transaction. 2. Partial map Py3 IDL test can now support multiple elements. 3. SetAttr overrides pre-existing insert and remove updates. 4. addvalue/delvalue contains unique elements Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> --- python/ovs/db/

Re: [ovs-dev] [PATCH v2] ovn: Delete stale MAC_Binding records

uot;Datapath_Binding", + "refType": "weak"}, + "min": 0}}}, "indexes": [["logical_port", "ip"]], "isRoot": true}, &

Re: [ovs-dev] [PATCH v3 2/5] ovn: Introduce l3 gateway router.

Hi Steve, Here is the link to the networking-ovn patch: https://review.openstack.org/#/c/346646/ Thanks Amitabha From: "Shi Xin Ruan" To: Gurucharan Shetty Cc: dev@openvswitch.org Date: 08/31/2016 10:51 PM Subject:Re: [ovs-dev] [PATCH

[ovs-dev] [PATCH] ovs-monitor-ipsec: Fix flake8-check semicolon error

--- debian/ovs-monitor-ipsec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/ovs-monitor-ipsec b/debian/ovs-monitor-ipsec index e6617b0..6bc26aa 100755 --- a/debian/ovs-monitor-ipsec +++ b/debian/ovs-monitor-ipsec @@ -333,7 +333,7 @@ class IPsec(object):

[ovs-dev] [PATCH v2] ovs-monitor-ipsec: Fix flake8-check semicolon error

Fixes: 19cd0a87 Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> Acked-by: Numan Siddique <nusid...@redhat.com> --- debian/ovs-monitor-ipsec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/debian/ovs-monitor-ipsec b/debian/ovs-monitor-ipsec index e6617b0..6b

Re: [ovs-dev] [PATCH] ovsdb-idlc: Fix memory leak in smap add and remove clause functions.

Hi Ben, This seems to take care of datums of type is_map, do we need to need to the same for type is_set()? Thanks Amitabha > On Aug 26, 2016, at 9:03 AM, Ben Pfaff wrote: > > Found by inspection. > > Signed-off-by: Ben Pfaff > --- > ovsdb/ovsdb-idlc.in | 4

Re: [ovs-dev] [PATCH v2] ovsdb-idlc: Fix memory leaks in add and remove clause functions.

v2 addresses the concern I had about v1. So… Acked-by: Amitabha Biswas <abis...@us.ibm.com <mailto:abis...@us.ibm.com>> > On Aug 26, 2016, at 2:18 PM, Ben Pfaff <b...@ovn.org> wrote: > > Found by inspection. > > Signed-off-by: Ben Pfaff <b...@ovn.org&

[ovs-dev] [PATCH v1] Python IDL Mutate Fix

s modified to use the named-uuid to identify a row created in the current transaction. Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> Suggested-by: Richard Theis <rth...@us.ibm.com> --- python/ovs/db/idl.py | 8 +++- tests/ovsdb-idl.at | 14 ++ tests/test-ov

[ovs-dev] [PATCH v1] Python-IDL: getattr after mutate fix

This commit returns the updated column value when getattr is done after a mutate operation is performed (but before the commit). Signed-off-by: Amitabha Biswas <azbis...@gmail.com> Reported-by: Richard Theis <rth...@us.ibm.com> Reported-at: http://openvswitch.org/pipermail/dev/20

Re: [ovs-dev] [PATCH v1] Python-IDL: getattr after mutate fix

Thanks! I reposted the patch with the update to the commit message. Amitabha > On Oct 12, 2016, at 1:16 PM, Russell Bryant <russ...@ovn.org> wrote: > > > > On Wed, Oct 12, 2016 at 10:32 AM, Amitabha Biswas <azbis...@gmail.com > <mailto:azbis...@gmail.com&

[ovs-dev] [PATCH v1] Python-IDL: getattr after mutate fix

This commit returns the updated column value when getattr is done after a mutate operation is performed (but before the commit). It addresses the bug reported in http://openvswitch.org/pipermail/dev/2016-September/080120.html Signed-off-by: Amitabha Biswas <abis...@us.ibm.com> Re