Re: [ovs-dev] [PATCH] fedora: Add python3-openvswitch split package

2017-02-23 Thread Russell Bryant
> > > --- > > Looks good to me. > Acked-by: Flavio Leitner <f...@sysclose.org> Thanks to both of you. I applied this to master and branch-2.7. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] ovn-controller: Assign ct_zone id to local datapaths instead of lports

2017-02-23 Thread Russell Bryant
| 17 +++-- > ovn/controller/physical.c | 6 -- > 3 files changed, 15 insertions(+), 16 deletions(-) > Have you looked at the optimization yet? I'm not sure if it makes sense to merge this patch unless the optimization comes with it. -- Russell Bryant ___

Re: [ovs-dev] [PATCH 1/1] python: Fix logic resulting in too many unexpected replies Fix logic resulting in too many unexpected replies

2017-02-17 Thread Russell Bryant
t line of the commit message and then applied this to master, branch-2.7, and branch-2.6. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] manpages in rst?

2017-02-16 Thread Russell Bryant
n is available at build time. > > Does anyone have comments? > +1 for rst. That should make it easier to integrate the same content into docs.openvswitch.org. I think sphinx has support for man pages, but it has been a long time since I've used it. -- Russell Bryant

Re: [ovs-dev] Quick question on Linux expo 2017

2017-02-15 Thread Russell Bryant
On Thu, Feb 16, 2017 at 12:53 AM, Valentine Sinitsyn < valentine.sinit...@gmail.com> wrote: > Hi all, > > I feel strange about replying to seemingly spam emails, but is it the same > Southern California Linux Expo as in [1]? > Yes, I believe this was spam.

Re: [ovs-dev] [PATCH] ovn-northd ipam: handle the static MAC updates by the user

2017-02-15 Thread Russell Bryant
e. > > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1408121 > Signed-off-by: Numan Siddique <nusid...@redhat.com> > Thanks for the patch. I applied this to master and branch-2.7. -- Russell Bryant ___ dev mailing list d...@

Re: [ovs-dev] [PATCH] rhel: make openvswitch service start return when ready

2017-02-13 Thread Russell Bryant
unit.html#Requires= > > Signed-off-by: Alan Pevec <alan.pe...@redhat.com> > Reviewed-by: Aaron Conole <acon...@redhat.com> > --- > rhel/usr_lib_systemd_system_openvswitch.service | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Thanks for the patch! I

Re: [ovs-dev] [PATCH] packaging: Add hostname as dependency

2017-02-13 Thread Russell Bryant
| 2 +- > rhel/openvswitch-fedora.spec.in | 2 +- > rhel/openvswitch.spec.in| 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > ACK for the rhel side, but I'd prefer someone else to ack the debian side. Acked-by: Russell Bry

Re: [ovs-dev] [PATCH v2] rhel: Firewall service files for OVN.

2017-02-03 Thread Russell Bryant
On Fri, Feb 3, 2017 at 3:50 PM, Russell Bryant <russ...@ovn.org> wrote: > From: Marcin Mirecki <mmire...@redhat.com> > > Firewall service files allowing to open firewalld > ports required for running OVN > > Signed-off-by: Marcin Mirecki <mmire...@redhat.com>

[ovs-dev] [PATCH v2] rhel: Firewall service files for OVN.

2017-02-03 Thread Russell Bryant
From: Marcin Mirecki <mmire...@redhat.com> Firewall service files allowing to open firewalld ports required for running OVN Signed-off-by: Marcin Mirecki <mmire...@redhat.com> Acked-by: Ben Pfaff <b...@ovn.org> Signed-off-by: Russell Bryant <russ...@ovn.

Re: [ovs-dev] [PATCH] ovn: Add missing netdev_close in setup_qos.

2017-02-03 Thread Russell Bryant
On Thu, Feb 2, 2017 at 12:41 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Feb 02, 2017 at 02:04:07AM -0500, Russell Bryant wrote: > > We missed calling netdev_close in a couple of places. One was in an > error > > condition rarely hit. The second was just introduced and

Re: [ovs-dev] [PATCH] docs: Add OVS and OVN headings to pages.

2017-02-03 Thread Russell Bryant
On Thu, Feb 2, 2017 at 12:40 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Feb 02, 2017 at 02:02:34AM -0500, Russell Bryant wrote: > > Update the "deep dive" and "howto" pages with headings that more clearly > > indicate the separate lists of OVS or

Re: [ovs-dev] [PATCH v2] ovn-controller: Provide the option to set Encap.options:csum

2017-02-02 Thread Russell Bryant
support for tx-udp_tnl-csum-segmentation, which is present in the upstream kernel but not the RHEL kernel yet. So, right now to get the best overall performance, we have to disable checksums (for current CentOS / RHEL, anyway). -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] ovn-trace: Style improvements.

2017-02-01 Thread Russell Bryant
On Fri, Jan 13, 2017 at 2:30 PM, Ben Pfaff <b...@ovn.org> wrote: > Signed-off-by: Ben Pfaff <b...@ovn.org> > --- > ovn/utilities/ovn-trace.c | 12 +--- > 1 file changed, 9 insertions(+), 3 deletions(-) > Acked-by: Russell Bryant <russ...

[ovs-dev] [PATCH] ovn: Add missing netdev_close in setup_qos.

2017-02-01 Thread Russell Bryant
Signed-off-by: Russell Bryant <russ...@ovn.org> --- ovn/controller/binding.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ovn/controller/binding.c b/ovn/controller/binding.c index 2d2da16..c90cb65 100644 --- a/ovn/controller/binding.c +++ b/ovn/controller/binding.c @@ -263,6 +263,7 @@

[ovs-dev] [PATCH] docs: Add OVS and OVN headings to pages.

2017-02-01 Thread Russell Bryant
Update the "deep dive" and "howto" pages with headings that more clearly indicate the separate lists of OVS or OVN content. Also add a link to ovn-architecture from the "deep dive" page as it seems quite relevant there. Signed-off-by: Russell Bryant <russ...@

Re: [ovs-dev] [PATCH 2/2] ovn-controller: Configure interface QoS only if it would actually be used.

2017-02-01 Thread Russell Bryant
Ah, I noticed after reviewing that the patches were already merged. Sorry. I'll submit my suggestions as a follow-up patch. On Thu, Feb 2, 2017 at 1:33 AM, Russell Bryant <russ...@ovn.org> wrote: > > > On Wed, Feb 1, 2017 at 12:22 PM, Ben Pfaff <b...@ovn.org> wrote

Re: [ovs-dev] [PATCH 1/1] OVN: Firewall service files for OVN

2017-01-31 Thread Russell Bryant
because it was also sent directly to me. Marcin, can you try submitting this using the "git-send-email" utility? Thanks, -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] ovn-northd: Add flows in DHCP_OPTIONS pipeline to support renew requests

2017-01-24 Thread Russell Bryant
s pipeline. > > Signed-off-by: Numan Siddique <nusid...@redhat.com> Thanks for working on this, Numan! I see some DHCP test cases in tests/ovn.at. Could you look at extending those to cover this case as well? Thanks, -- Russell Bryant ___

Re: [ovs-dev] [PATCH v2] ovn: Document upgrade procedure.

2017-01-23 Thread Russell Bryant
On Mon, Jan 23, 2017 at 11:36 AM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Jan 23, 2017 at 09:50:02AM -0500, Russell Bryant wrote: > > Signed-off-by: Russell Bryant <russ...@ovn.org> > > Thanks Russell! > > s/its/it's/ here: > > +You should start by up

Re: [ovs-dev] [PATCH] doc: Remove tutorials/ovn-basics.

2017-01-23 Thread Russell Bryant
On Mon, Jan 23, 2017 at 11:25 AM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Jan 23, 2017 at 09:02:07AM -0500, Russell Bryant wrote: > > On Thu, Jan 19, 2017 at 6:01 PM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On Thu, Jan 19, 2017 at 02:17:09PM -0500, Russell

Re: [ovs-dev] [PATCH 1/2] ovn: Document upgrade procedure.

2017-01-23 Thread Russell Bryant
Oops, I forgot to follow up on this ... On Thu, Dec 22, 2016 at 12:21 AM, Ben Pfaff <b...@ovn.org> wrote: > On Wed, Dec 21, 2016 at 10:56:35PM -0500, Russell Bryant wrote: > > On Wed, Dec 21, 2016 at 5:06 PM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On W

[ovs-dev] [PATCH v2] ovn: Document upgrade procedure.

2017-01-23 Thread Russell Bryant
Signed-off-by: Russell Bryant <russ...@ovn.org> --- Documentation/automake.mk| 1 + Documentation/intro/install/index.rst| 8 +++ Documentation/intro/install/ovn-upgrades.rst | 87 3 files changed, 96 insertions(+) create mode

Re: [ovs-dev] [PATCH] doc: Remove tutorials/ovn-basics.

2017-01-23 Thread Russell Bryant
On Thu, Jan 19, 2017 at 6:01 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Jan 19, 2017 at 02:17:09PM -0500, Russell Bryant wrote: > > The only thing worse than a lack of documentation is incorrect or > > out-of-date documentation. Over time, this document has not kept

[ovs-dev] [PATCH] doc: Remove tutorials/ovn-basics.

2017-01-19 Thread Russell Bryant
. An even more important type of tutorial would be something along the lines of: http://blog.spinhirne.com/p/blog-series.html That blog series was fantastic and has been the primary tutorial reference I have been sending people to since it was written. Signed-off-by: Russell Bryant <russ...@ovn.

Re: [ovs-dev] [PATCH] vlan.rst: Strip leftover HTML.

2017-01-17 Thread Russell Bryant
On Mon, Jan 16, 2017 at 5:14 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Jan 16, 2017 at 04:46:58PM -0500, Russell Bryant wrote: > > String a couple of closing HTML tags that were left over from when this > doc > > s/String/Strip/ > Oops, fixed. > > > wa

[ovs-dev] [PATCH] vlan.rst: Strip leftover HTML.

2017-01-16 Thread Russell Bryant
String a couple of closing HTML tags that were left over from when this doc was converted from the web site to RST. Signed-off-by: Russell Bryant <russ...@ovn.org> --- Documentation/howto/vlan.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/howto/vlan

Re: [ovs-dev] [PATCH] ovn-controller: Provide the option to set Encap.options:csum

2017-01-16 Thread Russell Bryant
On Mon, Jan 16, 2017 at 12:01 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Jan 16, 2017 at 09:28:28AM -0500, Russell Bryant wrote: > > On Sat, Jan 14, 2017 at 11:29 AM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On Sat, Jan 14, 2017 at 07:37:53PM +0530, Numan

Re: [ovs-dev] [PATCH] python: Add TCP/SSL probes for OVSDB python lib

2017-01-05 Thread Russell Bryant
; is aborted > > > > Signed-off-by: Guoshuai Li <l...@dtdream.com> > > Applied to master, thanks! Are you OK with backporting this to branch-2.6? I don't mind doing it if so. -- Russell Bryant ___ dev mailing list d...@openvswitc

Re: [ovs-dev] [PATCH 1/2] ovn: Document upgrade procedure.

2016-12-21 Thread Russell Bryant
On Wed, Dec 21, 2016 at 5:06 PM, Ben Pfaff <b...@ovn.org> wrote: > On Wed, Dec 07, 2016 at 02:28:13PM -0500, Russell Bryant wrote: > > Signed-off-by: Russell Bryant <russ...@ovn.org> > > This procedure isn't what I expected. It recommends: > > 1. U

Re: [ovs-dev] [RFC: fedora ovn packaging 0/5] RFC fedora ovn packaging

2016-12-16 Thread Russell Bryant
-southbound-db.service > > -- > 1.8.3.1 > > _______ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] docs: Add link to man pages.

2016-12-16 Thread Russell Bryant
On Wed, Dec 14, 2016 at 5:01 AM, Stephen Finucane <step...@that.guru> wrote: > On Tue, 2016-12-13 at 15:09 -0500, Russell Bryant wrote: > > > > > > On Tue, Dec 13, 2016 at 1:30 PM, Stephen Finucane <step...@that.guru> > > wrote: > > > On Tue, 2

Re: [ovs-dev] Proposal to move the Python lib to its own repo

2016-12-16 Thread Russell Bryant
months for the next ovs release. The new release schedule means more like 5-6 months, max, but perhaps a similar situation could arise. My suggestion is to wait until something like that comes up again. The rest isn't a strong enough justification to me. -- Russell Bryant On Mon, Dec 12, 2016

Re: [ovs-dev] [RFC] ovn: Securing from compromised host - removal of MAC_Binding

2016-12-09 Thread Russell Bryant
to track many additional MAC addresses per router. Right now we allow the MAC address for router interfaces to be configured through the northbound interface. I suppose ovn-northd could generate random MAC addresses for distributed routers, instead. Are there are other key thin

Re: [ovs-dev] [PATCH] rpms: Remove more OVN files form openvswitch rpm builds

2016-12-09 Thread Russell Bryant
On Thu, Dec 8, 2016 at 8:31 PM, Andy Zhou <az...@ovn.org> wrote: > OVN is packaged with openvswitch-fedora.spec.in, but not with > openvswitch.spec.in. Remove OVN files from openvswitch.spec.in > builds to make rpm build happy. > > Signed-off-by: Andy Zhou <az...@ovn.o

Re: [ovs-dev] [PATCH 1/2] ovn-sbctl: Fix a few minor typesetting errors.

2016-12-09 Thread Russell Bryant
rally be used for hyphens > in English text, as it was used here in italic (metasyntactic variable) > text. This fixes the problem. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > Acked-by: Russell Bryant <russ...@ovn.org> ___

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Russell Bryant
Numan, > > This would imply that unauthenticated/insecure remote access to the > NB database is always available. > > Users wanting to use SSL for the NB database connection are likely > to be concerned about security and authentication for this connection, > having a defau

[ovs-dev] [PATCH 1/2] ovn: Document upgrade procedure.

2016-12-07 Thread Russell Bryant
Signed-off-by: Russell Bryant <russ...@ovn.org> --- ovn/automake.mk | 3 +- ovn/docs/upgrades.rst | 89 +++ 2 files changed, 91 insertions(+), 1 deletion(-) create mode 100644 ovn/docs/upgrades.rst diff --git a/ovn/automake.mk

Re: [ovs-dev] [patch_v10] ovn: Add datapaths of interest filtering.

2016-12-06 Thread Russell Bryant
..@gmail.com> > --- > When I apply this patch to master, one of the tests is failing for me. This test passes if I drop the patch. 2296: ovn-controller - ovn-bridge-mappingsFAILED ( ovn-controller.at:45) -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [patch_v10] ovn: Add datapaths of interest filtering.

2016-12-06 Thread Russell Bryant
On Tue, Dec 6, 2016 at 3:36 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Dec 06, 2016 at 12:35:16PM -0800, Ben Pfaff wrote: > > On Tue, Dec 06, 2016 at 02:45:19PM -0500, Russell Bryant wrote: > > > On Mon, Dec 5, 2016 at 2:22 AM, Ben Pfaff <b...@ovn.org> wrote: >

Re: [ovs-dev] [patch_v10] ovn: Add datapaths of interest filtering.

2016-12-06 Thread Russell Bryant
ntrol plane performance testing using OpenStack and OVN. I may have time to do another run with some additional patches applied to see how they affect performance. Ideally I'd be able to do that this week, though. If I'm able to fit this in, what do you g

Re: [ovs-dev] [PATCH 0/3] ovn: support ssl connections to nb/sb dbs

2016-12-06 Thread Russell Bryant
vn-nb: remote connection management in nb db > ovn-sb: remote connection management in sb db > ovn-ctl: add support for SSL nb/sb db connections > Tested-by: Russell Bryant <russ...@ovn.org> I haven't actually tested the SSL portion, but I am using these patches in an OVN test env

Re: [ovs-dev] [PATCH 2/2] tests: Add tracing to "ovn -- 2 HVs, 4 lports/HV, localnet ports" test.

2016-12-05 Thread Russell Bryant
> 1 file changed, 60 insertions(+), 36 deletions(-) > Acked-by: Russell Bryant <russ...@ovn.org> -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 1/2] tests: Fix order confusion in "ovn -- 2 HVs, 4 lports/HV, localnet ports".

2016-12-05 Thread Russell Bryant
t}${src}${eth} > hv=`vif_to_hv $inport` > vif=vif$inport > as $hv ovs-appctl netdev-dummy/receive $vif $packet > -- > 2.1.3 > > ___ > dev mailing list > d...@openvswitch.org > http://openvswitch.o

Re: [ovs-dev] [PATCH] ovn-trace: Add output format options to usage message.

2016-12-05 Thread Russell Bryant
@ovn.org> > > This still needs a review, which should be trivial. Acked-by: Russell Bryant <russ...@ovn.org> ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH v2] ovn-controller(8): ovn-remote must be an ovsdb remote.

2016-12-05 Thread Russell Bryant
Document that the value of the ovn-remote configuration option must be in the form of an ovsdb remote as previously documented in this man page. This came up on IRC where someone trying OVN put a hostname here and observed that it did not work. Signed-off-by: Russell Bryant <russ...@ovn.

Re: [ovs-dev] [PATCH] ovn-controller(8): ovn-remote must be an IP adddress.

2016-12-05 Thread Russell Bryant
On Mon, Dec 5, 2016 at 1:13 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Dec 05, 2016 at 10:10:43AM -0800, Ben Pfaff wrote: > > On Mon, Dec 05, 2016 at 12:05:33PM -0500, Russell Bryant wrote: > > > Document that the value of the ovn-remote configuration > >

[ovs-dev] [PATCH] ovn-controller(8): ovn-remote must be an IP adddress.

2016-12-05 Thread Russell Bryant
Document that the value of the ovn-remote configuration option must be an IP address. This came up on IRC where someone trying OVN put a hostname here and observed that it did not work. Signed-off-by: Russell Bryant <russ...@ovn.org> --- ovn/controller/ovn-controller.8.xml | 2 +-

Re: [ovs-dev] [PATCH] ovsdb-tool: Document database numbering scheme.

2016-12-02 Thread Russell Bryant
ed, 27 insertions(+), 9 deletions(-) > Thanks, Acked-by: Russell Bryant <russ...@ovn.org> ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] physical: Remove obsolete comments.

2016-12-01 Thread Russell Bryant
On Thu, Dec 1, 2016 at 2:11 AM, Ben Pfaff <b...@ovn.org> wrote: > These comments were added above code that removed flows from the flow > table. Now that that code was deleted, the comments no longer make sense. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > Ack

Re: [ovs-dev] [PATCH] dist-docs: typo in IntegrationGuide.rst

2016-11-28 Thread Russell Bryant
urce_Agents`_ > +`OCF <http://www.linux-ha.org/wiki/OCF_Resource_Agents>`_ > > OCF is nothing but a shell script which accepts a set of actions and > returns an > appropriate status code. Thanks, applied to master. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v2 2/2] rhel: Add EnvironmentFile for ovn services.

2016-11-28 Thread Russell Bryant
On Fri, Nov 25, 2016 at 9:16 AM, Simon Horman <simon.hor...@netronome.com> wrote: > On Wed, Nov 23, 2016 at 03:01:24PM -0500, Russell Bryant wrote: > > A previous commit documented how you do this using systemd > > native interfaces for customizing services. However, i

Re: [ovs-dev] [PATCH v2 1/2] rhel: Fix Environment for ovn-controller-vtep.

2016-11-28 Thread Russell Bryant
On Fri, Nov 25, 2016 at 9:14 AM, Simon Horman <simon.hor...@netronome.com> wrote: > Hi Russell, > > On Wed, Nov 23, 2016 at 03:01:23PM -0500, Russell Bryant wrote: > > The systemd unit for ovn-controller-vtep incorrectly specified > > Environment multiple times. Mul

[ovs-dev] [PATCH v2 2/2] rhel: Add EnvironmentFile for ovn services.

2016-11-23 Thread Russell Bryant
it. The "-" prefix to the filename means that systemd will ignore this setting if the file does not exist and will not report any error. Signed-off-by: Russell Bryant <russ...@ovn.org> --- rhel/usr_lib_systemd_system_ovn-controller-vtep.service | 10 -- rhel/usr_lib_sy

[ovs-dev] [PATCH v2 1/2] rhel: Fix Environment for ovn-controller-vtep.

2016-11-23 Thread Russell Bryant
The systemd unit for ovn-controller-vtep incorrectly specified Environment multiple times. Multiple environment variables must be specified separated by a space to a single Environment option. Signed-off-by: Russell Bryant <russ...@ovn.org> --- rhel/usr_lib_systemd_system_ovn-cont

Re: [ovs-dev] [PATCH] utilities: Use FQDN for external_ids:hostname in Openvswitch table

2016-11-23 Thread Russell Bryant
et. We can hope for that! If anyone complains, we could just add a new external-id called "fqdn" or something, but that would also require changes to ovn-controller to expose that via the chassis record and some changes to the Neutron OVN driver to make use of the new field. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH 2/2] rhel: Add EnvironmentFile for ovn services.

2016-11-21 Thread Russell Bryant
it. The "-" prefix to the filename means that systemd will ignore this setting if the file does not exist and will not report any error. Signed-off-by: Russell Bryant <russ...@ovn.org> --- rhel/usr_lib_systemd_system_ovn-controller-vtep.service | 10 -- rhel/usr_lib_sy

[ovs-dev] [PATCH 1/2] rhel: Fix Environment for ovn-controller-vtep.

2016-11-21 Thread Russell Bryant
The systemd unit for ovn-controller-vtep incorrectly specified Environment multiple times. Multiple environment variables must be specified separated by a space to a single Environment option. Signed-off-by: Russell Bryant <russ...@ovn.org> --- rhel/usr_lib_systemd_system_ovn-cont

Re: [ovs-dev] [RFC] ovn: Securing from compromised host - removal of MAC_Binding

2016-11-18 Thread Russell Bryant
> > > What do you all think of this approach? > I'll add that while adding a broadcast doesn't sound great, it's not drastically different from the current code. Today we're effectively broadcasting the MAC binding through OVSDB instead of broadcasting the ARP reply. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 2/2] rhel: Support ovn-ctl args through env vars.

2016-11-17 Thread Russell Bryant
On Thu, Nov 17, 2016 at 10:46 AM, Simon Horman <simon.hor...@netronome.com> wrote: > On Thu, Nov 17, 2016 at 08:39:28AM -0500, Russell Bryant wrote: > > Update the systemd units for ovn-controller and ovn-northd to support > > passing additional arguments to the ov

[ovs-dev] [PATCH] .mailmap: Document file format.

2016-11-17 Thread Russell Bryant
-by: Russell Bryant <russ...@ovn.org> --- .mailmap | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/.mailmap b/.mailmap index a5eea0c..f01664e 100644 --- a/.mailmap +++ b/.mailmap @@ -1,13 +1,24 @@ +# See git-shortlog(1) for official documentation of this file

Re: [ovs-dev] [PATCH] rhel: Environment file option for northd service

2016-11-17 Thread Russell Bryant
On Thu, Nov 17, 2016 at 6:42 AM, Babu Shanmugam <bscha...@redhat.com> wrote: > > > On Wednesday 16 November 2016 10:07 PM, Russell Bryant wrote: > > > > On Wed, Nov 16, 2016 at 4:02 AM, Babu Shanmugam <bscha...@redhat.com> > wrote: > >> >> >&g

[ovs-dev] [PATCH 1/2] ovn-ctl: Fix --ovn-controller-log.

2016-11-17 Thread Russell Bryant
This option was documented, but didn't actually work because it wasn't used when starting ovn-controller. Signed-off-by: Russell Bryant <russ...@ovn.org> --- ovn/utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/o

Re: [ovs-dev] [PATCH] rhel: Environment file option for northd service

2016-11-16 Thread Russell Bryant
On Wed, Nov 16, 2016 at 4:02 AM, Babu Shanmugam <bscha...@redhat.com> wrote: > > > On Wednesday 16 November 2016 11:45 AM, Babu Shanmugam wrote: > > > > On Tuesday 15 November 2016 08:11 PM, Russell Bryant wrote: > > > > On Tue, Nov 15, 2016 at 6:19 AM,

Re: [ovs-dev] [PATCH] rhel: Environment file option for northd service

2016-11-15 Thread Russell Bryant
/fedoraproject.org/wiki/Systemd#How_do_I_customize_a_unit_file.2F_add_a_custom_unit_file.3F or: "Example 2. Overriding vendor settings" https://www.freedesktop.org/software/systemd/man/systemd.unit.html For ovn-northd, you would create a directory and config file, /etc/systemd/system/ovn-nor

Re: [ovs-dev] [PATCH v3 1/2] logical-fields: Add ct.dnat and ct.snat symbols.

2016-11-14 Thread Russell Bryant
19 +-- > tests/ovn.at | 4 +++- > 3 files changed, 22 insertions(+), 3 deletions(-) > Acked-by: Russell Bryant <russ...@ovn.org> -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] doc: Remove final markdown references

2016-11-07 Thread Russell Bryant
On Mon, Nov 7, 2016 at 2:03 PM, Russell Bryant <russ...@ovn.org> wrote: > > > On Mon, Nov 7, 2016 at 3:22 AM, Stephen Finucane <step...@that.guru> > wrote: > >> There are some leftover Markdown references and Markdown-related >> tooling. Seeing as there

Re: [ovs-dev] [PATCH] doc: Remove final markdown references

2016-11-07 Thread Russell Bryant
On Mon, Nov 7, 2016 at 3:22 AM, Stephen Finucane wrote: > There are some leftover Markdown references and Markdown-related > tooling. Seeing as there are no Markdown files left, we can kill it all > with fire. > > Signed-off-by: Stephen Finucane > --- >

Re: [ovs-dev] [PATCH 0/4] Convert remaining docs to rST

2016-11-04 Thread Russell Bryant
On Fri, Nov 4, 2016 at 11:03 AM, Stephen Finucane wrote: > This is the fourth and final conversion series. This was kept separate > due to the size and complexity of the tutorial guides. There are also > ancillary changes to a previously unformatted TODO file (missed >

Re: [ovs-dev] [PATCH 23/23] doc: Convert vswitchd/INTERNALS to rST

2016-11-04 Thread Russell Bryant
On Fri, Nov 4, 2016 at 10:40 AM, Stephen Finucane <step...@that.guru> wrote: > On 2016-11-04 00:43, Russell Bryant wrote: > >> On Sun, Oct 30, 2016 at 2:30 PM, Stephen Finucane <step...@that.guru> >> wrote: >> >> Signed-off-by: Stephen Finucane <step

Re: [ovs-dev] [PATCH 23/23] doc: Convert vswitchd/INTERNALS to rST

2016-11-03 Thread Russell Bryant
ERNALS.rst > This whole series has been applied to master. Thanks! -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 14/23] doc: Convert INSTALL.SELinux to rST

2016-11-03 Thread Russell Bryant
TALL.SELinux.rst > I have applied all patches from this series up to this one. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 09/23] doc: Convert INSTALL.Fedora to rST

2016-11-03 Thread Russell Bryant
edora Linux host, you will need the packages described -in the `general installation guide <INSTALL.rst`__. Specific packages (by +in the `general installation guide `__. Specific packages (by package name) include: - rpm-build -- Russell Bryant ___ dev mailin

Re: [ovs-dev] [PATCH v2] release: Propose a shorter release cycle for 2.7.

2016-11-03 Thread Russell Bryant
On Thu, Nov 3, 2016 at 10:46 PM, Justin Pettit <jpet...@ovn.org> wrote: > > > On Nov 3, 2016, at 10:06 AM, Russell Bryant <russ...@ovn.org> wrote: > > > > diff --git a/Documentation/release-process.md b/Documentation/release- > process.md > > index 0f8f4

Re: [ovs-dev] [PATCH 06/23] doc: Convert rhel/README.RHEL to rST

2016-11-03 Thread Russell Bryant
tch were changed to rhel/README.RHEL.rst. I renamed the file to README.RHEL.rst. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 06/23] doc: Convert rhel/README.RHEL to rST

2016-11-03 Thread Russell Bryant
On Thu, Nov 3, 2016 at 8:50 PM, Russell Bryant <russ...@ovn.org> wrote: > > On Sun, Oct 30, 2016 at 2:29 PM, Stephen Finucane <step...@that.guru> > wrote: > >> This is a top-level document, so plain old rST is preferred. >> >> Signed-o

Re: [ovs-dev] [PATCH 05/23] doc: Convert OVSDB-replication to rST

2016-11-03 Thread Russell Bryant
of the object members: - If ``old`` member is not present, execute an insert operation using ```` from the ``new`` member. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 02/23] doc: Convert committer-grant-revocation to rST

2016-11-03 Thread Russell Bryant
o be accepted +to be accepted - Consistent participation in code review of other's patches, including existing committers, with comments consistent with the overall project -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 04/23] doc: Convert release-process to rST

2016-11-03 Thread Russell Bryant
.5 | Sep 15, Mar 15| branch-x.y released as version x.y.0 | ++---+---+----------+ Contact --- -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] doc: Convert DESIGN to rST

2016-11-03 Thread Russell Bryant
- Resolve some issues with table headers not displaying > --- > I applied this to master. Thanks! -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-11-03 Thread Russell Bryant
gt; Reported-by: Kevin Lin <kevin...@berkeley.edu> > > Reported-at: http://openvswitch.org/pipermail/dev/2016-October/ > 080386.html > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > This needs a review, which should be easy. Acked-by: Russell Bryant <russ...@ovn.

[ovs-dev] [PATCH v2] release: Propose a shorter release cycle for 2.7.

2016-11-03 Thread Russell Bryant
to indicate February as the target release month instead of March. Signed-off-by: Russell Bryant <russ...@ovn.org> --- Documentation/release-process.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/release-process.md b/Documentation/release-process.md

Re: [ovs-dev] [PATCH v2] doc: fix duplicated colons in rst documents

2016-11-01 Thread Russell Bryant
LL.userspace.rst | 6 +-- > 8 files changed, 165 insertions(+), 165 deletions(-) > Applied to master, thanks! -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] release: Propose a shorter release cycle for 2.7.

2016-11-01 Thread Russell Bryant
On Mon, Oct 31, 2016 at 10:53 PM, Justin Pettit <jpet...@ovn.org> wrote: > > > On Oct 31, 2016, at 12:22 PM, Russell Bryant <russ...@ovn.org> wrote: > > > > > > > > On Mon, Oct 31, 2016 at 5:23 PM, Justin Pettit <jpet...@ovn.org> wrote: > >

Re: [ovs-dev] [PATCH] release: Propose a shorter release cycle for 2.7.

2016-10-31 Thread Russell Bryant
On Mon, Oct 31, 2016 at 5:23 PM, Justin Pettit <jpet...@ovn.org> wrote: > > > On Oct 29, 2016, at 9:19 AM, Russell Bryant <russ...@ovn.org> wrote: > > > > diff --git a/Documentation/release-process.md b/Documentation/release- > process.md > > index 0f8f4

Re: [ovs-dev] [PATCH 15/15] trivial: Resolve whitespace issues with acinclude

2016-10-29 Thread Russell Bryant
es to DESIGN. There were some tables that weren't rendering properly for me. -- Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 14/15] doc: Convert FAQ to rST

2016-10-29 Thread Russell Bryant
`__. Q: What Linux kernel versions does each Open vSwitch release work with? -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 12/15] doc: Convert OPENFLOW-1.1+ to rST

2016-10-29 Thread Russell Bryant
On Tue, Oct 18, 2016 at 10:03 PM, Stephen Finucane <step...@that.guru> wrote: > Rename it to OPENFLOW. > > Signed-off-by: Stephen Finucane <step...@that.guru> > I have applied patches 11 and 12 to master. -- Russell Bryant

Re: [ovs-dev] [PATCH 13/15] doc: Convert DESIGN to rST

2016-10-29 Thread Russell Bryant
ADD MODIFY STRICT DELETE STRICT > + === == == == == > This (and other similar tables) are not rendering properly for me. All of the cells in this first row are empty in the HTML version. -- Russell Bryant _

Re: [ovs-dev] [PATCH 13/15] doc: Convert DESIGN to rST

2016-10-29 Thread Russell Bryant
REQUESTFORWARD`` (OF1.4+)| > ++---+-+---+ > +| ``OFPRFR_GROUP_MOD`` | --- | --- | > ++---+-+---+ > +| ``OFPRFR_METER_MOD``

Re: [ovs-dev] [PATCH 10/15] doc: Convert README-native-tunneling to rST

2016-10-29 Thread Russell Bryant
On Sat, Oct 29, 2016 at 1:50 PM, Stephen Finucane <step...@that.guru> wrote: > On 28 Oct 23:25, Russell Bryant wrote: > > On Tue, Oct 18, 2016 at 10:03 PM, Stephen Finucane <step...@that.guru> > > wrote: > > > > > Signed-off-by: Stephen Finucane <st

[ovs-dev] [PATCH] release: Propose a shorter release cycle for 2.7.

2016-10-29 Thread Russell Bryant
://releases.openstack.org/ocata/schedule.html Signed-off-by: Russell Bryant <russ...@ovn.org> --- Documentation/release-process.md | 26 ++ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/Documentation/release-process.md b/Documentation/release-process.md

Re: [ovs-dev] [PATCH 10/15] doc: Convert README-native-tunneling to rST

2016-10-28 Thread Russell Bryant
etions(-) > delete mode 100644 README-native-tunneling.md > create mode 100644 README-native-tunneling.rst > I've applied patches 9 and 10 to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 08/15] doc: Convert SECURITY to rST

2016-10-26 Thread Russell Bryant
ant. Reporters may ask for a GPG key while initiating contact with the security team to deliver more sensitive reports. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 07/15] doc: Convert WHY-OVS to rST

2016-10-26 Thread Russell Bryant
On Tue, Oct 25, 2016 at 5:55 AM, Russell Bryant <russ...@ovn.org> wrote: > > > On Sun, Oct 23, 2016 at 8:00 AM, Stephen Fincane <step...@that.guru> > wrote: > >> On Fri, 2016-10-21 at 16:09 -0400, Russell Bryant wrote: >> > >> > >>

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

2016-10-25 Thread Russell Bryant
On Tue, Oct 25, 2016 at 10:31 AM, Lance Richardson <lrich...@redhat.com> wrote: > > From: "Russell Bryant" <russ...@ovn.org> > > To: dev@openvswitch.org > > Sent: Tuesday, October 25, 2016 10:12:32 AM > > Subject: [ovs-dev] [PATCH] ovn/TODO:

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

2016-10-25 Thread Russell Bryant
Signed-off-by: Russell Bryant <russ...@ovn.org> --- ovn/TODO | 5 - 1 file changed, 5 deletions(-) diff --git a/ovn/TODO b/ovn/TODO index 7d071d3..b719cfd 100644 --- a/ovn/TODO +++ b/ovn/TODO @@ -37,11 +37,6 @@ Russell Bryant: "Today that would require creating 4096 ports

Re: [ovs-dev] [PATCH 07/15] doc: Convert WHY-OVS to rST

2016-10-25 Thread Russell Bryant
On Sun, Oct 23, 2016 at 8:00 AM, Stephen Fincane <step...@that.guru> wrote: > On Fri, 2016-10-21 at 16:09 -0400, Russell Bryant wrote: > > > > > > On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane <step...@that.guru> > > wrote: > > > Signed-off-by

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

2016-10-21 Thread Russell Bryant
On Thu, Oct 20, 2016 at 5:52 PM, Han Zhou <zhou...@gmail.com> wrote: > > On Thu, Oct 20, 2016 at 11:51 AM, Russell Bryant <russ...@ovn.org> wrote: > > > > On Thu, Oct 20, 2016 at 1:47 PM, Ben Pfaff <b...@ovn.org> wrote: > > > > > On Thu, Oct

Re: [ovs-dev] [PATCH 07/15] doc: Convert WHY-OVS to rST

2016-10-21 Thread Russell Bryant
t; +virtualized environments. If physical switches also expose the Open > vSwitch > +control abstractions, both bare-metal and virtualized hosting > environments can > +be managed using the same mechanism for automated network control. > + > +In many ways, Open vSwitch targets a different point in the design space > than > +previous hypervisor networking stacks, focusing on the need for automated > and > +dynamic network control in large-scale Linux-based virtualization > environments. > + > +The goal with Open vSwitch is to keep the in-kernel code as small as > possible > +(as is necessary for performance) and to re-use existing subsystems when > +applicable (for example Open vSwitch uses the existing QoS stack). As of > Linux > +3.3, Open vSwitch is included as a part of the kernel and packaging for > the > +userspace utilities are available on most popular distributions. > These last two paragraphs were not part of the "hardware integration" section in the original doc. They were the closing paragraphs of the document. I haven't thought of a good heading for them, though. Maybe "design"? Thoughts? -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

<    1   2   3   4   5   6   7   8   9   10   >