Re: [ovs-dev] [PATCH v4 3/3] ovn-northd: Add logical flows to support native DNS

2017-04-17 Thread Numan Siddique
On Sat, Apr 15, 2017 at 8:51 AM, Ben Pfaff wrote: > On Mon, Apr 03, 2017 at 04:10:27PM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > OVN implements native DNS resolution which can be used to resolve the > > internal DNS names belonging to a logical datapath. > > > > To suppo

[ovs-dev] django Business Directory

2017-04-17 Thread Irene Murillo
Hi, A quick follow up to know if you would be interested in django Users List. We provide data across the globe. We also provide other related Technology Users like. WordPress HubSpot SAP Episerver Oracle WebCenter AWS Oracle eBusiness Suite IBM FileNet QuickBooks Infor ERP Ope

Re: [ovs-dev] [PATCH 1/2] install-doc: suggest to use ovs-ctl for start/stop

2017-04-17 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Apr 13, 2017 at 01:51:29PM -0400, Aaron Conole wrote: >> The install documentation guided users to manually start/stop >> daemons. This is good information to have, but with the >> existence of ovs-ctl, is probably not the best way to start >> guiding new users of ovs

Re: [ovs-dev] [PATCH 1/2] install-doc: suggest to use ovs-ctl for start/stop

2017-04-17 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 10:48:35AM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > On Thu, Apr 13, 2017 at 01:51:29PM -0400, Aaron Conole wrote: > >> The install documentation guided users to manually start/stop > >> daemons. This is good information to have, but with the > >> existence of

Re: [ovs-dev] [PATCH] debian, xenserver: Update logrotate config to match RHEL.

2017-04-17 Thread Ben Pfaff
On Sun, Apr 16, 2017 at 09:10:59PM -0700, Greg Rose wrote: > On Sun, 2017-04-16 at 13:19 -0700, Ben Pfaff wrote: > > On Fri, Apr 14, 2017 at 10:41:01AM -0700, Greg Rose wrote: > > > On Fri, 2017-04-14 at 10:03 -0700, Ben Pfaff wrote: > > > > On Fri, Apr 14, 2017 at 09:05:41AM -0700, Greg Rose wrote

Re: [ovs-dev] [PATCH v2 1/2] ovn-northd: Add hint in lflow to link back to acl

2017-04-17 Thread Ben Pfaff
On Sat, Apr 15, 2017 at 12:12:36AM -0700, Han Zhou wrote: > On Fri, Apr 14, 2017 at 9:48 PM, Ben Pfaff wrote: > > > > On Thu, Mar 23, 2017 at 11:43:25PM -0700, Han Zhou wrote: > > > It will be helpful for trouble-shooting if we can link a logical flow > > > back to the ACL that generated it. This

[ovs-dev] [PATCH v5 0/3] ovn: Native DNS support

2017-04-17 Thread nusiddiq
From: Numan Siddique v4 -> v5 Addressed the review comments v3 -> v4 * PS1 (ovn-util: Add a new util function extract_ip_addresses) - Added this patch in v4 to extract the IP addresses from a string * PS2 (ovn-controller: Add 'dns_lookup' action) - Addressed the review

[ovs-dev] [PATCH v5 1/3] ovn-util: Add a new util function extract_ip_addresses

2017-04-17 Thread nusiddiq
From: Numan Siddique An upcoming commit will use this function to extract the IP (v4 and v6) addresses from a string without a preceding eth address. Signed-off-by: Numan Siddique Acked-by: Ben Pfaff --- ovn/lib/ovn-util.c | 72 ++ ovn/lib/o

[ovs-dev] [PATCH v5 2/3] ovn-controller: Add 'dns_lookup' action

2017-04-17 Thread nusiddiq
From: Numan Siddique This patch adds a new OVN action 'dns_lookup' to support native DNS. ovn-controller parses this action and adds a NXT_PACKET_IN2 OF flow with 'pause' flag set. A new table 'DNS' is added in the SB DB to look up and resolve the DNS queries. When a valid DNS packet is received

[ovs-dev] [PATCH v5 3/3] ovn-northd: Add logical flows to support native DNS

2017-04-17 Thread nusiddiq
From: Numan Siddique OVN implements native DNS resolution which can be used to resolve the internal DNS names belonging to a logical datapath. To support this, a new table 'DNS' is added in the NB DB. A new column 'dns_records' is added in 'Logical_Switch' table which references to the 'DNS' tab

Re: [ovs-dev] [PATCH] flow: Further refinements to flow_pop_vlan().

2017-04-17 Thread Eric Garver
On Fri, Apr 14, 2017 at 09:25:41PM -0700, Ben Pfaff wrote: > This may help to suppress warnings from know-it-all compilers, and it helps > to make the code clearer too. > > Reported-by: Darrell Ball > Signed-off-by: Ben Pfaff > --- > Sorry, I didn't see v2 of the memset patch until I'd already a

[ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Matthew Thode via dev
Fixes the following warning. WARNING: 'default' html theme has been renamed to 'classic'. Please change your html_theme setting either to the new 'alabaster' default theme, or to 'classic' to keep using the old default. As reported by https://bugs.gentoo.org/show_bug.cgi?id=614520 Signed-off-by:

[ovs-dev] Bug#860479: openvswitch-datapath-dkms (and openvswitch-datapath-source) fail compiling on fresh install of 7.11

2017-04-17 Thread Bogdan Ilisei
Package: openvswitch-datapath-dkms Version: 1.4.2+git20120612-9.1~deb7u1.1 Severity: grave Justification: renders package unusable Dear Maintainer, Installing openvswitch-datapath-dkms (OR openvswitch-datapath-source) is currently broken in Wheezy 7.11. I have tested both on a fresh install and

[ovs-dev] [PATCH] xenserver: Add install files to RPM file list

2017-04-17 Thread Greg Rose
Add missing intall files to RPM file list to prevent file check error so that RPM files will be built. Signed-off-by: Greg Rose --- xenserver/openvswitch-xen.spec.in | 4 1 file changed, 4 insertions(+) diff --git a/xenserver/openvswitch-xen.spec.in b/xenserver/openvswitch-xen.spec.in ind

Re: [ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 10:36:26AM -0500, Matthew Thode via dev wrote: > Fixes the following warning. > > WARNING: 'default' html theme has been renamed to 'classic'. Please change > your > html_theme setting either to the new 'alabaster' default theme, or to > 'classic' > to keep using the old

Re: [ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Matthew Thode via dev
On 04/17/2017 11:20 AM, Ben Pfaff wrote: > On Mon, Apr 17, 2017 at 10:36:26AM -0500, Matthew Thode via dev wrote: >> Fixes the following warning. >> >> WARNING: 'default' html theme has been renamed to 'classic'. Please change >> your >> html_theme setting either to the new 'alabaster' default the

Re: [ovs-dev] [PATCH] xenserver: Add install files to RPM file list

2017-04-17 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 09:18:53AM -0700, Greg Rose wrote: > Add missing intall files to RPM file list to prevent file check error so > that RPM files will be built. > > Signed-off-by: Greg Rose Thanks! I applied this to master. Would you mind figuring out similar fixes for branch-2.7 and bran

Re: [ovs-dev] [PATCH] xenserver: Add install files to RPM file list

2017-04-17 Thread Greg Rose
On Mon, 2017-04-17 at 09:38 -0700, Ben Pfaff wrote: > On Mon, Apr 17, 2017 at 09:18:53AM -0700, Greg Rose wrote: > > Add missing intall files to RPM file list to prevent file check error so > > that RPM files will be built. > > > > Signed-off-by: Greg Rose > > Thanks! I applied this to master.

Re: [ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 11:32:13AM -0500, Matthew Thode wrote: > On 04/17/2017 11:20 AM, Ben Pfaff wrote: > > On Mon, Apr 17, 2017 at 10:36:26AM -0500, Matthew Thode via dev wrote: > >> Fixes the following warning. > >> > >> WARNING: 'default' html theme has been renamed to 'classic'. Please change

Re: [ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Matthew Thode via dev
On 04/17/2017 11:42 AM, Ben Pfaff wrote: > On Mon, Apr 17, 2017 at 11:32:13AM -0500, Matthew Thode wrote: >> On 04/17/2017 11:20 AM, Ben Pfaff wrote: >>> On Mon, Apr 17, 2017 at 10:36:26AM -0500, Matthew Thode via dev wrote: Fixes the following warning. WARNING: 'default' html theme

Re: [ovs-dev] [PATCH] doc: fix doc build when not using ovs theme

2017-04-17 Thread Ben Pfaff
On Mon, Apr 17, 2017 at 11:47:30AM -0500, Matthew Thode wrote: > On 04/17/2017 11:42 AM, Ben Pfaff wrote: > > On Mon, Apr 17, 2017 at 11:32:13AM -0500, Matthew Thode wrote: > >> On 04/17/2017 11:20 AM, Ben Pfaff wrote: > >>> On Mon, Apr 17, 2017 at 10:36:26AM -0500, Matthew Thode via dev wrote: > >

[ovs-dev] MAC in MAC support in openvswitch

2017-04-17 Thread Bochra Boughzala
Hi, I haven't use openvswitch for a while and I was wondering if there is a support to add rules for pushing an extra MAC header and create MAC in MAC encapsulation as defined for example in PBB (IEEE 802.1ah). If not is there a more generic way for inserting a new header to a packet by specif

[ovs-dev] [PATCH v2 0/2] installation: remove the runtime directories

2017-04-17 Thread Aaron Conole
This series refactors the documentation and then changes the 'make install' behavior so that it does not automatically create the runtime directories, because they will have ownership of root:root, instead of a user-defined option that may be passed to ovs-ctl. RFC discussion here: https://mail.op

[ovs-dev] [PATCH v2 1/2] install-doc: suggest to use ovs-ctl for start/stop

2017-04-17 Thread Aaron Conole
The install documentation guided users to manually start/stop daemons. This is good information to have, but with the existence of ovs-ctl, is probably not the best way to start guiding new users of ovs. Suggest that users start by running ovs-ctl start, and document the ability to selectively st

[ovs-dev] [PATCH v2 2/2] lib/automake.mk: don't install runtime directories

2017-04-17 Thread Aaron Conole
The Open vSwitch run, log, and DB directories are installed as part of the normal `make install` process. However, this means they are created with user and group ownership that may conflict with the desired user. For example, running `make install` as root will install those files as root:root,

Re: [ovs-dev] [PATCH 0/6] Sphinx for Manpages

2017-04-17 Thread Ben Pfaff
On Thu, Apr 13, 2017 at 09:43:13PM -0700, Ben Pfaff wrote: > This is Stephen Finucane's RFC series from > https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/330666.html > with my changes plus two extra patches at the beginning. Please see my > comments on the former postings for more on the

Re: [ovs-dev] [PATCH] ofp-parse: Fix match parsing with [x..y]=z format.

2017-04-17 Thread Jarno Rajahalme
> On Apr 14, 2017, at 8:58 PM, Ben Pfaff wrote: > > On Thu, Apr 13, 2017 at 06:31:06PM -0700, Jarno Rajahalme wrote: >> Commit 21b2fa617126 ("ofp-parse: Allow match field names in actions >> and brackets in matches.") added support for matching a consecutive >> set of bits with the [x..y]=z form

Re: [ovs-dev] MAC in MAC support in openvswitch

2017-04-17 Thread Eric Garver
On Mon, Apr 17, 2017 at 05:53:44PM +, Bochra Boughzala wrote: > Hi, > > I haven't use openvswitch for a while and I was wondering if there is a > support to add rules for pushing an extra MAC header and create MAC in MAC > encapsulation as defined for example in PBB (IEEE 802.1ah). > > If n

Re: [ovs-dev] MAC in MAC support in openvswitch

2017-04-17 Thread Joe Stringer
On 17 April 2017 at 10:53, Bochra Boughzala wrote: > Hi, > > I haven't use openvswitch for a while and I was wondering if there is a > support to add rules for pushing an extra MAC header and create MAC in MAC > encapsulation as defined for example in PBB (IEEE 802.1ah). No-one has contributed

Re: [ovs-dev] [PATCH 1/2] acinclude: Allow compile with Linux 4.11.

2017-04-17 Thread Jarno Rajahalme
> On Apr 14, 2017, at 8:56 PM, Ben Pfaff wrote: > > On Thu, Apr 13, 2017 at 04:47:35PM -0700, Jarno Rajahalme wrote: >> Change the Linux kernel tests in OVS configuration. >> >> While the backports may still be a little behind, it is useful to be >> able to test the OVS tree kernel module with

Re: [ovs-dev] [PATCH 2/2] dpif: Log packet metadata on execute.

2017-04-17 Thread Jarno Rajahalme
> On Apr 14, 2017, at 8:54 PM, Ben Pfaff wrote: > > On Thu, Apr 13, 2017 at 04:47:36PM -0700, Jarno Rajahalme wrote: >> Debug log output for execute operations is missing the packet >> metadata, which can be instrumental in tracing what the datapath >> should be executing. No reason to have the

Re: [ovs-dev] [branch-2.7 0/4] Backport of variable length metaflow field fixes.

2017-04-17 Thread Joe Stringer
On 14 April 2017 at 20:48, Ben Pfaff wrote: > On Wed, Mar 15, 2017 at 04:01:37PM -0700, Joe Stringer wrote: >> Commit 04f48a68c428 ("ofp-actions: Fix variable length meta-flow OXMs."), on >> branch-2.7 as 9554b03d6ab7, attempted to address incorrect encode and decode >> of >> variable length meta

Re: [ovs-dev] [branch-2.7 1/4] nx-match: Fix oxm decode.

2017-04-17 Thread Jarno Rajahalme
This patch should be prepended by a prior patch, as some required changes are now missing: 7befb20d0f70 (“ofp-util: Ignore unknown fields in nx_decode_packet_in2().”) Jarno > On Mar 15, 2017, at 4:01 PM, Joe Stringer wrote: > > From: Yi-Hung Wei > > decode_nx_packet_in2() may be used by t

[ovs-dev] [PATCH branch-2.6] xenserver: Add install files to RPM file list

2017-04-17 Thread Greg Rose
Add missing install files to RPM file list to prevent file check error so that RPM files will be built. Signed-off-by: Greg Rose --- xenserver/openvswitch-xen.spec.in | 2 ++ 1 file changed, 2 insertions(+) diff --git a/xenserver/openvswitch-xen.spec.in b/xenserver/openvswitch-xen.spec.in inde

Re: [ovs-dev] [PATCH] ofproto/bond: Make bond_may_recirc() private within bond.c

2017-04-17 Thread Andy Zhou
On Fri, Apr 14, 2017 at 2:32 PM, Ben Pfaff wrote: > On Thu, Mar 09, 2017 at 04:52:27PM -0800, Andy Zhou wrote: >> Minor refactoring to make the bond code easier to read. >> >> Signed-off-by: Andy Zhou > > Acked-by: Ben Pfaff Thanks for the review. Pushed to master. _

Re: [ovs-dev] [PATCH] xenserver: Add install files to RPM file list

2017-04-17 Thread Greg Rose
On Mon, 2017-04-17 at 09:38 -0700, Ben Pfaff wrote: > On Mon, Apr 17, 2017 at 09:18:53AM -0700, Greg Rose wrote: > > Add missing intall files to RPM file list to prevent file check error so > > that RPM files will be built. > > > > Signed-off-by: Greg Rose > > Thanks! I applied this to master.

[ovs-dev] [PATCH 1/2] connmgr: Fix internal packet-in reason code mask

2017-04-17 Thread Yi-Hung Wei
Starting from OpenFlow 1.4+, OFPR_ACTION is split into four more descriptive reasons, OFPR_APPLY_ACTION, OFPR_ACTION_SET, OFPR_GROUP, and OFPR_PACKET_OUT. OVS maintains the new reason code internally, and it currently supports the first three reason code. If the version of an established OpenFlow c

[ovs-dev] [PATCH 2/2] ofproto: Add support of OFPR_PACKET_OUT as packet-in reason

2017-04-17 Thread Yi-Hung Wei
This patch adds support of OFPR_PACKET_OUT as the packet-in reason. This packet-in reason is a required feature for OF1.4+, and it indicates that the associated packet-in message to the controller is triggered when the switch is processing a packet-out message. This reason code is enabled by defaul

Re: [ovs-dev] [PATCH] ofproto: Add support of OFPR_PACKET_OUT as packet-in reason

2017-04-17 Thread Yi-Hung Wei
On Fri, Apr 14, 2017 at 2:17 PM, Ben Pfaff wrote: > On Fri, Apr 14, 2017 at 10:20:10AM -0700, Yi-Hung Wei wrote: >> This patch adds support of OFPR_PACKET_OUT as the packet-in reason. >> This packet-in reason is a required feature for OF1.4+, and it indicates >> that the associated packet-in messa

Re: [ovs-dev] [branch-2.7 2/4] nx-match: Use vl_mff_map to parse match field.

2017-04-17 Thread Jarno Rajahalme
Looks good to me: Acked-by: Jarno Rajahalme > On Mar 15, 2017, at 4:01 PM, Joe Stringer wrote: > > From: Yi-Hung Wei > > vl_mff_map is introduced in commit 04f48a68c428 ("ofp-actions: Fix variable > length meta-flow OXMs") to account variable length mf_field, and it is used > to decode varia

Re: [ovs-dev] [branch-2.7 3/4] ofproto: Add ref counting for variable length mf_fields.

2017-04-17 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme with two notes: 1. Maybe patch 4 should be applied before this one to avoid creating a potential memory leak in the history? 2. taking new references before releasing old ones in modify_flows_start__() would seem better. Since the table holds a reference this does n

Re: [ovs-dev] [branch-2.7 4/4] ofproto: Move tun_table and vl_mff_map deletion.

2017-04-17 Thread Jarno Rajahalme
Maybe apply this before 3/4? Acked-by: Jarno Rajahalme Jarno > On Mar 15, 2017, at 4:01 PM, Joe Stringer wrote: > > From: Yi-Hung Wei > > In this patch, we move the tun_table and vl_mff_map deletion in > ofproto_destory__() to be in the following order. >1. Delete all the flows. >

Re: [ovs-dev] [PATCH 1/3] ofproto-dpif: Check support for CT action force commit flag.

2017-04-17 Thread Ben Pfaff
On Fri, Apr 14, 2017 at 05:25:47PM -0700, Jarno Rajahalme wrote: > So far the force commit feature is implemented together with the > original direction tuple feature by all datapaths, so we can use the > support flag for the 'ct_orig_tuple' to indicate support for the force > commit feature as wel

Re: [ovs-dev] [PATCH 2/3] ofproto-dpif: Check if original direction matches are supported.

2017-04-17 Thread Ben Pfaff
On Fri, Apr 14, 2017 at 05:25:48PM -0700, Jarno Rajahalme wrote: > Use the existing probed support flag for the original direction tuple > to determine if matches on the original direction tuple can be supported. > > Signed-off-by: Jarno Rajahalme > --- > ofproto/ofproto-dpif.c | 14

Re: [ovs-dev] [PATCH 2/3] ofproto-dpif: Check if original direction matches are supported.

2017-04-17 Thread Jarno Rajahalme
> On Apr 17, 2017, at 4:47 PM, Ben Pfaff wrote: > > On Fri, Apr 14, 2017 at 05:25:48PM -0700, Jarno Rajahalme wrote: >> Use the existing probed support flag for the original direction tuple >> to determine if matches on the original direction tuple can be supported. >> >> Signed-off-by: Jarno R