Re: [ovs-dev] [PATCH 0/2] travis: add OS X support

2016-03-23 Thread Lance Richardson
A example of travis build results with these changes can be found here: https://travis-ci.org/hlrichardson/ovs-1/builds/118047167 - Original Message - > From: "Lance Richardson" > To: dev@openvswitch.org > Sent: Wednesday, March 23, 2016 4:04:45 PM > Subject

[ovs-dev] [PATCH] netdev: avoid null pointer dereference in netdev_get_addrs()

2016-03-30 Thread Lance Richardson
iven device.") Signed-off-by: Lance Richardson --- lib/netdev.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/netdev.c b/lib/netdev.c index 31998b3..a57c238 100644 --- a/lib/netdev.c +++ b/lib/netdev.c @@ -1884,6 +1884,10 @@ netdev_get_addrs(const char dev[], struct i

[ovs-dev] [PATCH] tests: fix *bsd failure in OVS_APP_EXIT_AND_WAIT

2016-03-31 Thread Lance Richardson
ss termination") Reported-by: Aaron Conole Signed-off-by: Lance Richardson --- tests/ovs-macros.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index 20003b8..e5710a0 100644 --- a/tests/ovs-macros.at +++ b/tests/ovs-macros.at @

Re: [ovs-dev] [PATCH v3] Dynamically reconnect ovn-controller if ovn-remote value changes

2016-04-05 Thread Lance Richardson
- Original Message - > From: "Ryan Moats" > To: dev@openvswitch.org > Sent: Tuesday, April 5, 2016 9:53:44 AM > Subject: [ovs-dev] [PATCH v3] Dynamically reconnect ovn-controller if > ovn-remote value changes > > From: RYAN D. MOATS > > Allows for auto detection and reconnect if the ov

Re: [ovs-dev] [PATCH v3] Dynamically reconnect ovn-controller if ovn-remote value changes

2016-04-05 Thread Lance Richardson
- Original Message - > From: "Ryan Moats" > To: "Lance Richardson" > Cc: dev@openvswitch.org > Sent: Tuesday, April 5, 2016 11:20:52 AM > Subject: Re: [ovs-dev] [PATCH v3] Dynamically reconnect ovn-controller if > ovn-remote value changes &g

[ovs-dev] [RFC] ovn: minimize the impact of a compromised chassis

2016-08-16 Thread Lance Richardson
Soliciting feedback/discussion to sanity-check proposed solution (described below) before implementation. Thanks,    Lance --- Limiting the impact of a compromised chassis. Problem Description: --- Each ovn-controller instance currently has full write access to the OVN southbou

[ovs-dev] [PATCH] ovn: include ovn-trace.8 in DISTCLEANFILES

2016-08-16 Thread Lance Richardson
Found by 'make distcheck'. Fixes: 4acd1e87a8bf ("ovn-trace: New utility.") Signed-off-by: Lance Richardson --- ovn/utilities/automake.mk | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ovn/utilities/automake.mk b/ovn/utilities/automake.mk index aae

Re: [ovs-dev] [PATCH] ovn: include ovn-trace.8 in DISTCLEANFILES

2016-08-16 Thread Lance Richardson
> From: "Lance Richardson" > To: dev@openvswitch.org > Sent: Tuesday, August 16, 2016 3:11:28 PM > Subject: [ovs-dev] [PATCH] ovn: include ovn-trace.8 in DISTCLEANFILES > > Found by 'make distcheck'. > > Fixes: 4acd1e87a8bf ("ovn-trace: New utili

Re: [ovs-dev] Mac OS X?

2016-08-19 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Martin Segeth" > Cc: dev@openvswitch.org > Sent: Friday, August 19, 2016 3:52:05 PM > Subject: Re: [ovs-dev] Mac OS X? > > On Fri, Aug 19, 2016 at 09:44:50PM +0200, Martin Segeth wrote: > > I was trying to build OVS 2.5.0 on a Mac but get a few a errors. While > > rese

Re: [ovs-dev] [PATCH 1/3] OVN-Tutorial: Replace example with ASCII quotes.

2016-08-22 Thread Lance Richardson
t; +$ make sandbox SANDBOXFLAGS="--ovn" > > Running the sandbox with OVN enabled does the following additional steps to > the > environment: Nice, I fell into this trap the first time I tried to run the ovn tutorials. Acked-by: Lance Richardson ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [RFC] ovn: minimize the impact of a compromised chassis

2016-08-22 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Russell Bryant" > Cc: "Lance Richardson" , "ovs dev" > , "Russell Bryant" > Sent: Monday, August 22, 2016 1:22:43 PM > Subject: Re: [ovs-dev] [RFC] ovn: minimize the impact of a compromised chassis

[ovs-dev] [PATCH] fedora.spec: include ovn-trace

2016-08-23 Thread Lance Richardson
Include ovn-trace and accompanying man page in fedora/rhel rpm. Fixes: 4acd1e87a8bf ("ovn-trace: New utility.") Signed-off-by: Lance Richardson --- rhel/openvswitch-fedora.spec.in | 2 ++ 1 file changed, 2 insertions(+) diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/o

Re: [ovs-dev] [PATCH] fedora.spec: include ovn-trace

2016-08-24 Thread Lance Richardson
> Thanks for the fix! Babu submitted the same patch a little while before > you did, so I'm going to apply that one. > > https://patchwork.ozlabs.org/patch/661860/ > > -- > Russell Bryant > Sorry for the noise, didn't think to check the queue. I'm glad to see it fixed! Lance __

Re: [ovs-dev] [net] openvswitch: Allow deferred action fifo to expand during run time

2016-08-24 Thread Lance Richardson
> From: "David Miller" > To: az...@ovn.org > Cc: d...@openvswitch.com, net...@vger.kernel.org > Sent: Friday, March 18, 2016 5:19:09 PM > Subject: Re: [net] openvswitch: Allow deferred action fifo to expand during > run time > > From: Andy Zhou > Date: Thu, 17 Mar 2016 21:32:13 -0700 > > > Cur

[ovs-dev] [PATCH] system-traffic: Ignore "no reference for recirc flow" in ct controller test

2016-08-29 Thread Lance Richardson
for the kernel data path. This log message means that an upcall with a recirc reference was received when the associated recirc id was expired/expiring. Eliminate the failure cause by ignoring the "no reference for recirc flow" message when checking vswitchd log. Signed-off-by: Lance

Re: [ovs-dev] [PATCH] system-traffic: Ignore "no reference for recirc flow" in ct controller test

2016-08-29 Thread Lance Richardson
> From: "Jarno Rajahalme" > To: "Lance Richardson" > Cc: dev@openvswitch.org > Sent: Monday, August 29, 2016 4:41:28 PM > Subject: Re: [ovs-dev] [PATCH] system-traffic: Ignore "no reference for > recirc flow" in ct controller test > > La

[ovs-dev] [PATCH v2] upcall: reduce log level for "no reference to recirc flow" message

2016-08-29 Thread Lance Richardson
Reduce log level from "warn" to "debug" for "upcall: no reference to recirc flow" log message. Suggested-by: Jarno Rajahalme Signed-off-by: Lance Richardson --- v2: Instead of ignoring log message in system-traffic.at, reduce level so it no longer appears in th

[ovs-dev] [PATCH] system-traffic: add a bonding test case

2016-08-30 Thread Lance Richardson
Add a test case to check connectivity over an OVS bond, using a Linux bond over veth interfaces. Also added a new macro "ADD_VETH_BOND", modeled after "ADD_VETH", in anticipation of future additional bonding test cases. Signed-off-by: Lance Richardson --- tests/system-c

[ovs-dev] [PATCH 0/3] support kernel datapath check when building rpms

2016-09-01 Thread Lance Richardson
atapath_kernel" option is provided on rpmbuild command line. One use case is to be able to download the source RPM to a test system and easily execute kernel datapath tests in addition to the normal test suite as a side-effect of building the binary RPMs. Lance Richardson (3): che

[ovs-dev] [PATCH 1/3] check-kernel: add recheck support

2016-09-01 Thread Lance Richardson
Support "make check-kernel RECHECK=yes", similar to existing support for "make check RECHECK=yes". Signed-off-by: Lance Richardson --- tests/automake.mk | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/automake.mk b/tests/automake.mk index 5d1

[ovs-dev] [PATCH 3/3] rhel: add option to run kernel datapath test when building rpms

2016-09-01 Thread Lance Richardson
production environments (it is generally not recommended to run rpmbuild as root). Suggested-by: Flavio Leitner Signed-off-by: Lance Richardson --- rhel/openvswitch-fedora.spec.in | 16 +++- rhel/openvswitch.spec.in| 17 - 2 files changed, 31 insertions(+), 2 deletion

[ovs-dev] [PATCH 2/3] rhel: build source rpm packages

2016-09-01 Thread Lance Richardson
Build source RPM packages for rpm-fedora and rpm-fedora-kmod targets. Signed-off-by: Lance Richardson --- rhel/automake.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rhel/automake.mk b/rhel/automake.mk index a3c180c..541f3fc 100644 --- a/rhel/automake.mk +++ b/rhel

Re: [ovs-dev] [PATCH 3/3] rhel: add option to run kernel datapath test when building rpms

2016-09-01 Thread Lance Richardson
> From: "Joe Stringer" > To: "Lance Richardson" > Cc: "ovs dev" > Sent: Thursday, September 1, 2016 4:51:19 PM > Subject: Re: [ovs-dev] [PATCH 3/3] rhel: add option to run kernel datapath > test when building rpms > > On 1 September 2016

[ovs-dev] [PATCH] check-system-userspace: add recheck support

2016-09-02 Thread Lance Richardson
Add support for "make check-system-userspace RECHECK=yes", similar to existing support for "make check RECHECK=yes". Signed-off-by: Lance Richardson --- tests/automake.mk | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/automake.mk b/tests/autom

[ovs-dev] [PATCH 0/2] support kernel datapath check when building rpms

2016-09-02 Thread Lance Richardson
mmand line. One use case is to be able to download the source RPM to a test system and easily execute kernel datapath tests in addition to the normal test suite as a side-effect of building the binary RPMs. Lance Richardson (2): rhel: build source rpm packages rhel: add option to run kerne

[ovs-dev] [PATCH 1/2] rhel: build source rpm packages

2016-09-02 Thread Lance Richardson
Build source RPM packages for rpm-fedora and rpm-fedora-kmod targets. Signed-off-by: Lance Richardson --- v1->v2: No changes. rhel/automake.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rhel/automake.mk b/rhel/automake.mk index a3c180c..541f3fc 100644 --- a/r

[ovs-dev] [PATCH 2/2] rhel: add option to run kernel datapath test when building rpms

2016-09-02 Thread Lance Richardson
production environments (it is generally not recommended to run rpmbuild as root). Suggested-by: Flavio Leitner Signed-off-by: Lance Richardson --- v1->v2: Added comments about testing with out-of-tree modules. rhel/openvswitch-fedora.spec.in | 20 +++- rhel/openvswitch.spec.i

Re: [ovs-dev] [PATCH 0/2] support kernel datapath check when building rpms

2016-09-02 Thread Lance Richardson
Apologies, this series should have been marked "PATCH v2". - Original Message - > From: "Lance Richardson" > To: dev@openvswitch.org > Cc: f...@sysclose.org > Sent: Friday, September 2, 2016 9:55:25 AM > Subject: [ovs-dev] [PATCH 0/2] support

Re: [ovs-dev] [PATCH 0/2] support kernel datapath check when building rpms

2016-09-08 Thread Lance Richardson
Hi Flavio, Could you review this series? Thanks! Lance - Original Message - > From: "Lance Richardson" > To: dev@openvswitch.org > Cc: f...@sysclose.org > Sent: Friday, September 2, 2016 9:55:25 AM > Subject: [ovs-dev] [PATCH 0/2] support kernel dat

[ovs-dev] [PATCH net-next] openvswitch: avoid deferred execution of recirc actions

2016-09-13 Thread Lance Richardson
are three or fewer recursions through ovs_execute_actions(). Suggested-by: Pravin Shelar Signed-off-by: Lance Richardson --- net/openvswitch/actions.c | 30 -- 1 file changed, 28 insertions(+), 2 deletions(-) diff --git a/net/openvswitch/actions.c b/net/openvswitch

[ovs-dev] backport request for 2.6: enable kernel datapath check in rpms

2016-09-13 Thread Lance Richardson
Requesting backports of these commits to the 2.6 branch:     commit 8ef22bb1ceb7b2841d2e29eb283387f0a3a67ff9         rhel: add option to run kernel datapath test when building rpms     commit ed71ac3e23e472378b1d292fcd8d3c43e46fe153         check-kernel: add recheck support These appear safe, an

Re: [ovs-dev] backport request for 2.6: enable kernel datapath check in rpms

2016-09-14 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Russell Bryant" > Cc: "ovs dev" , "Flavio Leitner" , > "Lance Richardson" > Sent: Wednesday, September 14, 2016 1:50:33 PM > Subject: Re: [ovs-dev] backport request for 2.6: enable kernel datapath c

[ovs-dev] [PATCH] ovn: make ipam tests more reliable

2016-09-14 Thread Lance Richardson
-by: Lance Richardson --- tests/ovn.at | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/ovn.at b/tests/ovn.at index 5707f47..1fcd666 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -4436,7 +4436,7 @@ AT_CHECK([ovn-nbctl get Logical-Switch-Port p0

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

2016-09-14 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: dev@openvswitch.org > Sent: Wednesday, September 14, 2016 8:03:16 PM > Subject: Re: [ovs-dev] [PATCH] ovn: make ipam tests more reliable > > On Wed, Sep 14, 2016 at 07:30:50PM -0400, Lance Richardso

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

2016-09-15 Thread Lance Richardson
04:05 dynamic" Signed-off-by: Lance Richardson --- ovn/lib/ovn-util.c | 38 ++ ovn/lib/ovn-util.h | 2 +- ovn/northd/ovn-northd.c | 30 +++--- ovn/ovn-nb.xml | 35 --- te

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

2016-09-19 Thread Lance Richardson
04:05 dynamic" Signed-off-by: Lance Richardson --- v2: Eliminated compilation warning, dropped usage of isspace(). ovn/lib/ovn-util.c | 38 ++ ovn/lib/ovn-util.h | 2 +- ovn/northd/ovn-northd.c | 30 +++--- ovn/

[ovs-dev] [PATCH] datapath: avoid deferred execution of recirc actions

2016-09-20 Thread Lance Richardson
f43e6dfb056b58628e43179d8f6b59eae417754d Author: Lance Richardson Date: Mon Sep 12 17:07:23 2016 -0400 openvswitch: avoid deferred execution of recirc actions The ovs kernel data path currently defers the execution of all recirc actions until stack utilization is at a minimum. This is too limiting for some

Re: [ovs-dev] [PATCH] travis: Work around glibtoolize error.

2016-09-21 Thread Lance Richardson
ninstall libtool && brew install libtool || true > -- > 2.9.3 > LGTM, although I wonder what changed in the base image and hope we don't see similar drift in the future... Acked-by: Lance Richardson ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-09-22 Thread Lance Richardson
04:05 dynamic" Signed-off-by: Lance Richardson --- ovn/lib/ovn-util.c | 38 ++ ovn/lib/ovn-util.h | 2 +- ovn/northd/ovn-northd.c | 30 +++--- ovn/ovn-nb.xml | 35 --- te

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

2016-10-03 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Russell Bryant" > Cc: "Lance Richardson" , "ovs dev" > Sent: Monday, October 3, 2016 4:24:22 PM > Subject: Re: [ovs-dev] [PATCH] ovn-northd: support IPAM with externally > specified MAC > > On Mon, Oc

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

2016-10-06 Thread Lance Richardson
04:05 dynamic" Acked-by: Ben Pfaff Acked-by: Russell Bryant Signed-off-by: Lance Richardson --- v2: - Simplified is_dynamic_lsp_address() using implementation by Ben Pfaff (with minor adjustment, removed a space from scan string). - Use ovs_scan() to extract sta

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:51

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-13 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Andy Zhou" > Cc: "ovs dev" , "Numan Siddique" , > "Babu Shanmugam" , > "Lance Richardson" , "Justin Pettit" , > "Russell Bryant" > Sent: Thursday, October 13, 2016

[ovs-dev] [RFC] ovsdb: implement read-only remote connection type

2016-10-13 Thread Lance Richardson
n ok 1880: ovsdb-server/read-only unix connection ok Shouldn't they all match the "read-only" keyword?? Signed-off-by: Lance Richardson --- lib/stream-ssl.c | 11 +- lib/stream-tcp.c

[ovs-dev] [RFC v2] ovsdb: implement read-only remote connection type

2016-10-13 Thread Lance Richardson
n ok 1880: ovsdb-server/read-only unix connection ok Shouldn't they all match the "read-only" keyword?? Signed-off-by: Lance Richardson --- v2: resending with corrected cc: list lib/stream-ssl.c | 11 +- lib/stream

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-13 Thread Lance Richardson
> From: "Andy Zhou" > To: "Ben Pfaff" > Cc: "ovs dev" , "Numan Siddique" , > "Babu Shanmugam" , > "Lance Richardson" , "Justin Pettit" , > "Russell Bryant" > Sent: Thursday, October 13, 2016

Re: [ovs-dev] [RFC] ovsdb: implement read-only remote connection type

2016-10-13 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: dev@openvswitch.org, jpe...@ovn.org > Sent: Thursday, October 13, 2016 5:08:42 PM > Subject: Re: [ovs-dev] [RFC] ovsdb: implement read-only remote connection type > > On Thu, Oct 13, 2016 at 03:08:46PM

[ovs-dev] [PATCH] ovsdb: implement read-only remote connection type

2016-10-14 Thread Lance Richardson
--remote=ssl:ro:192.168.0.10: Operations that would alter the state of the database are not permitted on connections for which the "ro" access-type is specified. Signed-off-by: Lance Richardson --- RFC->v1: - Added autotest keywords for ssl tests. - Changed active tcp

[ovs-dev] [PATCH v2] ovsdb: implement read-only remote connection type

2016-10-14 Thread Lance Richardson
--remote=ssl:ro:192.168.0.10: Operations that would alter the state of the database are not permitted on connections for which the "ro" access-type is specified. Signed-off-by: Lance Richardson --- RFC->v1: - Added autotest keywords for ssl tests. - Changed active tcp

Re: [ovs-dev] [PATCH v2] ovsdb: implement read-only remote connection type

2016-10-18 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: dev@openvswitch.org, az...@ovn.org, nusid...@redhat.com, > bscha...@redhat.com, jpet...@ovn.org, russ...@ovn.org > Sent: Monday, October 17, 2016 2:35:22 PM > Subject: Re: [PATCH v2] ovsdb: implement read-o

Re: [ovs-dev] ovn: Improving southbound database security

2016-10-21 Thread Lance Richardson
> From: "Ben Pfaff" > To: "Russell Bryant" > Cc: "ovs dev" > Sent: Friday, October 21, 2016 4:33:33 PM > Subject: Re: [ovs-dev] ovn: Improving southbound database security > > On Fri, Oct 21, 2016 at 04:10:58PM -0400, Russell Bryant wrote: > > On Thu, Oct 20, 2016 at 5:52 PM, Han Zhou wrote: >

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

2016-10-25 Thread Lance Richardson
deal." > > -* IPAM enhancements. > - > -Russell Bryant: "Be able to specify a MAC and only have the IP be > -dynamic. Right now both are dynamic." > - > * Native DNS support > > Russell Bryant: "This is

[ovs-dev] [PATCH 0/2] ovn: Implement read-only connection support for sb db

2016-10-25 Thread Lance Richardson
Add read-only connection support for "remote=db:" connection type to ovsdb-server. Implement connection table with read-only attribute in southbound db. Lance Richardson (2): ovsdb: implement read-only remote connection type ovn-southbound: implement read-only connection support

[ovs-dev] [PATCH 2/2] ovn-southbound: implement read-only connection support

2016-10-25 Thread Lance Richardson
punix, ssl, tcp, unix). - SSL configuration table for southbound db. - Connection table for NB schema. Signed-off-by: Lance Richardson --- ovn/ovn-sb.ovsschema | 37 ++- ovn/ovn-sb.xml| 245 ++ ovn/utilities/ovn-sbctl.c |

[ovs-dev] [PATCH 1/2] ovsdb: implement read-only remote connection type

2016-10-25 Thread Lance Richardson
Adds a new "read_only" column for remote connections. Operations that would alter the state of the database are not permitted on connections for which the "read_only" column is set to "true". Signed-off-by: Lance Richardson --- ovsdb/jsonrpc-server.c | 7 --

Re: [ovs-dev] [PATCH 2/2] ovn-southbound: implement read-only connection support

2016-10-25 Thread Lance Richardson
> From: "Lance Richardson" > To: dev@openvswitch.org > Sent: Tuesday, October 25, 2016 12:38:49 PM > Subject: [ovs-dev] [PATCH 2/2] ovn-southbound: implement read-only > connection support > > Add a connection table to the southbound db schema, similar > t

[ovs-dev] [PATCH] python: ignore warnings from flake8 version 2.0

2016-01-08 Thread Lance Richardson
Ignore warnings emitted by flake8 version 2.0 (this is the version currently provided in the CentOS/RHEL 7 EPEL repository). Signed-off-by: Lance Richardson --- Makefile.am | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index ef921ee..2537532

[ovs-dev] [PATCH] system-traffic: use appropriate nc options for installed version

2016-01-21 Thread Lance Richardson
command-line option: --send-only Only send data, ignoring received; quit on EOF Signed-off-by: Lance Richardson --- tests/atlocal.in| 7 +++ tests/system-traffic.at | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/tests/atlocal.in b/tests/atlocal.in index 5815c6c..

Re: [ovs-dev] [PATCH] system-traffic: use appropriate nc options for installed version

2016-01-21 Thread Lance Richardson
- Original Message - > > > On Jan 21, 2016, at 1:19 PM, Joe Stringer wrote: > > > > On 21 January 2016 at 11:57, Lance Richardson wrote: > >> The "conntrack - ICMP related with NAT" test case currently fails > >> on systems using the

[ovs-dev] [PATCH v2 1/2] Revert "system-traffic: Remove netcat from ICMP test."

2016-01-22 Thread Lance Richardson
Revert previous workaround for netcat portability issue. This reverts commit dc55e9465511dee6c12dbf0edb4ce2d9af57cb15. Signed-off-by: Lance Richardson --- tests/system-traffic.at | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/tests/system-traffic.at b

[ovs-dev] [PATCH v2 0/2] system-traffic: use appropriate nc options for installed version

2016-01-22 Thread Lance Richardson
--send-only" command-line option when the nmap flavor is detected, and using "-q 1" otherwise (openbsd and traditional versions). Tested via "make check-kernel" on RHEL7 (nmap version of nc) and Debian 8.2 (openbsd version of nc). Lance Richardson (2): Revert "

[ovs-dev] [PATCH v2 2/2] system-traffic: use appropriate nc options for installed version

2016-01-22 Thread Lance Richardson
CMP related" test case, using appropriate nc options based on installed version. - Removed comments in system-traffic.at regarding the use of the -q option. Signed-off-by: Lance Richardson --- tests/atlocal.in| 7 +++ tests/system-traffic.at | 6 ++ 2 files changed, 9 in

[ovs-dev] [RFC] ovsdb-idl: improved error handling for non-existent db

2016-01-27 Thread Lance Richardson
this particular error can be improved. Signed-off-by: Lance Richardson --- lib/ovsdb-idl.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index 7e84138..3f4be51 100644 --- a/lib/ovsdb-idl.c +++ b/lib/ovsdb-idl.c @@ -453,6 +453,16 @@ ovsdb_idl_run(s

Re: [ovs-dev] [RFC] ovsdb-idl: improved error handling for non-existent db

2016-01-27 Thread Lance Richardson
- Original Message - > On Wed, Jan 27, 2016 at 02:46:07PM -0500, Lance Richardson wrote: > > A common error scenario with OVN is to attempt to use ovn-nbctl when > > the OVN databases have not been created: > >1. ovn-nbctl sends "get_schema" request for O

[ovs-dev] [PATCH] ovsdb-idl: log error in client when db schema does not exist on server

2016-01-28 Thread Lance Richardson
3. ovn-nbctl receives the error response in ovsdb_idl_run(), but takes no specific action. 4. ovn-nbctl hangs forever in IDL_S_SCHEMA_REQUESTED state (assuming a timeout wasn't requested on the command line). Add a log message to inform the user of this situation. Signe

[ovs-dev] [PATCH] lib: Fix netbsd compilation error

2016-02-15 Thread Lance Richardson
NetBSD requires to be included before . Without this fix we have: In file included from lib/netdev-vport.c:25:0: /usr/include/netinet/ip6.h:82:18: error: field 'ip6_src' has incomplete type /usr/include/netinet/ip6.h:83:18: error: field 'ip6_dst' has incomplete type

[ovs-dev] [PATCH] ovsdb-idl: improve error handling when schema is not available

2016-02-16 Thread Lance Richardson
entered when a negative response to a schema request is received. When in this state, ovsdb_idl_is_alive() now returns 'false', allowing clients (currently ovn-nbctl, ovn-sbctl, vtep-ctl, and ovs-vsctl) to detect this condition and exit with an appropriate error message. Signed-off-by: L

[ovs-dev] [PATCH] rhel: Add 'rpm-fedora' and 'rpm-fedora-kmod' targets

2016-02-19 Thread Lance Richardson
Add make targets for Fedora and RHEL7 RPMs, update INSTALL.Fedora.md to document their use Added distribution tarball and rpm build directory to .gitignore. Signed-off-by: Lance Richardson --- .gitignore| 2 + INSTALL.Fedora.md | 197

<    1   2