[ovs-dev] Question on L3 Logical Router in vtep-emulator

2016-05-26 Thread Kyle Mestery
Greetings! I was looking for some L3 related items and came across the following patch on patchworks [1]. This was some work done to add L3 routing to the vtep emulator. Guru had commented on this indicating the usefulness for using the vtep-emulator as a L3 edge device, but he had a bunch of comm

Re: [ovs-dev] Question on L3 Logical Router in vtep-emulator

2016-05-26 Thread Kyle Mestery
On Thu, May 26, 2016 at 9:44 AM, Guru Shetty wrote: > > On 26 May 2016 at 07:13, Kyle Mestery wrote: >> >> Greetings! >> >> I was looking for some L3 related items and came across the following >> patch on patchworks [1]. This was some work done to add L3 r

Re: [ovs-dev] Question on L3 Logical Router in vtep-emulator

2016-05-26 Thread Kyle Mestery
On Thu, May 26, 2016 at 10:20 AM, Guru Shetty wrote: > > > On 26 MaVTEP schema based L3 gateway in particular? >> >> > >> Thanks Guru. I was looking at the VTEP schema based L3 GW as well, and >> wondering if this patch had pretty much been abandoned. I can see some >> use for using VTEP as an L3

Re: [ovs-dev] Question on L3 Logical Router in vtep-emulator

2016-05-26 Thread Kyle Mestery
On Thu, May 26, 2016 at 10:46 AM, Guru Shetty wrote: > >> >> >> OK, I'm going to play with this a bit. I was mostly interested in >> feedback from others. We're interested in this approach and so if we >> take the patch up does it stand a chance of merging? > > > If I remember correctly, the main

Re: [ovs-dev] SFC-Summary: MultiTenant

2016-07-12 Thread Kyle Mestery
On Tue, Jul 12, 2016 at 9:52 AM, Russell Bryant wrote: > On Tue, Jun 28, 2016 at 12:05 PM, Ryan Moats wrote: > >> John McDowall wrote on 06/28/2016 >> 10:54:31 >> AM: >> >> > From: John McDowall >> > To: Ryan Moats/Omaha/IBM@IBMUS, Na Zhu >> > Cc: "dev@openvswitch.org" >> > Date: 06/28/2016 1

Re: [ovs-dev] SFC-Summary: MultiTenant

2016-07-13 Thread Kyle Mestery
On Tue, Jul 12, 2016 at 6:57 PM, Cathy Zhang wrote: > Hi Kyle/Russell, > > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Kyle Mestery > Sent: Tuesday, July 12, 2016 9:19 AM > To: Russell Bryant > Cc: dev@openvswitch.org; John McDowal

Re: [ovs-dev] SFC-Summary: MultiTenant

2016-07-13 Thread Kyle Mestery
viding multi-tenancy for VNFs using the chain, and you need to build chains (and VNFs) per-tenant. > On Tue, Jul 12, 2016 at 9:18 AM, Kyle Mestery wrote: >> >> On Tue, Jul 12, 2016 at 9:52 AM, Russell Bryant wrote: >> > On Tue, Jun 28, 2016 at 12:05 PM, Ryan Moats wrote: &

Re: [ovs-dev] [PATCH] ovn: Remove a completed TODO item.

2016-07-18 Thread Kyle Mestery
have adequate features for OVN but it probably > -- > 2.7.4 > This one is simple enough, thanks Russell! Acked-By: Kyle Mestery > ___ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev __

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

2016-07-19 Thread Kyle Mestery
On Tue, Jul 19, 2016 at 12:21 AM, Amitabha Biswas wrote: > NB and SB dbs are handled by separate ovsdb-server processes. The > ovsdb-server processes manage dbs based on the schemas for OVN NorthBound > and SouthBound. These schemas do not have any Manager Options similar > to the Open_vSwitch sch

Re: [ovs-dev] [PATCH] ovn: Rename "gateway" to "l3gateway".

2016-07-26 Thread Kyle Mestery
This seems very reasonable to me Russell, and makes sense given the two disparate gateway port types. Acked-by: Kyle Mestery > --- > ovn/controller/binding.c | 2 +- > ovn/controller/patch.c | 4 ++-- > ovn/northd/ovn-northd.c | 12 ++-- > ovn/ovn-sb.xml

[ovs-dev] [PATCH] doc: Update INSTALL.Docker.md to reflect it's focus on OVN

2016-07-27 Thread Kyle Mestery
more accurately reflect what the document is talking about. Signed-off-by: Kyle Mestery --- INSTALL.Docker.md | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/INSTALL.Docker.md b/INSTALL.Docker.md index b7ca4ba..08758ae 100644 --- a/INSTALL.Docker.md

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Kyle Mestery
On Sun, Jul 31, 2016 at 7:40 PM, Ryan Moats wrote: > Ben Pfaff wrote on 07/31/2016 11:53:15 AM: > >> From: Ben Pfaff >> To: Ryan Moats/Omaha/IBM@IBMUS >> Cc: dev@openvswitch.org, Kyle Mestery/Silicon Valley/IBM@IBMUS >> Date: 07/31/2016 11:53 AM >> Subject:

Re: [ovs-dev] Read only versions of the *ctl binaries

2016-08-01 Thread Kyle Mestery
On Mon, Aug 1, 2016 at 12:49 PM, Ben Pfaff wrote: > On Mon, Aug 01, 2016 at 12:00:17PM -0500, Ryan Moats wrote: >> When it comes to ovs-appctl, we're looking to set log level access only. >> Since this doesn't really fit into what I think of when I see "--dry-run", >> I'm wondering if this variati

Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

2016-08-02 Thread Kyle Mestery
On Tue, Aug 2, 2016 at 12:13 PM, Ryan Moats wrote: > > Russell Bryant wrote on 08/02/2016 12:00:08 PM: > >> From: Russell Bryant >> To: Ben Pfaff >> Cc: Ryan Moats/Omaha/IBM@IBMUS, ovs dev >> Date: 08/02/2016 12:00 PM >> Subject: Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 10:30 AM, Ryan Moats wrote: > > Russell Bryant wrote on 08/03/2016 10:11:57 AM: > >> From: Russell Bryant >> To: Ryan Moats/Omaha/IBM@IBMUS >> Cc: Ben Pfaff , ovs-dev >> Date: 08/03/2016 10:12 AM >> Subject: Re: [ovs-dev] Let's talk the NB DB IDL Part I - things >> we've

Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 7:35 AM, Russell Bryant wrote: > > On Tue, Aug 2, 2016 at 1:16 PM, Kyle Mestery wrote: >> >> On Tue, Aug 2, 2016 at 12:13 PM, Ryan Moats wrote: >> > >> > Russell Bryant wrote on 08/02/2016 12:00:08 PM: >> > >> >>

Re: [ovs-dev] [PATCH v1] ovn: Extend Address_Set to Macros to support define port name sets

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 11:47 AM, Russell Bryant wrote: > On Wed, Aug 3, 2016 at 11:14 AM, Zong Kai LI wrote: > >> This patch aims to extend Address_Set to Macros, make it more common to >> accept variable set, not only address. And by that, we can skinny down >> ACLs, >> if we use Macros to defin

[ovs-dev] [PATCH] ovs-vsctl: Change log level of vsctl_parent_process_info

2016-08-04 Thread Kyle Mestery
/openvswitch/ovn-scale-test [2] https://github.com/openvswitch/ovn-scale-test/blob/master/rally_ovs/plugins/ovs/scenarios/ovn.py#L255 Signed-off-by: Kyle Mestery --- utilities/ovs-vsctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-vsctl.c b/utilities/ovs

Re: [ovs-dev] [PATCH] ovs-vsctl: Change log level of vsctl_parent_process_info

2016-08-04 Thread Kyle Mestery
On Thu, Aug 4, 2016 at 12:56 PM, Andy Zhou wrote: > > > On Thu, Aug 4, 2016 at 6:54 AM, Kyle Mestery wrote: >> >> While running the ovn-scale-test [1] port-binding tests [2], I notice a >> continual stream of messages such as this: >> &g

Re: [ovs-dev] [PATCH v3 1/2] ovsdb: Add/use partial set updates.

2016-08-05 Thread Kyle Mestery
o 2.6, because for large deployments, this is a huge performance win! Note, I have not verified the OpenStack portion of this yet, this was purely with the OVN control plane. Tested-by: Kyle Mestery > Signed-off-by: Ryan Moats > --- > lib/automake.mk | 2 + > li

[ovs-dev] [PATCH] ovs-vsctl: Change log level of vsctl_parent_process_info

2016-08-08 Thread Kyle Mestery
. [1] https://github.com/openvswitch/ovn-scale-test [2] https://github.com/openvswitch/ovn-scale-test/blob/master/rally_ovs/plugins/ovs/scenarios/ovn.py#L255 Signed-off-by: Kyle Mestery --- V1->V2 Remove log message instead of changing the level from WARN to DBG. --- utilities/ovs-vsctl.c

Re: [ovs-dev] [PATCH] ovs-vsctl: Change log level of vsctl_parent_process_info

2016-08-08 Thread Kyle Mestery
On Thu, Aug 4, 2016 at 3:38 PM, Andy Zhou wrote: > > > On Thu, Aug 4, 2016 at 11:39 AM, Kyle Mestery wrote: >> >> On Thu, Aug 4, 2016 at 12:56 PM, Andy Zhou wrote: >> > >> > >> > On Thu, Aug 4, 2016 at 6:54 AM, Kyle Mestery >> > wro

Re: [ovs-dev] [PATCH] ovs-vsctl: Change log level of vsctl_parent_process_info

2016-08-08 Thread Kyle Mestery
On Mon, Aug 8, 2016 at 2:47 PM, Andy Zhou wrote: > > > On Mon, Aug 8, 2016 at 6:48 AM, Kyle Mestery wrote: >> >> While running the ovn-scale-test [1] port-binding tests [2], I notice a >> continual stream of messages such as this: >> &g

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

2016-06-08 Thread Kyle Mestery
On Tue, Jun 7, 2016 at 3:50 PM, Amitabha Biswas wrote: > 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 usin

Re: [ovs-dev] [PATCH] ovs-bugtool: Fix flake8 errors.

2016-06-09 Thread Kyle Mestery
") > Signed-off-by: Russell Bryant > --- Thanks Russell! Acked-By: Kyle Mestery > utilities/bugtool/ovs-bugtool.in | 48 > ++-- > 1 file changed, 26 insertions(+), 22 deletions(-) > > diff --git a/utilities/bugtool/ovs-bugtool.in

Re: [ovs-dev] [PATCH v6 1/2] Add address set support.

2016-06-30 Thread Kyle Mestery
y to define address sets for use. > > Signed-off-by: Russell Bryant > Co-authored-by: Babu Shanmugam > Signed-off-by: Babu Shanmugam Tested-by: Kyle Mestery I've run these through the ovn-scale-test CI setup [1]. We have a test which creates ports and puts ACLs on them. This is

Re: [ovs-dev] [PATCH] [RFC Patch] ovn-controller: ignore lflow matching remote VM port

2016-07-07 Thread Kyle Mestery
On Thu, Jul 7, 2016 at 12:07 PM, Zong Kai Li wrote: > Currently, ovn-controller will install all lflows for a logical > switch, when ovn-controller determines not to skip processing of > that logical switch. > > This will install too many OVS flows. We have 11 tables for logical > switch ingress p

Re: [ovs-dev] [PATCH] ovn: Document chassis->name for ovn-controller-vtep.

2016-03-23 Thread Kyle Mestery
r-vtep populates this > + column with + db="hardware_vtep"/> in the hardware_vtep database's > + table. > > > > -- > 2.5.5 > LGTM. Acked-By: Kyle Mestery > ___ > dev mailin

Re: [ovs-dev] RFCv2: OVN database options

2016-03-25 Thread Kyle Mestery
On Thu, Mar 24, 2016 at 7:35 PM, Justin Pettit wrote: > >> On Mar 24, 2016, at 1:29 PM, Russell Bryant wrote: >> >> My preferred approach is to continue on the current path and build on the >> success we've had with OVSDB so far. We should take regular checkpoints >> and still remain willing to

Re: [ovs-dev] OVN - Pluggable Distributed DB Infrastructure for OVSDB

2015-06-29 Thread Kyle Mestery
On Thu, Jun 25, 2015 at 8:07 AM, Russell Bryant wrote: > On 06/25/2015 07:21 AM, Gal Sagie wrote: > > Hello All, > > > > Currently OVN uses centralized ovsdb-server to serve the Northbound and > the > > Southbound DB to all the local controllers (sitting at each of the > compute > > nodes). > > >

Re: [ovs-dev] OVN - Pluggable Distributed DB Infrastructure for OVSDB

2015-06-30 Thread Kyle Mestery
On Tue, Jun 30, 2015 at 2:35 PM, Russell Bryant wrote: > On 06/30/2015 04:42 AM, Gal Sagie wrote: > > Hi Justin, > > > > The idea was not to choose one database implementation over the other, > > but design things in such a way > > that its pluggable, so in terms of OVN, switching between > > imp

Re: [ovs-dev] Allowed Address Pairs - OVN

2015-07-01 Thread Kyle Mestery
On Wed, Jul 1, 2015 at 12:11 PM, Ben Pfaff wrote: > On Wed, Jul 01, 2015 at 11:11:05AM +0300, Gal Sagie wrote: > > As you might know, allowed address pairs in neutron is an extension to > > allow port > > to have more then a pair of MAC-IP addresses assigned to it. > > This is useful for cases of

Re: [ovs-dev] kernel module testing

2015-07-16 Thread Kyle Mestery
On Thu, Jul 16, 2015 at 12:55 PM, Ben Pfaff wrote: > On Tue, Jul 14, 2015 at 07:45:42AM +, Pritesh Kothari (pritkoth) wrote: > > > > > On Jul 13, 2015, at 9:40 PM, Ben Pfaff wrote: > > > > > > On Tue, Jul 14, 2015 at 12:34:14AM +, Pritesh Kothari (pritkoth) > wrote: > > >> How about auto

Re: [ovs-dev] [PATCH v2] Start fixing some Python 3 compatibility issues

2015-07-30 Thread Kyle Mestery
is is a start. > > > > Signed-off-by: Terry Wilson > > This is just a rebase, right? All of the changes still look good to me. > This should be fine now that the min Python version was increased to 2.7. > > Acked-by: Russell Bryant > > Looks good to me to

[ovs-dev] [PATCH] AUTHORS: Update my author entry.

2014-10-09 Thread Kyle Mestery
Update the email address for myself in the AUTHORS file. Signed-off-by: Kyle Mestery --- AUTHORS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AUTHORS b/AUTHORS index 2c7c107..d5adbce 100644 --- a/AUTHORS +++ b/AUTHORS @@ -83,7 +83,7 @@ Keith Amidonke

Re: [ovs-dev] [PATCH v1 4/4] openvswitch: Userspace tunneling.

2014-10-24 Thread Kyle Mestery
On Fri, Oct 24, 2014 at 2:23 PM, Pravin Shelar wrote: > On Fri, Oct 24, 2014 at 7:05 AM, Thomas Graf wrote: >> First of all, great to see this work. This is awesome! I read through >> the code a first time. Planning to do more reviewing but looks very >> sane already. >> >> On 10/16/14 at 11:38am

Re: [ovs-dev] ovn-controller appears to be in a tight loop

2016-01-10 Thread Kyle Mestery
On Sun, Jan 10, 2016 at 7:34 PM, Kyle Mestery wrote: > Greetings: > > I'm running the latest OVN/OVS code from last Friday with the latest > OpenStack code from master from the same time period. I've setup a 5-node > OpenStack clluster using the devstack script

Re: [ovs-dev] ovn-controller appears to be in a tight loop

2016-01-11 Thread Kyle Mestery
On Mon, Jan 11, 2016 at 1:22 PM, Russell Bryant wrote: > On 01/10/2016 09:39 PM, Kyle Mestery wrote: > > Jan 10 20:38:00 oc-syd01-prod-compute-191 ovs-vswitchd: > > ovs|110086385|netlink_socket|ERR|fcntl: Too many open files > > After looking around, I think this was th

Re: [ovs-dev] [PATCH 2/2] ovn: Fix localnet ports on the same chassis.

2016-01-19 Thread Kyle Mestery
and it does indeed work as advertised! Nice work Russell! Tested-By: Kyle Mestery Acked-By: Kyle Mestery > --- > ovn/controller/ovn-controller.c | 10 +- > ovn/controller/patch.c | 80 +- > ovn/controller/patch.h | 4 +- > ovn/controller/physic

[ovs-dev] [PATCH] README.ovs-vtep.md: Fix incorrect spacing

2016-01-27 Thread Kyle Mestery
This fixes a simple formatting issue with this file I noticed while reviewing the example of experimenting with the OVS HW-VTEP simulator. Signed-off-by: Kyle Mestery --- vtep/README.ovs-vtep.md | 1 + 1 file changed, 1 insertion(+) diff --git a/vtep/README.ovs-vtep.md b/vtep/README.ovs

Re: [ovs-dev] [PATCH] ovn-northd: Don't set custon log level defaults.

2016-02-01 Thread Kyle Mestery
amp;argc, &argv); > -vlog_set_levels(NULL, VLF_CONSOLE, VLL_WARN); > -vlog_set_levels(&VLM_reconnect, VLF_ANY_DESTINATION, VLL_WARN); > parse_options(argc, argv); > > daemonize_start(false); > -- > 2.5.0 > Minor nit in the spelling of "custom&qu

Re: [ovs-dev] [RFC] ovn-controller: Only process lflows for local datapaths.

2016-02-02 Thread Kyle Mestery
P in a logical datapath in S, add the logical >>>>datapath of the remote end of P to S. >>>> >>>> Extra credit if there's a way to infer (or specify) that a logical >>>> datapath is "terminal", that is, that a packet th

Re: [ovs-dev] [PATCH v4 3/3] ovn-controller: Allocate ct zones for localnet ports.

2016-02-03 Thread Kyle Mestery
On Wed, Feb 3, 2016 at 9:14 AM, Russell Bryant wrote: > On 02/02/2016 05:19 PM, Ben Pfaff wrote: >> On Mon, Jan 25, 2016 at 05:02:03PM -0500, Russell Bryant wrote: >>> Previously, all ct() actions applied to localnet ports used the default >>> conntrack zone. We should allocate a ct zone ID for a

Re: [ovs-dev] [PATCH] ovn-controller: Fix typo in comment.

2016-02-05 Thread Kyle Mestery
that logical datapath, so there's no > * point in maintaining any flows for it anyway, so skip it. */ > -- > 2.1.3 > lol :) Acked-By: Kyle Mestery > ___ > dev mailing list > dev@openvswitch.org > http://openvswit

Re: [ovs-dev] [PATCH V2] Separating ovn nb&sb databases

2016-02-09 Thread Kyle Mestery
On Tue, Feb 9, 2016 at 5:24 AM, Michael wrote: > Options for running ovsdb separately is done and also described in the help > file. > > NEWS file is updated. > > I just have to rise a few points: > > - the way thing are running with start/stop daemon from ova-lib require > the daemon to have a p

Re: [ovs-dev] [PATCH V2] Separating ovn nb&sb databases

2016-02-09 Thread Kyle Mestery
On Tue, Feb 9, 2016 at 12:18 PM, Russell Bryant wrote: > On 02/09/2016 01:12 PM, Kyle Mestery wrote: >> On Tue, Feb 9, 2016 at 5:24 AM, Michael >> wrote: >>> Options for running ovsdb separately is done and also described in the help >>> file. >>> >

Re: [ovs-dev] [PATCH V2] Separating ovn nb&sb databases

2016-02-09 Thread Kyle Mestery
reasonable to me as a follow-on patch perhaps. > >> On 09 Feb 2016, at 19:28, Russell Bryant wrote: >> >> On 02/09/2016 01:21 PM, Kyle Mestery wrote: >>> On Tue, Feb 9, 2016 at 12:18 PM, Russell Bryant wrote: >>>> On 02/09/2016 01:12 PM, Kyle Mestery

Re: [ovs-dev] [PATCH V3] Separating ovn nb&sb databases

2016-02-10 Thread Kyle Mestery
t; > Signed-off-by: Michael Arnaldi This looks fine to me now, thanks Michael! This is going to make the default deployments of OVN match what the recommended deployment pattern should be (separate ovsdb-server's for NB and SB DBs). Acked-By: Kyle Mestery > --- > NEWS

Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint separation proposal

2016-02-15 Thread Kyle Mestery
On Fri, Jan 29, 2016 at 8:36 PM, Darrell Lu wrote: > The following is a proposal regarding how to allow logical > and physical endpoint contexts for OVN to stay separated, from > a CMS POV. > > A logical endpoint has the form: ls1-port1. > A physical endpoint has the general form: > chassis/physic

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

2016-02-15 Thread Kyle Mestery
On Mon, Feb 15, 2016 at 2:02 PM, Russell Bryant wrote: > On 02/01/2016 01:19 PM, Amitabha Biswas wrote: >> 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/. >> >> T

Re: [ovs-dev] [PATCH ovn] ovn-nbctl: Don't loop forever on lost db connection.

2015-03-31 Thread Kyle Mestery
break; > } > > if (seqno != ovsdb_idl_get_seqno(nb_ctx.idl)) { > @@ -628,5 +631,5 @@ main(int argc, char *argv[]) > } > ovsdb_idl_destroy(nb_ctx.idl); > > -exit(0); > +exit(res); > } > -- > 2.1.0 > > Looks fine to me. Acked-by

Re: [ovs-dev] [PATCH] ovn: Add logo in SVG and PNG format.

2015-04-01 Thread Kyle Mestery
On Wed, Apr 1, 2015 at 5:03 AM, Thomas Graf wrote: > On 04/01/15 at 11:52am, Thomas Graf wrote: > > On 03/31/15 at 09:17pm, Ben Pfaff wrote: > > > Signed-off-by: Ben Pfaff > > > --- > > > I'd like to order some OVN stickers for distribution at OpenStack and > by > > > mail, so here's my OVN logo

Re: [ovs-dev] [PATCH] ovn: Add logo in SVG and PNG format.

2015-04-01 Thread Kyle Mestery
On Wed, Apr 1, 2015 at 10:39 AM, Ben Pfaff wrote: > On Wed, Apr 01, 2015 at 08:15:52AM -0500, Kyle Mestery wrote: > > On Wed, Apr 1, 2015 at 5:03 AM, Thomas Graf > wrote: > > > > > On 04/01/15 at 11:52am, Thomas Graf wrote: > > > > On 03/31/15 at 09:17pm

Re: [ovs-dev] [PATCH] ovn: Add logo in SVG and PNG format.

2015-04-02 Thread Kyle Mestery
On Thu, Apr 2, 2015 at 12:57 AM, Ben Pfaff wrote: > On Thu, Apr 02, 2015 at 12:35:21AM +0200, Thomas Graf wrote: > > On 04/01/15 at 09:17am, Ben Pfaff wrote: > > > On Wed, Apr 01, 2015 at 11:12:39AM -0500, Kyle Mestery wrote: > > > > On Wed, Apr 1, 2015

Re: [ovs-dev] [PATCH v2] Add setuptools for Python lib for PyPI

2015-04-10 Thread Kyle Mestery
On Fri, Apr 10, 2015 at 3:42 PM, Terry Wilson wrote: > - Original Message - > > On 04/10/2015 03:57 PM, Terry Wilson wrote: > > > This adds very basic support for setuptools so that the OVS Python > > > lib can be added to PyPI. > > > > Most Python libraries are on PyPI, so it makes sense

Re: [ovs-dev] [PATCH v2] Add setuptools for Python lib for PyPI

2015-04-10 Thread Kyle Mestery
switch.org/', > +author='Open vSwitch', > +author_email='dev@openvswitch.org', > +packages=['ovs', 'ovs.db', 'ovs.unixctl'], > +keywords=['openvswitch', 'ovs', 'OVSDB'], > +license=&#

Re: [ovs-dev] ovn-nbd - name change?

2015-04-10 Thread Kyle Mestery
On Fri, Apr 10, 2015 at 3:29 PM, Russell Bryant wrote: > Someone just pointed out to me that the name "ovn-nbd" might cause > confusion, as "nbd" is already used in the virtualization context for > "Network Block Device". > > https://github.com/qemu/qemu/blob/master/qemu-nbd.c > > So, what would

Re: [ovs-dev] Neutron Container integration.

2015-04-22 Thread Kyle Mestery
On Wed, Apr 22, 2015 at 8:28 AM, Russell Bryant wrote: > On 04/22/2015 09:21 AM, Salvatore Orlando wrote: > > Polling for port status is the easiest solution. > > In some cases excessive polling does not scale very well. > > For instance in large cloud with 1000s of compute nodes, this will put a

Re: [ovs-dev] [PATCH ovn 1/2] ovn-controller: Fix invalid br_int_name handling.

2015-05-01 Thread Kyle Mestery
630f7 100644 > --- a/ovn/controller/ovn-controller.h > +++ b/ovn/controller/ovn-controller.h > @@ -19,7 +19,7 @@ > > struct controller_ctx { > char *chassis_id; /* ID for this chassis. */ > -const char *br_int_name;/* Name of l

Re: [ovs-dev] [PATCH ovn 2/2] ovn-controller: Free some more memory before exit.

2015-05-01 Thread Kyle Mestery
b.sock", ovs_rundir()); > } else if (argc == 1) { > -ovs_remote = argv[0]; > +ovs_remote = xstrdup(argv[0]); > } else { > VLOG_FATAL("exactly zero or one non-option argument required; " > "use --help for usa

Re: [ovs-dev] OVN and OpenStack Provider Networks

2015-06-14 Thread Kyle Mestery
On Sat, Jun 13, 2015 at 5:12 PM, Salvatore Orlando wrote: > Hi Russel, > > thanks for sharing these thoughts. I was indeed thinking as well we need to > support this in OVN as the provider networks are a fairly basic neutron > feature - despite being an "extensions". > I have some comments inlin

[ovs-dev] Support for Linux kernels >= 3.12 with the out of tree kernel module

2014-02-26 Thread Kyle Mestery
I see this patch [1] in patchwork now from Pravin via James Page. I pulled this down and started to look at what it would take to get this also working with 3.13. It looks like the netlink interfaces changed a bit here. Is anyone working on this at the moment? If not, I'd like to take this on. Seco

Re: [ovs-dev] Support for Linux kernels >= 3.12 with the out of tree kernel module

2014-02-27 Thread Kyle Mestery
On Wed, Feb 26, 2014 at 5:07 PM, Pravin Shelar wrote: > On Wed, Feb 26, 2014 at 2:09 PM, Kyle Mestery > wrote: > > I see this patch [1] in patchwork now from Pravin via James Page. > > I pulled this down and started to look at what it would take to get this > > also wor

[ovs-dev] Announcing an Open vSwitch Hackathon March 26-27

2014-03-04 Thread Kyle Mestery
I'd like to announce that Cisco is going to host an Open vSwitch Hackathon on March 26 and 27. The event is setup through Eventbrite at the link provided here [1]. Please register for either an in-person or remote attendee link if you're interested in spending two days deep-diving into OVS code, pa

Re: [ovs-dev] Support for Linux kernels >= 3.12 with the out of tree kernel module

2014-03-06 Thread Kyle Mestery
On Thu, Feb 27, 2014 at 3:28 PM, Pravin Shelar wrote: > On Thu, Feb 27, 2014 at 12:41 PM, Kyle Mestery > wrote: > > On Wed, Feb 26, 2014 at 5:07 PM, Pravin Shelar > wrote: > >> > >> On Wed, Feb 26, 2014 at 2:09 PM, Kyle Mestery < > mest...@noironetworks.c

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

2014-03-21 Thread Kyle Mestery
Add support for building the in-tree kernel datapath for Linux kernel 3.13. There were some changes in the netlink area which required adding new compatibility code for this layer. Signed-off-by: Kyle Mestery --- FAQ | 2 +- acinclude.m4

Re: [ovs-dev] [PATCH] bridge: don't bring up internal ports by default.

2014-03-25 Thread Kyle Mestery
lan_mode, "splinter")) { > netdev_turn_flags_on(netdev, NETDEV_UP, NULL); > } > > This looks ok to me, though it does change the behavior of the internal ports here. Should there be a NEWS entry for this to alert users?

Re: [ovs-dev] [PATCH v2] bridge: don't bring up internal ports by default.

2014-03-25 Thread Kyle Mestery
_cfg, br->cfg)) { > +if (port_cfg->vlan_mode && !strcmp(port_cfg->vlan_mode, "splinter")) { > netdev_turn_flags_on(netdev, NETDEV_UP, NULL); > } > > Thanks for the NEWS addition Flavio! LGTM now. Acked-by: Kyle Mestery > -- > 1.8.5.3 > > ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] FAQ: Fix supported kernel version.

2014-03-25 Thread Kyle Mestery
2.6.32 to 3.11 > + 2.2.x 2.6.32 to 3.12 > > Open vSwitch userspace should also work with the Linux kernel module > built into Linux 3.3 and later. > -- > Acked-by: Kyle Mestery > 1.7.9.5 > > ___ >

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

2014-03-26 Thread Kyle Mestery
On Tue, Mar 25, 2014 at 3:36 PM, Pravin Shelar wrote: > On Fri, Mar 21, 2014 at 10:41 AM, Kyle Mestery > wrote: > > Add support for building the in-tree kernel datapath for Linux kernel > 3.13. > > There were some changes in the netlink area which required adding new >

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

2014-03-26 Thread Kyle Mestery
On Tue, Mar 25, 2014 at 3:36 PM, Pravin Shelar wrote: > On Fri, Mar 21, 2014 at 10:41 AM, Kyle Mestery > wrote: > > Add support for building the in-tree kernel datapath for Linux kernel > 3.13. > > There were some changes in the netlink area which required adding new >

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

2014-03-27 Thread Kyle Mestery
Add support for building the in-tree kernel datapath for Linux kernel 3.13. There were some changes in the netlink area which required adding new compatibility code for this layer. Signed-off-by: Kyle Mestery --- v2: Address a few comments from Pravin. Some of those comments proved

Re: [ovs-dev] [PATCH] stream-tcp: Fix error message for failed TCP_NODELAY setting on Windows.

2014-03-27 Thread Kyle Mestery
On Thu, Mar 27, 2014 at 11:56 AM, Ben Pfaff wrote: > From: Ben Pfaff > > Is this the right author? > Reported-by: Gurucharan Shetty > Signed-off-by: Ben Pfaff > --- > lib/stream-tcp.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/lib/stream-tcp.c b/lib/strea

Re: [ovs-dev] [PATCH v2] stream-tcp: Fix error message for failed TCP_NODELAY setting on Windows.

2014-03-27 Thread Kyle Mestery
gt; -return errno; > +return error; > } > > return new_fd_stream(name, fd, connect_status, streamp); > -- > LGTM. Acked-by: Kyle Mestery > 1.8.5.3 > > ___ > dev mailing list > dev@openvswitch.org >

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

2014-03-27 Thread Kyle Mestery
On Thu, Mar 27, 2014 at 12:09 PM, Pravin Shelar wrote: > On Wed, Mar 26, 2014 at 4:32 PM, Kyle Mestery > wrote: > > On Tue, Mar 25, 2014 at 3:36 PM, Pravin Shelar > wrote: > >> > >> On Fri, Mar 21, 2014 at 10:41 AM, Kyle Mestery > >> wrote: > >

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

2014-03-27 Thread Kyle Mestery
Add support for building the in-tree kernel datapath for Linux kernel 3.13. There were some changes in the netlink area which required adding new compatibility code for this layer. Signed-off-by: Kyle Mestery --- v3: Correctly make genl_register_family backwards compatible. v2: Address a few

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

2014-03-27 Thread Kyle Mestery
hanks, Kyle On Thu, Mar 27, 2014 at 1:17 PM, Kyle Mestery wrote: > Add support for building the in-tree kernel datapath for Linux kernel 3.13. > There were some changes in the netlink area which required adding new > compatibility code for this layer. > > Signed-off-by: Kyle

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

2014-03-27 Thread Kyle Mestery
dle > errors" > > OK, given this, I'll wait for adding 3.14. > On Thu, Mar 27, 2014 at 11:19 AM, Kyle Mestery > wrote: > > Pravin, one more change I want to make here. Since 3.14 is almost out, > > would you be ok if I bump this up to claim 3.14 support? I don&#

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

2014-03-30 Thread Kyle Mestery
On Fri, Mar 28, 2014 at 5:25 PM, Jesse Gross wrote: > On Thu, Mar 27, 2014 at 11:17 AM, Kyle Mestery > wrote: > > Add support for building the in-tree kernel datapath for Linux kernel > 3.13. > > There were some changes in the netlink area which required adding new >

[ovs-dev] [PATCH] datapath: Add support for kernels 3.13 and 3.14

2014-04-01 Thread Kyle Mestery
Linux kernel commits as well: 36d5fe6a000790f56039afe26834265db0a3ad4c: "core, nfqueue, openvswitch: Orphan frags in skb_zerocopy and handle errors" 63862b5bef7349dd1137e4c70702c67d77565785: "net: replace macros net_random and net_srandom with direct calls to prandom" Signed-

Re: [ovs-dev] datapath/linux/compat question

2014-04-01 Thread Kyle Mestery
On Tue, Apr 1, 2014 at 1:27 PM, Zoltan Kiss wrote: > Hi, > > I have a recent patch on net-next which affects OVS as well: > > http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=36d5fe6a000790f56039afe26834265db0a3ad4c > > I guess the changes for datapatch.c will turn up auto

Re: [ovs-dev] datapath/linux/compat question

2014-04-03 Thread Kyle Mestery
On Wed, Apr 2, 2014 at 7:05 PM, Jesse Gross wrote: > On Wed, Apr 2, 2014 at 9:56 AM, Zoltan Kiss wrote: >> On 01/04/14 20:26, Jesse Gross wrote: >>> >>> On Tue, Apr 1, 2014 at 11:41 AM, Kyle Mestery >>> wrote: >>>> >>>> On Tue, Apr

Re: [ovs-dev] [PATCH] datapath: Add support for kernels 3.13

2014-04-03 Thread Kyle Mestery
more comment inline. Acked-by: Kyle Mestery On Wed, Apr 2, 2014 at 3:37 PM, Pravin wrote: > From: Pravin Shelar > > Add support for building the in-tree kernel datapath for > Linux kernels up to 3.13. There were some changes in the This patch also supports 3.14. Please adjust

Re: [ovs-dev] [PATCH] datapath: Add support for kernels 3.13

2014-04-04 Thread Kyle Mestery
On Thu, Apr 3, 2014 at 5:52 PM, Pravin Shelar wrote: > On Thu, Apr 3, 2014 at 12:56 PM, Kyle Mestery > wrote: >> Thanks for fixing this up a bit more Pravin! I tested this against the latest >> master, it builds fine and loads on Fedora 20 with 3.13. >> >> Also

Re: [ovs-dev] [PATCH RFC] SECURITY: New document describing proposed security process for OVS.

2015-01-05 Thread Kyle Mestery
On Mon, Jan 5, 2015 at 9:23 AM, Jiri Benc wrote: > On Fri, 2 Jan 2015 17:57:14 -0800, Ben Pfaff wrote: > > 1) Consider provisions for ensuring privacy and integrity of > > communications around disclosure (eg, use PGP for all comms). > > That never hurts. I'd argue that's not strictly req

Re: [ovs-dev] [PATCH] vagrant: Provide basic Vagrantfile

2015-01-05 Thread Kyle Mestery
On Mon, Jan 5, 2015 at 9:31 AM, Thomas Graf wrote: > This is a basic Vagrantfile which compiles and provisions the local OVS > source tree in a Fedora 20 VM. The Vagrantfile.in can be extended to > support additional distributions. > > To use this Vagrantfile: > 1. Install Vagrant plus a compati

Re: [ovs-dev] [PATCH] vagrant: Provide basic Vagrantfile

2015-01-05 Thread Kyle Mestery
On Mon, Jan 5, 2015 at 9:47 AM, Thomas Graf wrote: > On 01/05/15 at 09:41am, Kyle Mestery wrote: > > Would it be useful to also have this run the OVS tests as a part of this > > after installing and starting it? We could also make this use a local OVS > > repository in

Re: [ovs-dev] [PATCH RFC] SECURITY: New document describing proposed security process for OVS.

2015-01-06 Thread Kyle Mestery
On Tue, Jan 6, 2015 at 11:58 AM, Ben Pfaff wrote: > On Mon, Jan 05, 2015 at 09:29:47AM -0600, Kyle Mestery wrote: > > On Mon, Jan 5, 2015 at 9:23 AM, Jiri Benc wrote: > > > > > On Fri, 2 Jan 2015 17:57:14 -0800, Ben Pfaff wrote: > > > > 1) Consider

Re: [ovs-dev] [ovs-discuss] OVN, Bringing Native Virtual Networking to OVS

2015-01-13 Thread Kyle Mestery
On Tue, Jan 13, 2015 at 1:28 PM, Justin Pettit wrote: > The Open vSwitch team is pleased to announce OVN, a new subproject in > development within the Open vSwitch. The full project announcement is at > Network Heresy and reproduced below. OVN complements the existing > capabilities of OVS to a

Re: [ovs-dev] OVN architecture

2015-01-14 Thread Kyle Mestery
On Tue, Jan 13, 2015 at 2:43 PM, Thomas Graf wrote: > On 01/13/15 at 11:29am, Ben Pfaff wrote: > > Open Virtual Network (OVN) Proposed Architecture > > > > > > The Open vSwitch team is pleased to announce OVN, a new subproject in > > development wi

Re: [ovs-dev] [PATCH 0/4] Apple Mac OSX Support for CTL commands

2015-01-23 Thread Kyle Mestery
On Fri, Jan 23, 2015 at 5:16 AM, Thomas Graf wrote: > On 01/22/15 at 09:22pm, Ben Pfaff wrote: > > On Fri, Jan 23, 2015 at 01:17:34AM +, Dave Tucker wrote: > > > This patch sets adds Apple Mac OSX Support for the ovs-vsctl and > > > ovs-ofctl. This is useful for administrators who are managin

Re: [ovs-dev] OVN to-do list

2015-02-20 Thread Kyle Mestery
On Fri, Feb 20, 2015 at 7:15 PM, Russell Bryant wrote: > On 02/20/2015 06:46 PM, Ben Pfaff wrote: > > * Not yet scoped: > > > > ** Neutron plugin. > > > >I'd appreciate input on this. > > I've just started looking into this. As a start, from a high level my > to-do thoughts are: > > *** Crea

Re: [ovs-dev] OVN to-do list

2015-02-20 Thread Kyle Mestery
On Fri, Feb 20, 2015 at 8:23 PM, Jesse Gross wrote: > On Fri, Feb 20, 2015 at 4:36 PM, Ben Pfaff wrote: > > On Fri, Feb 20, 2015 at 04:03:21PM -0800, Jesse Gross wrote: > >> On Fri, Feb 20, 2015 at 3:46 PM, Ben Pfaff wrote: > >> > *** Tunnel encapsulation to publish. > >> > > >> > We can pr

Re: [ovs-dev] OVN to-do list

2015-02-23 Thread Kyle Mestery
On Sat, Feb 21, 2015 at 6:02 AM, Russell Bryant wrote: > On 02/20/2015 09:51 PM, Kyle Mestery wrote: > > On Fri, Feb 20, 2015 at 7:15 PM, Russell Bryant > <mailto:rbry...@redhat.com>> wrote: > > *** Create a Neutron ML2 mechanism driver that implements the &g

Re: [ovs-dev] [PATCH 2/3] RHEL: update rpm build instructions after trying them with CentOS 7

2015-10-04 Thread Kyle Mestery
On Sun, Oct 4, 2015 at 5:20 PM, Ansis Atteka wrote: > Mention: > 1. that these instructions have been tested with CentOS 7 (which is >almost identical distribution to RHEL 7). > 2. directory where rpm packages that were just built can be found. > 3. mention SElinux implications that could pre

Re: [ovs-dev] [PATCH 1/3] Fedora: update rpm build instructions after trying them with Fedora 22

2015-10-04 Thread Kyle Mestery
On Sun, Oct 4, 2015 at 5:20 PM, Ansis Atteka wrote: > Mention: > 1. that these instructions have been tested with Fedora 22. > 2. dependencies that need to be installed on Fedora 22. Otherwise, >rpmbuild command simply fails. > 3. that rpmdev-setuptree utility can be used to create rpmbuild

Re: [ovs-dev] [PATCH 3/3] RHEL: create /etc/openvswitch directory

2015-10-04 Thread Kyle Mestery
etc/bash_completion.d/ovs-vsctl-bashcomp.bash > /etc/init.d/openvswitch > -- > 2.1.4 > > Ackedy-by: Kyle Mestery > ___ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > __

Re: [ovs-dev] [PATCH v2] ovn-tutorial: Add a section on ACLs.

2015-11-04 Thread Kyle Mestery
Thanks for writing this up Russell! I found super pedantic (possible) nit, but otherwise, this reads fine to me and was helpful in understanding how ACLs work. Thanks! Acked-by: Kyle Mestery On Wed, Nov 4, 2015 at 9:53 AM, Russell Bryant wrote: > Add a section that gives a quick introduct

Re: [ovs-dev] [PATCH 1/2] AUTHORS: Update Justin Pettit's email address.

2015-11-06 Thread Kyle Mestery
On Fri, Nov 6, 2015 at 4:48 PM, Ben Pfaff wrote: > On Fri, Nov 06, 2015 at 02:00:30PM -0800, Justin Pettit wrote: > > Signed-off-by: Justin Pettit > > Acked-by: Ben Pfaff > My address isn't that cool, but: Acked-By: Kyle Mestery :) > ___

Re: [ovs-dev] [PATCH] ovn-tutorial: Use github instead of relative links.

2015-11-12 Thread Kyle Mestery
; internet access). > > Signed-off-by: Russell Bryant > This LGTM. Acked-By: Kyle Mestery --- > tutorial/OVN-Tutorial.md | 46 > +++--- > 1 file changed, 23 insertions(+), 23 deletions(-) > > diff --git a/tutorial/OVN-Tutorial.md b/tutorial

  1   2   3   4   5   >