[ovs-dev] RETURNED MAIL: SEE TRANSCRIPT FOR DETAILS

2016-10-06 Thread teffal
The original message was received at Fri, 7 Oct 2016 10:53:46 +0530 from 79.79.101.11 - The following addresses had permanent fatal errors - - Transcript of session follows - ... while talking to host openvswitch.org.: >>> MAIL FROM:tef...@yazaki-europe.com <<< 506 Access denied

[ovs-dev] [PATCH 4/4] expr: Better simplify some special cases of expressions.

2016-10-06 Thread Ben Pfaff
It's pretty unlikely that a human would write expressions like these, but they can come up in machine-generated expressions and it seems best to simplify them in an efficient way. Signed-off-by: Ben Pfaff --- ovn/lib/expr.c | 41 ++--- tests/ovn.at | 12

[ovs-dev] [PATCH 2/4] expr: Fix abort when simplifying "x != 0/0".

2016-10-06 Thread Ben Pfaff
The test added by this commit is very specific to the particular problem, whereas a more general test would be better. A later commit adds the general test. Signed-off-by: Ben Pfaff --- ovn/lib/expr.c | 11 ++- tests/ovn.at | 2 ++ 2 files changed, 12 insertions(+), 1 deletion(-) di

[ovs-dev] [PATCH 3/4] expr: Improve test so that it would have found the bugs I just fixed.

2016-10-06 Thread Ben Pfaff
The test didn't check for x == 0/0 or x != 0/0 cases, and thus they were buggy. Also, add "expression" as a keyword for tests that only had "expressions" (plural). Signed-off-by: Ben Pfaff --- tests/ovn.at | 28 +--- tests/test-ovn.c | 10 -- 2 files changed,

[ovs-dev] [PATCH 1/4] expr: Simplify "x == 0/0" into 1.

2016-10-06 Thread Ben Pfaff
An expression like "x == 0/0" does not test any actual bits in field x, so it resolves to true, but expr_simplify() was not smart enough to see this. This goes beyond an optimization, to become a bug fix, because expr_normalize() will assert-fail for expressions that become trivial when this simpl

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

2016-10-06 Thread Andy Zhou
Babu, Thank you for working on this. At a high level, it is not clear to me the boundary between ocf scripts and the ovn-ctl script -- i.e. which aspect is managed by which entity. For example, 1) which scripts are responsible for starting the ovsdb servers. 2) Which script should manage the fai

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

2016-10-06 Thread Ethan Rahn
Third time is the charm :) Sent via git send-email this time around. Thanks for your patience and sorry for the trouble. Cheers, Ethan On Thu, Oct 6, 2016 at 4:21 PM, Ethan Rahn wrote: > --- > AUTHORS | 1 + > lib/automake.mk | 2 +

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

2016-10-06 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-10-06 Thread Ethan Rahn
--- AUTHORS | 1 + lib/automake.mk | 2 + lib/ssl-connect-syn.man | 5 +++ lib/ssl-connect.man | 16 +++ lib/stream-ssl.c | 70 +++ lib/st

Re: [ovs-dev] [PATCH] ovn-northd: Drop redundant matching constraints in build_stateful().

2016-10-06 Thread Justin Pettit
> On Oct 5, 2016, at 6:27 PM, Ben Pfaff wrote: > > ip4.dst implies ip, udp.dst implies udp, and tcp.dst implies tcp. > > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/

Re: [ovs-dev] Hyphens in address set names

2016-10-06 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 02:56:44PM -0700, Kevin Lin wrote: > I’m a developer under Ethan Jackson on Quilt. We’re using address sets in our > ACL match strings, > but they don’t seem to parse properly for names with hypens: > > error parsing match "ip4.src == $foo-bar": Syntax error at `$foo' expe

[ovs-dev] Hyphens in address set names

2016-10-06 Thread Kevin Lin
Hi, I’m a developer under Ethan Jackson on Quilt. We’re using address sets in our ACL match strings, but they don’t seem to parse properly for names with hypens: error parsing match "ip4.src == $foo-bar": Syntax error at `$foo' expecting address set name. Is this expected? Thanks, —Kevin

Re: [ovs-dev] [PATCH] NEWS: Add release date for 2.6.0.

2016-10-06 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 05:52:33PM -0300, Thadeu Lima de Souza Cascardo wrote: > On Thu, Oct 06, 2016 at 01:51:08PM -0700, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > --- > > NEWS | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/NEWS b/NEWS > > index dc16b71..

Re: [ovs-dev] [PATCH v2] ovn-northd: support IPAM with externally specified MAC

2016-10-06 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 04:59:32PM -0400, Lance Richardson wrote: > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: dev@openvswitch.org, russ...@ovn.org > > Sent: Thursday, October 6, 2016 4:48:45 PM > > Subject: Re: [PATCH v2] ovn-northd: support IPAM with externally specified > > MAC >

Re: [ovs-dev] [PATCH v2] ovn-northd: support IPAM with externally specified MAC

2016-10-06 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: dev@openvswitch.org, russ...@ovn.org > Sent: Thursday, October 6, 2016 4:48:45 PM > Subject: Re: [PATCH v2] ovn-northd: support IPAM with externally specified MAC > > On Thu, Oct 06, 2016 at 03:58:51PM -0400, Lance Richardson wrote: > > The curr

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

2016-10-06 Thread Gurucharan Shetty
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 its eventual destination. This is achieved with proper static routes configur

Re: [ovs-dev] [PATCH] NEWS: Add release date for 2.6.0.

2016-10-06 Thread Thadeu Lima de Souza Cascardo
On Thu, Oct 06, 2016 at 01:51:08PM -0700, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff > --- > NEWS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/NEWS b/NEWS > index dc16b71..9b6fa35 100644 > --- a/NEWS > +++ b/NEWS > @@ -34,7 +34,7 @@ Post-v2.6.0 > * New option '

[ovs-dev] [PATCH] NEWS: Add release date for 2.6.0.

2016-10-06 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- NEWS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/NEWS b/NEWS index dc16b71..9b6fa35 100644 --- a/NEWS +++ b/NEWS @@ -34,7 +34,7 @@ Post-v2.6.0 * New option 'n_rxq_desc' and 'n_txq_desc' fields for DPDK interfaces which set the num

Re: [ovs-dev] [PATCH v2] ovn-northd: support IPAM with externally specified MAC

2016-10-06 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 03:58:51PM -0400, Lance Richardson wrote: > The current IPAM implementation allocates both a MAC address and > an IPv4 address when dynamic address allocation is requested. This > patch adds the ability to specify a fixed MAC address for use with > dynamic IPv4 address alloc

[ovs-dev] [PATCH v4] ovn-controller: add quiet mode

2016-10-06 Thread Ryan Moats
As discussed in [1], what the incremental processing code actually accomplished was that the ovn-controller would be "quiet" and not burn CPU when things weren't changing. This patch set recreates this state by calculating whether changes have occured that would require a full calculation to be per

Re: [ovs-dev] ovn-controller: honor ovs_idl_txn when calculating and installing flows.

2016-10-06 Thread Ben Pfaff
On Wed, Oct 05, 2016 at 08:55:03PM -0500, Ryan Moats wrote: > ovs_idl_txn is checked before various routines (like patch_run) execute. > However, flow calculation and installation does not also check this > variable, which can lead to oscillations as described in [1]. > > [1] http://openvswitch.or

Re: [ovs-dev] [PATCH v2] ovn: Make ipam tests more reliable

2016-10-06 Thread Ben Pfaff
On Thu, Oct 06, 2016 at 10:13:27AM -0700, nickcooper-zhangtonghao wrote: > CC: Nimay Desai > Fixes: 8639f9be("ovn-northd, tests: Adding IPAM to ovn-northd.") > The IPAM tests began to fail occasionally. Adding --wait=sb > to commands triggering address allocation eliminated these failures. > > Si

Re: [ovs-dev] [PATCH] ofproto: Always delete rules before deleting a meter.

2016-10-06 Thread Petr Machata
Jarno Rajahalme writes: > Pushed to master, but not sure if need to backport to earlier > branches, as meters are not really supported by any release yet? Nope, we only hit this because our own ofproto provider actually does support meters. Thanks for this! Petr ___

[ovs-dev] [PATCH v2] ovn-northd: support IPAM with externally specified MAC

2016-10-06 Thread Lance Richardson
The current IPAM implementation allocates both a MAC address and an IPv4 address when dynamic address allocation is requested. This patch adds the ability to specify a fixed MAC address for use with dynamic IPv4 address allocation. Example: ovn-nbctl lsp-set-addresses p1 "00:01:02:03:04:05 dyna

Re: [ovs-dev] [PATCH 1/4] ovn: make ipam tests more reliable

2016-10-06 Thread Ben Pfaff
I see it now, thanks. On Fri, Oct 07, 2016 at 01:09:25AM +0800, nickcooper-zhangtonghao wrote: > Yes, I submitted the v2 patch. I can not see also it in > http://patchwork.ozlabs.org/project/openvswitch/list/ > > > There may an error in pat

[ovs-dev] [patch_v4] ovn: Add additional comments regarding arp responders.

2016-10-06 Thread Darrell Ball
There has been enough confusion regarding logical switch datapath arp responders in ovn to warrant some additional comments; hence add a general description regarding why they exist and document the special cases. Signed-off-by: Darrell Ball --- Note this patch is meant to be merge with the code

Re: [ovs-dev] [PATCH ovs RFC 0/9] Introducing HW offload support for openvswitch

2016-10-06 Thread Rony Efraim
From: Joe Stringer [mailto:j...@ovn.org] Sent: Thursday, October 06, 2016 5:06 AM > > Subject: Re: [ovs-dev] [PATCH ovs RFC 0/9] Introducing HW offload support for > openvswitch > > On 27 September 2016 at 21:45, Paul Blakey wrote: > > Openvswitch currently configures the kerenel datapath via

Re: [ovs-dev] [PATCH 1/4] ovn: make ipam tests more reliable

2016-10-06 Thread nickcooper-zhangtonghao
Yes, I submitted the v2 patch. I can not see also it in http://patchwork.ozlabs.org/project/openvswitch/list/ There may an error in patchwork. I will send directly the patch to you. > On Oct 6, 2016, at 11:34 PM, Ben Pfaff wrote: > > Are

Re: [ovs-dev] [PATCH 1/4] ovn: make ipam tests more reliable

2016-10-06 Thread Ben Pfaff
Are you sure? I do not see it. Thanks, Ben. On Thu, Oct 06, 2016 at 06:55:02PM +0800, nickcooper-zhangtonghao wrote: > Good idea. I have submitted v2 patch. > > On Oct 6, 2016, at 8:30 AM, Ben Pfaff wrote: > > > > Thanks for working on this. > > > > When there are multiple ovn-nbctl calls in

Re: [ovs-dev] [Backport Request] For branch-2.5, backport rhel-systemd integration

2016-10-06 Thread Aaron Conole
Ben Pfaff writes: > On Wed, Oct 05, 2016 at 11:09:39AM -0400, Aaron Conole wrote: >> Ben Pfaff writes: >> >> > On Thu, Sep 08, 2016 at 07:58:21PM +0100, Markos Chandras wrote: >> >> On 09/08/2016 05:50 PM, Aaron Conole wrote: >> >> > >> >> >> It sounds like new feature territory, but you do ma

[ovs-dev] Returned mail: see transcript for details

2016-10-06 Thread Post Office
The original message was received at Thu, 6 Oct 2016 16:55:14 +0530 from openvswitch.org [38.164.67.218] - The following addresses had permanent fatal errors - - Transcript of session follows - ... while talking to 119.159.227.86: 550 5.1.2 ... Host unknown (Name server: host no

Re: [ovs-dev] [PATCH 1/4] ovn: make ipam tests more reliable

2016-10-06 Thread nickcooper-zhangtonghao
Good idea. I have submitted v2 patch. > On Oct 6, 2016, at 8:30 AM, Ben Pfaff wrote: > > Thanks for working on this. > > When there are multiple ovn-nbctl calls in a row, it's ordinarily only > necessary to add --wait to the last call. Can you make that change?

[ovs-dev] [PATCH 1/1] doc: Update DPDK pdump documentation

2016-10-06 Thread Mark Kavanagh
The DPDK pdump sample app was renamed from 'dpdk_pdump' to 'dpdk-pdump'. Update references to same within INSTALL.DPDK-ADVANCED.md. Add an additional sample command line that shows how to capture all traffic traversing an interface within a single pcap file - a useful tool for debugging DPDK-relat