Re: [ovs-dev] [patch_v5 2/3] ovn: Add additional comments regarding arp responders.

2016-11-02 Thread Darrell Ball
On Sun, Oct 23, 2016 at 11:19 AM, Mickey Spiegel wrote: > Acked-by: Mickey Spiegel > > A few very minor nits below. > > On Fri, Oct 21, 2016 at 1:36 PM, Darrell Ball wrote: > >> There has been enough confusion regarding logical switch datapath >> arp responders in ovn to warrant some additional

[ovs-dev] [PATCH] datapath-windows: Enable support for tracking ICMP code and type

2016-11-02 Thread Sairam Venugopal
Add support for tracking ICMP code and Type in the Hyper-V Conntrack module. This code is similar to the userspace connection tracker. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack-icmp.c | 80 +++ datapath-windows/ovsext/Conntrack-other.c | 1

Re: [ovs-dev] [PATCH] Add support for specifying SSL connection parameters to ovsdb

2016-11-02 Thread Ethan Rahn
Added a 'Signed-off-by' line. Thanks for getting back to me and this being patient while I learned how to use 'git send-email'. I'm glad I could contribute upstream to the project. Cheers, Ethan On Wed, Nov 2, 2016 at 4:19 PM, Ethan Rahn wrote: > > Signed-off-by: Ethan Rahn > --- > AUTHORS

[ovs-dev] [PATCH] Add support for specifying SSL connection parameters to ovsdb

2016-11-02 Thread Ethan Rahn
OVSDB currently does not support fine-tuning the SSL parameters used for connections. This means that users are unable to specify not using ciphers widely considered to be unsafe or to avoid using TLS protocols that do not meet their organizational standards. This adds two new commands "--ssl-p

[ovs-dev] [PATCH] Add support for specifying SSL connection parameters to ovsdb

2016-11-02 Thread Ethan Rahn
Signed-off-by: Ethan Rahn --- AUTHORS | 1 + lib/automake.mk | 2 + lib/ssl-connect-syn.man | 5 +++ lib/ssl-connect.man | 16 +++ lib/stream-ssl.c | 70 +++

Re: [ovs-dev] [PATCH 2/2] Prepare for 2.6.2.

2016-11-02 Thread Justin Pettit
> On Nov 1, 2016, at 7:56 PM, Ben Pfaff wrote: > > On Tue, Nov 01, 2016 at 03:44:13PM -0700, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > Acked-by: Ben Pfaff Thanks. I pushed the series to branch-2.6. I'll kick off the release process. --Justin _

Re: [ovs-dev] [PATCH v2] ovn-controller: Container can have connection to a hosting VM.

2016-11-02 Thread Guru Shetty
On 2 November 2016 at 09:09, Ben Pfaff wrote: > On Wed, Oct 05, 2016 at 03:17:44AM -0700, Gurucharan Shetty wrote: > > A Container running inside a VM can have a connection to the > > hosting VM (parent port) in the logical topology (for e.g via a router). > > So we should be able to loop-back in

[ovs-dev] Supply Chain Management Target Approach

2016-11-02 Thread Sarah Brush
Did you get a chance to review my previous email? Please let me know if you did like to review few sample records from your target audience. Look forward to hearing back. Regards, Sarah ___ From: Sarah Brush

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix crash in QoS.

2016-11-02 Thread Stokes, Ian
> qos_conf can be NULL. This can be easily reproduced by setting egress QoS > on a port: > > ``` > ovs-vsctl set port dpdk2 qos=@newqos -- --id=@newqos create qos > type=egress-policer other-config:cir=4600 other-config:cbs=2048 ``` > > Reported-by: Ian Stokes > Fixes: 78bd47cf44a5 ("netdev

Re: [ovs-dev] [PATCH 00/23] Convert next batch of docs to rST

2016-11-02 Thread Joe Stringer
On 30 October 2016 at 06:29, Stephen Finucane wrote: > This is the third conversion series. This one is larger yet again, > though many of the docs are rather small. After this series, only the > two tutorial docs in 'tutorial' require conversion (along with any > unsuffixed files I may have misse

Re: [ovs-dev] [PATCH] ovn-nb: Document the syntax for an address set name.

2016-11-02 Thread Ben Pfaff
On Fri, Oct 07, 2016 at 09:00:13AM -0700, Ben Pfaff wrote: > Also, it is not necessary to specify that the name must be unique because > the schema documentation generator does that for us. > > Reported-by: Kevin Lin > Reported-at: http://openvswitch.org/pipermail/dev/2016-October/080386.html > S

Re: [ovs-dev] [PATCH] Add support for specifying SSL connection parameters to ovsdb

2016-11-02 Thread Ben Pfaff
Thanks a lot. I have only a few style fixes for this. May I have a Signed-off-by? Thanks, Ben. On Thu, Oct 06, 2016 at 04:22:19PM -0700, Ethan Rahn wrote: > Third time is the charm :) Sent via git send-email this time around. > Thanks for your patience and sorry for the trouble. > > Cheers, >

Re: [ovs-dev] [PATCH v2] ovn: Add a case of policy based routing.

2016-11-02 Thread Ben Pfaff
On Wed, Nov 02, 2016 at 09:28:44AM -0700, Ben Pfaff wrote: > On Thu, Oct 06, 2016 at 03:53:12AM -0700, Gurucharan Shetty wrote: > > OVN currently supports multiple gateway routers (residing on > > different chassis) connected to the same logical topology. > > > > When external traffic enters the l

Re: [ovs-dev] [PATCH v2] ovn: Add a case of policy based routing.

2016-11-02 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 03:53:12AM -0700, Gurucharan Shetty wrote: > OVN currently supports multiple gateway routers (residing on > different chassis) connected to the same logical topology. > > When external traffic enters the logical topology, they can enter > from any gateway routers and reach

Re: [ovs-dev] [PATCH v2] ovn-controller: Container can have connection to a hosting VM.

2016-11-02 Thread Ben Pfaff
On Wed, Oct 05, 2016 at 03:17:44AM -0700, Gurucharan Shetty wrote: > A Container running inside a VM can have a connection to the > hosting VM (parent port) in the logical topology (for e.g via a router). > So we should be able to loop-back into the same VM, even if the > final packet delivered doe

Re: [ovs-dev] [PATCH] ovn: Fix QoS marking without match

2016-11-02 Thread Ben Pfaff
On Wed, Nov 02, 2016 at 08:38:29PM +0530, Babu Shanmugam wrote: > When a Logical_Switch's qos_rule does not have a match set, the rule should > apply for all the logical ports in that switch. > > Signed-off-by: Babu Shanmugam I don't think this is really a bug fix, since the documentation for th

[ovs-dev] [PATCH] ovn: Fix QoS marking without match

2016-11-02 Thread Babu Shanmugam
When a Logical_Switch's qos_rule does not have a match set, the rule should apply for all the logical ports in that switch. Signed-off-by: Babu Shanmugam --- ovn/northd/ovn-northd.c | 8 +++- tests/ovn.at| 4 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/ovn

[ovs-dev] [PATCH v4 4/4] ovn: Add ovndb servers ocf script in debian packager

2016-11-02 Thread Babu Shanmugam
The OCF script will be present in the ovn-common package and installed in the openvswitch scripts folder and a symbolic link to this file will be created in the OCF resources folder. The OCF resource agent name for this resource is ocf:ovn:ovndb-servers Signed-off-by: Babu Shanmugam --- debian/

[ovs-dev] [PATCH v4 0/4] High availability support for OVN DB servers using pacemaker

2016-11-02 Thread Babu Shanmugam
v3 -> v4: Following changes were suggested by Andy on v3 - Updated the ovn-ctl man page - Added information regarding the pacemaker integration in the IntegrationGuide.md - Added debug messages in the ocf script This patch contains changes required to run a pacemaker resource agent to ma

[ovs-dev] [PATCH v4 3/4] ovn: Add ovn db servers ocf script in fedora packager

2016-11-02 Thread Babu Shanmugam
The OCF script will be present in the ovn-common package and installed in the openvswitch scripts folder. A symbolic link to this file will be created in the OCF resources folder. The OCF resource agent name for this resource is ocf:ovn:ovndb-servers Signed-off-by: Babu Shanmugam --- rhel/openv

[ovs-dev] [PATCH v4 2/4] ovn: OCF script for OVN OVSDB servers

2016-11-02 Thread Babu Shanmugam
Co-authored-by: Numan Siddique Signed-off-by: Numan Siddique Co-authored-by: Andrew Beekhof Signed-off-by: Andrew Beekhof Signed-off-by: Babu Shanmugam --- IntegrationGuide.md | 63 +++ ovn/utilities/automake.mk | 6 +- ovn/utilities/ovndb-servers.ocf | 356 ++

[ovs-dev] [PATCH v4 1/4] ovn: ovn-ctl support for HA ovn DB servers

2016-11-02 Thread Babu Shanmugam
This patch adds support to start_ovsdb() function in ovn-ctl to start the ovn db servers in backup mode. This can be done in the following ways 1. Use parameters --ovn-nb-sync-from-addr and --ovn-sb-sync-from-addr to set the addresses of the active server. 2. Create files $etcdir/ovnnb-active.co