[ovs-dev] Pmgxwpdgtq

2016-06-03 Thread Mail Administrator
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 4/6] vagrant: Override box for libvirt provider.

2016-06-03 Thread Joe Stringer
On 23 May 2016 at 16:30, Andy Zhou wrote: > > > On Fri, May 20, 2016 at 11:49 AM, Joe Stringer wrote: >> >> The default vagrant box for fedora23 uses the images provided by Chef's >> "bento" project, which has support for 3 backend providers: parallels, >> virtualbox and vmware. However, they do

Re: [ovs-dev] [PATCH 5/6] vagrant: Ignore common ovs-dev.py paths for rsync.

2016-06-03 Thread Joe Stringer
On 23 May 2016 at 18:33, Joe Stringer wrote: > On 23 May 2016 at 16:25, Andy Zhou wrote: >> >> >> On Fri, May 20, 2016 at 11:49 AM, Joe Stringer wrote: >>> >>> Signed-off-by: Joe Stringer >>> --- >>> Vagrantfile | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/Vagrantfile b/Vag

Re: [ovs-dev] [PATCH 1/6] INSTALL.md: Document system-traffic tests.

2016-06-03 Thread Joe Stringer
On 2 June 2016 at 15:01, Ben Pfaff wrote: >> +Datapath traffic testing >> + >> + >> +Open vSwitch also includes a suite of tests specifically for datapath >> +functionality, which can be run against the userspace or kernel datapaths. >> +If you are developing datapath featu

[ovs-dev] (no subject)

2016-06-03 Thread The Post Office
The original message was included as attachment ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Resumed IPv6 packets from the controller are getting dropped

2016-06-03 Thread pravin shelar
On Fri, Jun 3, 2016 at 1:29 PM, Ben Pfaff wrote: > On Fri, Jun 03, 2016 at 08:25:53PM +0530, Numan Siddique wrote: >> I am working on native DHCPv6 in OVN. During my testing I have noticed that >> the DHCPv6 reply packets from the ovn-controller are dropped by the kernel >> for checksum errors. >>

Re: [ovs-dev] FW: OpenVSwitch Ext 112 Git Repository

2016-06-03 Thread Jean Tourrilhes
Vasundhara G wrote: > > Well, I was thinking about the big picture behind this > implementation, have searched a lot on internet but could not sum up > my findings. My implementation was just testing the OpenFlow feature EXT-112 is correct, nothing more, nothing less. It just tests that y

[ovs-dev] a job that changed my life

2016-06-03 Thread doy ned
Hello, Just wanted to know where I should send you the profits you'll be making? My team is all setup and ready to welcome you on board, so that we can start you on one of the most amazing pesos making journeys you'll ever take. >> CLICK HERE TO START YOUR JOURNEY Let's not waste another m

Re: [ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-06-03 Thread Jan Scheurich
On Thu, 2 Jun 2016 16:03:02 -0700 Ben Pfaff wrote: > > In the documentation, please note that this uses an Open vSwitch > extension to OpenFlow and that it only works with Open vSwitch 2.6 or > later. > I'm a bit lost where to put this information. Is there a documentation file that gathers al

[ovs-dev] [PATCH] tests: fix issue in use of OVS_APP_EXIT_AND_WAIT

2016-06-03 Thread Lance Richardson
Commit f9b11f2a09b4 introduced a loop to wait for process exit in OVS_APP_EXIT_AND_WAIT after the "exit" command has been sent. Unfortunately, this does not work for cases where a unixctl socket has to be used to send the "exit" command because the process ID cannot be determined from the socket pa

Re: [ovs-dev] [PATCH] OVN: datalog man page

2016-06-03 Thread FlaviofOvsMl
On Tue, May 17, 2016 at 7:31 PM, Yusheng Wang wrote: > > From 72e885fb895d740dc62d5ea42ced764cba527b2f Mon Sep 17 00:00:00 2001 > From: Yusheng Wang > Date: Tue, 17 May 2016 07:58:35 +0800 > Subject: [PATCH] OVN: datalog man page > > Signed-off-by: Yusheng Wang > > --- > Hi Yusheng, After lis

Re: [ovs-dev] [PATCH 0/5] Fixes for big-endian architectures

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 04:36:23PM +0200, Gerhard Stenzel wrote: > On 05/27/2016 02:17 AM, Ben Pfaff wrote: > >These patches fix all of the test failures on big-endian architectures. > >I tested on s390x, this machine; > > https://db.debian.org/machines.cgi?host=zelenka > > > >Ben Pfaff (5): >

Re: [ovs-dev] Resumed IPv6 packets from the controller are getting dropped

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 08:25:53PM +0530, Numan Siddique wrote: > I am working on native DHCPv6 in OVN. During my testing I have noticed that > the DHCPv6 reply packets from the ovn-controller are dropped by the kernel > for checksum errors. > > I am not seeing this issue with 4.6 kernels. Figured

Re: [ovs-dev] [PATCH v3 5/5] ovn: DNAT and SNAT on a gateway router.

2016-06-03 Thread Mickey Spiegel
Please see replies inline. -Guru Shetty wrote: - >To: Mickey Spiegel/San Jose/IBM@IBMUS >From: Guru Shetty >Date: 06/02/2016 10:41PM >Cc: ovs dev >Subject: Re: [ovs-dev] [PATCH v3 5/5] ovn: DNAT and SNAT on a gateway >router. > > >> >> Looking at the code, it looks like any IP packet

Re: [ovs-dev] [PATCH] FAQ: Explain that the order of actions is significant.

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 09:40:16AM -0700, Guru Shetty wrote: > On 3 June 2016 at 09:10, Ben Pfaff wrote: > > > I've seen users make this error several times. This FAQ will provide a > > useful answer to pass along. > > > > Signed-off-by: Ben Pfaff > > > Acked-by: Gurucharan Shetty Thanks, app

Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve support

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 07:15:43PM +, Nithin Raju wrote: > -Original Message- > From: Ben Pfaff > Date: Friday, June 3, 2016 at 11:55 AM > To: Nithin Raju > Cc: Yin Lin , "dev@openvswitch.org" > Subject: Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve > support > >

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

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 03:44:13PM -0400, Russell Bryant wrote: > On Fri, Jun 3, 2016 at 3:31 PM, Ben Pfaff wrote: > > > The name of the macro was wrong. > > > > Signed-off-by: Ben Pfaff > > > > Acked-by: Russell Bryant Thanks, applied. ___ dev mail

Re: [ovs-dev] [PATCH 0/6] create tunnel devices using rtnetlink interface

2016-06-03 Thread Ben Pfaff
On Thu, May 26, 2016 at 06:07:34PM -0300, Thadeu Lima de Souza Cascardo wrote: > I would still like to see any other comments on the patchset before I > send a v2 fixing that. I think I'll leave further review of this series to Jesse, unless he asks me to do it. Thanks, Ben.

Re: [ovs-dev] [PATCH 3/6] dpif-netlink: break code to add compat and non-compat vports

2016-06-03 Thread Ben Pfaff
On Thu, May 26, 2016 at 05:35:00PM -0300, Thadeu Lima de Souza Cascardo wrote: > The vport type for adding tunnels is now compatibility code and any new > features > from tunnels must configure the tunnel as an interface using the tunnel > metadata > support. > > In order to be able to add those

Re: [ovs-dev] [PATCH 2/6] netdev: get device type from vport prefix if not found

2016-06-03 Thread Ben Pfaff
On Thu, May 26, 2016 at 05:34:59PM -0300, Thadeu Lima de Souza Cascardo wrote: > If we cannot find the device type because it's not opened yet, check if it > uses > a reserved prefix for a vport type and return that type. > > Since these names are reserved, we can assume this is the right type. >

Re: [ovs-dev] [PATCH 1/6] route-table: if device is not there, route is still parseable

2016-06-03 Thread Ben Pfaff
On Thu, May 26, 2016 at 05:34:58PM -0300, Thadeu Lima de Souza Cascardo wrote: > Do not return failure to parse a route if device has been removed before we > are > able to parse the route. That prevents "received bad netlink message" warnings > on the log. > > This can be reproduced by simply re

Re: [ovs-dev] [PATCH] vswitch.xml: Document interface name length restrictions.

2016-06-03 Thread Ben Pfaff
On Tue, May 31, 2016 at 11:40:13AM -0500, Ryan Moats wrote: > "dev" wrote on 05/26/2016 12:30:39 PM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Ben Pfaff > > Date: 05/26/2016 12:30 PM > > Subject: [ovs-dev] [PATCH] vswitch.xml: Document interface name > > length restrictions. > >

Re: [ovs-dev] [PATCH v2 2/2] DSCP marking on packets egressing VIF interface

2016-06-03 Thread Ben Pfaff
On Wed, May 25, 2016 at 03:35:45PM +0530, bscha...@redhat.com wrote: > ovn-northd sets 'ip.dscp' to the DSCP value > > Signed-off-by: Babu Shanmugam Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/d

Re: [ovs-dev] [PATCH v2 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-06-03 Thread Ben Pfaff
On Wed, May 25, 2016 at 03:35:44PM +0530, bscha...@redhat.com wrote: > ovn-northd processes the list of Port_Bindings and hashes the list of > queues per chassis. When it finds a port with qos_parameters and without > a queue_id, it allocates a free queue for the chassis that this port belongs. > T

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

2016-06-03 Thread Russell Bryant
On Fri, Jun 3, 2016 at 3:31 PM, Ben Pfaff wrote: > The name of the macro was wrong. > > Signed-off-by: Ben Pfaff > Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] netdev: Fix typo in comment.

2016-06-03 Thread Ben Pfaff
The name of the macro was wrong. Signed-off-by: Ben Pfaff --- lib/netdev.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/netdev.h b/lib/netdev.h index cdefcd5..591d861 100644 --- a/lib/netdev.h +++ b/lib/netdev.h @@ -49,7 +49,7 @@ extern "C" { * create multiple n

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Lance Richardson" > Cc: "Joe Stringer" , dev@openvswitch.org, "Paul Boca" > > Sent: Friday, June 3, 2016 3:14:55 PM > Subject: Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of > OVS_APP_EXIT_AND_WAIT on Windows > > > > Another thought

Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve support

2016-06-03 Thread Nithin Raju
-Original Message- From: Ben Pfaff Date: Friday, June 3, 2016 at 11:55 AM To: Nithin Raju Cc: Yin Lin , "dev@openvswitch.org" Subject: Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve support >I wasn't sure how to interpret this. Does this mean that you will >review th

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 02:51:54PM -0400, Lance Richardson wrote: > > > - Original Message - > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: "Joe Stringer" , dev@openvswitch.org, "Paul Boca" > > > > Sent: Friday, June 3, 2016 2:31:30 PM > > Subject: Re: [ovs-dev] [PATCH V2 4/

Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve support

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 11:28:05PM +, Yin Lin wrote: > Signed-off-by: Yin Lin > --- > Fixed checksum flag issue brought up by Jesse and automake.mk issue by Nithin. This patch fails to apply, so I think that it will need to be rebased and re-sent. _

Re: [ovs-dev] [PATCH 2/2] [PATCH v5] datapath-windows: Add Geneve support

2016-06-03 Thread Ben Pfaff
I wasn't sure how to interpret this. Does this mean that you will review this patch later, or does it mean something else? Thanks, Ben. On Wed, Jun 01, 2016 at 04:48:58AM +, Nithin Raju wrote: > Yin, > I went ahead and sent out a patch for the updates w.r.t > OvsFindTunnelVportByDstPort().

Re: [ovs-dev] [PATCH V6] ovn-controller: reload configured SB probe timer

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 04:14:28PM -0700, ngh...@us.ibm.com wrote: > The probe timer between ovn-controller and OVN Southbound > can be configured using ovn-vsctl command, but that is not > effective on the fly. In other words, ovn-controller has > to be restarted to use that probe_timer value, thi

Re: [ovs-dev] [PATCH] ovs-lib: disable probe for local connections

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 03:47:20PM -0700, ngh...@us.ibm.com wrote: > There are four sessions established from ovn-controller to the following: > OVN Southbound — JSONRPC based > Local ovsdb — JSONRPC based > Local vswitchd — openflow based from ofctrl > Local vswitchd — openflow based from pinctrl

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Lance Richardson" > Cc: "Joe Stringer" , dev@openvswitch.org, "Paul Boca" > > Sent: Friday, June 3, 2016 2:31:30 PM > Subject: Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of > OVS_APP_EXIT_AND_WAIT on Windows > > On Fri, Jun 03, 2016

Re: [ovs-dev] [PATCH V5] add lrouter and lrport related commands to ovn-nbctl

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 03:18:18PM -0700, ngh...@us.ibm.com wrote: > ovn-nbctl provides a shortcut to perform commands related lswitch, lport > and such but it doesn't have similar commands related to logical routers > and logical router ports. Also, 'ovn-nbctl show' is supposed to show an > overvi

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Support user-defined socket attribs

2016-06-03 Thread Aaron Conole
Christian Ehrhardt writes: > On Tue, May 24, 2016 at 4:10 PM, Aaron Conole wrote: > >> Daniele Di Proietto writes: >> >> > Hi Aaron, >> > >> > I'm still a little bit nervous about calling chown on a (partially) >> > user controlled file name. >> >> I agree, that always seems scary. >> >> > Befo

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 12:49:54PM -0400, Lance Richardson wrote: > - Original Message - > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: "Joe Stringer" , dev@openvswitch.org, "Paul Boca" > > > > Sent: Friday, June 3, 2016 12:38:56 PM > > Subject: Re: [ovs-dev] [PATCH V2 4/4] te

Re: [ovs-dev] [PATCH] INSTALL.DPDK: Replace tabs with spaces

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 03:13:30PM +0100, Ciara Loftus wrote: > Signed-off-by: Ciara Loftus Applied, thanks. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v4] Add configurable OpenFlow port name.

2016-06-03 Thread Ben Pfaff
On Tue, May 24, 2016 at 03:07:23PM +0800, Xiao Liang wrote: > Add new column "ofname" in Interface table to configure port name reported > to controllers with OpenFlow protocol, thus decouple OpenFlow port name from > device name. > > For example: > # ovs-vsctl set Interface eth0 ofname=wan >

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Lance Richardson" > Cc: "Joe Stringer" , dev@openvswitch.org, "Paul Boca" > > Sent: Friday, June 3, 2016 12:38:56 PM > Subject: Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of > OVS_APP_EXIT_AND_WAIT on Windows > > On Fri, Jun 03, 2016 a

Re: [ovs-dev] [PATCH v3 5/5] ovn: DNAT and SNAT on a gateway router.

2016-06-03 Thread Guru Shetty
> > >> As mentioned above, in OpenStack, if the source IP address is a fixed IP >> that has a corresponding floating IP, then the source IP address should be >> mapped to the floating IP address rather than the router gateway address. >> >> Separate dnat and snat lists as defined here can be made t

Re: [ovs-dev] [PATCH] FAQ: Explain that the order of actions is significant.

2016-06-03 Thread Guru Shetty
On 3 June 2016 at 09:10, Ben Pfaff wrote: > I've seen users make this error several times. This FAQ will provide a > useful answer to pass along. > > Signed-off-by: Ben Pfaff > Acked-by: Gurucharan Shetty ___ dev mailing list dev@openvswitch.org http

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 12:00:11PM -0400, Lance Richardson wrote: > OK, it appears that I was probably a little too focused on the ~250 > locations where OVS_APP_EXIT_AND_WAIT() is invoked with a daemon > name (where the TMPPID stuff works fine) and not enough to the ~8 > locations where it's invok

Re: [ovs-dev] [PATCH v3 4/5] ovn-controller: Assign conntrack zones for gateway router.

2016-06-03 Thread Guru Shetty
On 3 June 2016 at 08:18, Ben Pfaff wrote: > On Thu, Jun 02, 2016 at 10:22:05PM -0700, Guru Shetty wrote: > > On 2 June 2016 at 14:19, Ben Pfaff wrote: > > > > > On Thu, May 19, 2016 at 01:02:33PM -0700, Gurucharan Shetty wrote: > > > > OVS NAT currently cannot do snat and dnat in the same zone.

[ovs-dev] [PATCH v2] datapath-windows: use ip proto for tunnel port lookup

2016-06-03 Thread Nithin Raju
In Actions.c, based on the IP Protocol type and L4 port of the outer packet, we lookup the tunnel port. The function that made this happen took the tunnel type as an argument. Semantically, is is better to pass the IP protocol type and let the lookup code map IP protocol type to tunnel type. In th

Re: [ovs-dev] [PATCH] datapath-windows: use ip proto for tunnel port lookup

2016-06-03 Thread Nithin Raju
Thanks for the review. From: Yin Lin mailto:yinli...@gmail.com>> Date: Thursday, June 2, 2016 at 3:00 PM To: Nithin Raju mailto:nit...@vmware.com>> Cc: "dev@openvswitch.org" mailto:dev@openvswitch.org>> Subject: Re: [ovs-dev] [PATCH] datapath-windows: use ip proto for

Re: [ovs-dev] [PATCH 0/6] Update xenserver scripts for newer python.

2016-06-03 Thread Ben Pfaff
On Mon, May 23, 2016 at 06:20:25PM -0700, Joe Stringer wrote: > Since commit e23812fc60af ("Increase prerequisite from Python 2.4 to > Python 2.7.") in OVS v2.5, we have required a newer version of python > to build and run OVS on all platforms, including Xenserver. However, > support for python3 w

[ovs-dev] [PATCH] FAQ: Explain that the order of actions is significant.

2016-06-03 Thread Ben Pfaff
I've seen users make this error several times. This FAQ will provide a useful answer to pass along. Signed-off-by: Ben Pfaff --- FAQ.md | 17 + 1 file changed, 17 insertions(+) diff --git a/FAQ.md b/FAQ.md index 5e95ccc..df6f225 100644 --- a/FAQ.md +++ b/FAQ.md @@ -2012,6 +2012

Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of OVS_APP_EXIT_AND_WAIT on Windows

2016-06-03 Thread Lance Richardson
- Original Message - > From: "Joe Stringer" > To: "Lance Richardson" > Cc: dev@openvswitch.org, "Paul Boca" > Sent: Thursday, June 2, 2016 9:41:23 PM > Subject: Re: [ovs-dev] [PATCH V2 4/4] tests: Fix fail of > OVS_APP_EXIT_AND_WAIT on Windows > > Bear with me, I'm a little confused -

Re: [ovs-dev] [PATCH 1/6] xenserver: Sort vsctl port options.

2016-06-03 Thread Ben Pfaff
On Mon, May 23, 2016 at 06:20:26PM -0700, Joe Stringer wrote: > In python3, dictionaries are less likely to be sorted consistently from > one run to the next, so sort port options when outputting to provide > reliable test results. > > Signed-off-by: Joe Stringer Thanks. Acked-by: Ben Pfaff __

Re: [ovs-dev] [ovs-dev, v17, 5/5] Add incremental proessing to lflow_run and physical_run

2016-06-03 Thread Ben Pfaff
On Sun, May 22, 2016 at 04:36:22PM -0500, Ryan Moats wrote: > This code changes to allow incremental processing of the > logical flow and physical binding tables whenver possible. > > Side Effects: > - Make flow table persistent in ovn controller > - Reset lflow processing when adding/removing

Re: [ovs-dev] [ovs-dev,v17,4/5] Persist ovn flow tables.

2016-06-03 Thread Ben Pfaff
On Sun, May 22, 2016 at 04:36:21PM -0500, Ryan Moats wrote: > Ensure that ovn flow tables are persisted so that changes to > them chan be applied incrementally - this is a prereq for > making lflow_run and physical_run incremental. > > Signed-off-by: Ryan Moats I don't really understand the desi

Re: [ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-06-03 Thread Ben Pfaff
On Fri, Jun 03, 2016 at 05:07:39PM +0200, Jan Scheurich wrote: > Thanks for the review. Will update. > > > In some places, this is called "add_or_modify" and in other places just > > "write". I suggest using a consistent name. > > Which one would you suggest? I don't like the idea of using "writ

Re: [ovs-dev] [PATCH v3 4/5] ovn-controller: Assign conntrack zones for gateway router.

2016-06-03 Thread Ben Pfaff
On Thu, Jun 02, 2016 at 10:22:05PM -0700, Guru Shetty wrote: > On 2 June 2016 at 14:19, Ben Pfaff wrote: > > > On Thu, May 19, 2016 at 01:02:33PM -0700, Gurucharan Shetty wrote: > > > OVS NAT currently cannot do snat and dnat in the same zone. > > > So we need two zones per gateway router. > > >

Re: [ovs-dev] [PATCH v2] ovn-northd.8.xml: fix sock path of NB and SB database.

2016-06-03 Thread Russell Bryant
On Wed, Jun 1, 2016 at 9:09 PM, Li Wei wrote: > commit 60bdd01148e4 ("Separating OVN NB and SB database processes") > introduced a separating OVN NB and SB database process, the path of > sock files need to be updated. > > Fixes: 60bdd01148e4 ("Separating OVN NB and SB database processes") > Sign

Re: [ovs-dev] [PATCH] ofproto: Add relaxed group_mod command ADD_OR_MOD

2016-06-03 Thread Jan Scheurich
Hi Ben, Thanks for the review. Will update. > In some places, this is called "add_or_modify" and in other places just > "write". I suggest using a consistent name. Which one would you suggest? BR, Jan ___ dev mailing list dev@openvswitch.org http://op

[ovs-dev] Resumed IPv6 packets from the controller are getting dropped

2016-06-03 Thread Numan Siddique
Hi, I am working on native DHCPv6 in OVN. During my testing I have noticed that the DHCPv6 reply packets from the ovn-controller are dropped by the kernel for checksum errors. I am not seeing this issue with 4.6 kernels. Figured out that its fixed in the latest kernels and the fix is here [1].

Re: [ovs-dev] [PATCH 0/5] Fixes for big-endian architectures

2016-06-03 Thread Gerhard Stenzel
On 05/27/2016 02:17 AM, Ben Pfaff wrote: These patches fix all of the test failures on big-endian architectures. I tested on s390x, this machine; https://db.debian.org/machines.cgi?host=zelenka Ben Pfaff (5): types: Change ofp_port_t from uint16_t to uint32_t. netdev-native-tnl:

[ovs-dev] [PATCH v9 2/2] ovn-northd: Add logical flows to support native DHCP

2016-06-03 Thread Numan Siddique
OVN implements a native DHCP support which caters to the common use case of providing an IP address to a booting instance by providing stateless replies to DHCP requests based on statically configured address mappings. To do this it allows a short list of DHCP options to be configured and applied a

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

2016-06-03 Thread Numan Siddique
This patch adds a new OVN action 'put_dhcp_opts' to support native DHCP in OVN. ovn-controller parses this action and adds a NXT_PACKET_IN2 OF flow with 'pause' flag set and the DHCP options stored in 'userdata' field. When the valid DHCP packet is received by ovn-controller, it frames a new DHCP

[ovs-dev] [PATCH v9 0/2] ovn: Support native DHCP using 'continuations'

2016-06-03 Thread Numan Siddique
v8 > v9 changes --- * Resolved the patch conflicts v7 -> v8 changes * Added warn logs in the failure cases in the function pinctrl_handle_put_dhcp_opts() as per Ramu' suggestion v6 -> v7 changes * In the patch 2 added "free(laddrs.ipv4_addrs)

[ovs-dev] [PATCH V3 3/4] tests: Modified OVS_APP_EXIT_AND_WAIT_BY_TARGET to work on windows too

2016-06-03 Thread Paul Boca
On Windows the CTL file doesn't contain the PID in it's name. Also this macro is used for other apps beside ovsdb-server. Signed-off-by: Paul-Daniel Boca --- V2: Changed this macro to accept any target besides ovsdb-server --- tests/ovs-macros.at | 6 +- 1 file changed, 5 insertions(+), 1 de

Re: [ovs-dev] [PATCH V2 3/4] tests: Modified OVS_APP_EXIT_AND_WAIT_BY_TARGET to work on windows too

2016-06-03 Thread Paul Boca
This should be V3 version. Disregard this one, I will send another one with the right version. > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Paul Boca > Sent: Friday, June 3, 2016 4:35 PM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH V2 3/4] te

[ovs-dev] [PATCH V2 3/4] tests: Modified OVS_APP_EXIT_AND_WAIT_BY_TARGET to work on windows too

2016-06-03 Thread Paul Boca
On Windows the CTL file doesn't contain the PID in it's name. Also this macro is used for other apps beside ovsdb-server. Signed-off-by: Paul-Daniel Boca --- V2: Changed this macro to accept any target besides ovsdb-server --- tests/ovs-macros.at | 6 +- 1 file changed, 5 insertions(+), 1 de

[ovs-dev] [PATCH] tests: Skip "daemon --service" test on Windows from non-admin console

2016-06-03 Thread Paul Boca
Check if we have enough rights to create a service on Windows otherwise we skip the test Signed-off-by: Paul-Daniel Boca --- tests/daemon.at | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/daemon.at b/tests/daemon.at index 41c5d07..d85bfc8 100644 --- a/tests/daemon.at +++

[ovs-dev] [PATCH] tests: Ignore ovs-appctl stdout for ovsdb-monitor tests

2016-06-03 Thread Paul Boca
Reverted OVS_APP_EXIT_AND_WAIT call for ovsdb-monitor tests, this doesn't ignores the output on stdout and stderr which causes the tests to fail. Signed-off-by: Paul-Daniel Boca --- tests/ovsdb-monitor.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ovsdb-monitor.at

[ovs-dev] [PATCH] tests: fixed ovn-controller - ovn-bridge-mappings test

2016-06-03 Thread Paul Boca
OVS_APP_EXIT_AND_WAIT_BY_TARGET([ovsdb-server-2]) expects to have an unixctl named ovsdb-server-2. --- tests/ovn-controller.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ovn-controller.at b/tests/ovn-controller.at index 13b3f53..35a55b6 100644 --- a/tests/ovn-contro

[ovs-dev] [PATCH] tests: Skip vlog close and set if no python is installed

2016-06-03 Thread Paul Boca
AT_CAPTURE_FILE([log.old]) and AT_CAPTURE_FILE([log]) will fail in case no python is installed on the system Signed-off-by: Paul-Daniel Boca --- tests/vlog.at | 4 1 file changed, 4 insertions(+) diff --git a/tests/vlog.at b/tests/vlog.at index b793611..c66c84a 100644 --- a/tests/vlog.at

[ovs-dev] [PATCH V3 1/4] tests: Fixed access denied on ovs-vswitchd.log

2016-06-03 Thread Paul Boca
On Windows trying to overwrite the opened ovs-vswitchd.log fails with access denied. Closing it before trying to overwrite it solves the problem Signed-off-by: Paul-Daniel Boca --- V2: Use same command both for Windows and Linux. Previous version had '.exe' extension at ovs-appctl. --- tests

Re: [ovs-dev] [PATCH v2 1/2] chutil: introduce a new change-utils lib

2016-06-03 Thread Aaron Conole
Ben Pfaff writes: > On Fri, May 20, 2016 at 04:32:05PM -0400, Aaron Conole wrote: >> It will be useful in the future to be able to set ownership and permissions >> on files which Open vSwitch creates. Allowing the specification of such >> ownership and permissions using the standard user:group, u

Re: [ovs-dev] [PATCH RFC v2 0/6] dpif-netdev: Manual pinning of RX queues + XPS.

2016-06-03 Thread Ilya Maximets
One addition below. On 02.06.2016 16:55, Ilya Maximets wrote: > Hi, Daniele. > Thanks for review. > > On 02.06.2016 04:33, Daniele Di Proietto wrote: >> Hi Ilya, >> >> apologies for the delay. >> >> I didn't take a extremely detailed look at this series, but I have >> a few high level comments. >