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 > previously due to the lack

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 wrote: > On 2016-11-04 00:43, Russell Bryant wrote: > >> On Sun, Oct 30, 2016 at 2:30 PM, Stephen Finucane >> wrote: >> >> Signed-off-by: Stephen Finucane >>> --- >>> lib/mac-learning

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

2016-11-03 Thread Russell Bryant
d/INTERNALS.rst | 244 ++ > +++ > vswitchd/automake.mk | 2 +- > 5 files changed, 248 insertions(+), 242 deletions(-) > delete mode 100644 vswitchd/INTERNALS > create mode 100644 vswitchd/INTERNALS.rst > This whole series has been applied t

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

2016-11-03 Thread Russell Bryant
- > INSTALL.SELinux.rst | 174 ++ > ++ > Makefile.am | 2 +- > README.rst | 2 +- > 4 files changed, 176 insertions(+), 175 deletions(-) > delete mode 100644 INSTALL.SELinux.md > create mode 100644 INSTALL.SELinux.rst > I have appl

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

2016-11-03 Thread Russell Bryant
`general installation guide `__. Specific packages (by package name) include: - rpm-build -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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 wrote: > > > On Nov 3, 2016, at 10:06 AM, Russell Bryant wrote: > > > > diff --git a/Documentation/release-process.md b/Documentation/release- > process.md > > index 0f8f49d..fcbd174 100644 > > --- a/Docume

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

2016-11-03 Thread Russell Bryant
7 files changed, 309 insertions(+), 274 deletions(-) > delete mode 100644 rhel/README.RHEL > create mode 100644 rhel/README.rst > This patch renamed rhel/README.RHEL to rhel/README.rst, but all other references to this file int he patch were changed to rhel/README.RHEL.rst. I renamed

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 wrote: > > On Sun, Oct 30, 2016 at 2:29 PM, Stephen Finucane > wrote: > >> This is a top-level document, so plain old rST is preferred. >> >> Signed-off-by: Stephen Finucane >> --- >> INSTALL.Fedora.md

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

2016-11-03 Thread Russell Bryant
ber 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
t 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
nch-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
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
Kevin Lin > > Reported-at: http://openvswitch.org/pipermail/dev/2016-October/ > 080386.html > > Signed-off-by: Ben Pfaff > > This needs a review, which should be easy. Acked-by: Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-11-03 Thread Russell Bryant
es to indicate February as the target release month instead of March. Signed-off-by: Russell Bryant --- Documentation/release-process.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/release-process.md b/Documentation/release-process.md index 0f8f49d..fc

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

2016-11-01 Thread Russell Bryant
; 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 wrote: > > > On Oct 31, 2016, at 12:22 PM, Russell Bryant wrote: > > > > > > > > On Mon, Oct 31, 2016 at 5:23 PM, Justin Pettit wrote: > > > >> > On Oct 29, 2016, at 9:19 AM, Russell Bryant

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 wrote: > > > On Oct 29, 2016, at 9:19 AM, Russell Bryant wrote: > > > > diff --git a/Documentation/release-process.md b/Documentation/release- > process.md > > index 0f8f49d..0c53812 100644 > > --- a/Docume

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

2016-10-29 Thread Russell Bryant
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
vSwitch release work with? -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH RFC v4 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-10-29 Thread Russell Bryant
StructuredText document so some changes are needed. For > to be clear, this is *now* a rst document ... > this to display correctly, you need to precede commands with a '$' > symbol, use the rST-style code block formatting and wrap at < 80 > characters, where possible. For example: > > :: > > $ ovs-vsctl add-port br0 dpdk0 \ > -- set Interface dpdk0 type=dpdk options:dpdk-devargs=:06: > 00.0 > > See the DocumentationStyle doc for more info. > ___ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > -- 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 wrote: > Rename it to OPENFLOW. > > Signed-off-by: Stephen Finucane > I have applied patches 11 and 12 to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://ope

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

2016-10-29 Thread Russell Bryant
DIFY 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
TFORWARD`` (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 wrote: > On 28 Oct 23:25, Russell Bryant wrote: > > On Tue, Oct 18, 2016 at 10:03 PM, Stephen Finucane > > wrote: > > > > > Signed-off-by: Stephen Finucane > > > --- > > > Makefile.am

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

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

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

2016-10-28 Thread Russell Bryant
e-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
ting 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 wrote: > > > On Sun, Oct 23, 2016 at 8:00 AM, Stephen Fincane > wrote: > >> On Fri, 2016-10-21 at 16:09 -0400, Russell Bryant wrote: >> > >> > >> > On Tue, Oct 18, 2016 at 4:03 PM, Stephen Fin

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 wrote: > > From: "Russell Bryant" > > To: dev@openvswitch.org > > Sent: Tuesday, October 25, 2016 10:12:32 AM > > Subject: [ovs-dev] [PATCH] ovn/TODO: Remove completed IPAM item. > > > > Signed-off

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

2016-10-25 Thread Russell Bryant
Signed-off-by: Russell Bryant --- 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 for the VM and attach to 409

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 wrote: > On Fri, 2016-10-21 at 16:09 -0400, Russell Bryant wrote: > > > > > > On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane > > wrote: > > > Signed-off-by: Stephen Finucane > > [snip] > > &

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 wrote: > > On Thu, Oct 20, 2016 at 11:51 AM, Russell Bryant wrote: > > > > On Thu, Oct 20, 2016 at 1:47 PM, Ben Pfaff wrote: > > > > > On Thu, Oct 13, 2016 at 07:32:53PM +0530, Numan Siddique wrote: > > > >

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

2016-10-21 Thread Russell Bryant
erformance within > +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

Re: [ovs-dev] [PATCH v2] doc: v2: fix bad link to dpdk advance installation guide

2016-10-21 Thread Russell Bryant
; > > Drat - case sensitivity. > > Acked-by: Stephen Finucane > I applied this patch to master. I'll look out for another revision of the first patch. Thanks, -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 06/15] doc: Convert REPORTING-BUGS to rST

2016-10-21 Thread Russell Bryant
G-BUGS.rst | 73 ++ > > SECURITY.md| 2 +- > 4 files changed, 75 insertions(+), 60 deletions(-) > delete mode 100644 REPORTING-BUGS.md > create mode 100644 REPORTING-BUGS.rst > applied to master. -- Russell Bryant ___ dev mail

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

2016-10-21 Thread Russell Bryant
On Fri, Oct 21, 2016 at 6:57 AM, Stephen Finucane wrote: > PS: The online viewer at http://rst.ninjs.org/ is helpful for previewing > these things. > This seems like a good tip for the doc guide. -- Russell Bryant ___ dev mailing

Re: [ovs-dev] [PATCH 05/15] doc: Convert README to rST

2016-10-21 Thread Russell Bryant
- `Fedora `__ - `RHEL `__ - `XenServer `__ -- `Windows `__ -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 04/15] doc: Convert PORTING to rST

2016-10-21 Thread Russell Bryant
e, however, because every packet passes through the ovs-vswitchd process. See the `userspace installation guide -` for instructions on how to configure a userspace +` for instructions on how to configure a userspace switch. -- Russell Bryant ___ de

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

2016-10-20 Thread Russell Bryant
's how we'd recommend it to be done in production, so the feature would become a test-only feature, but then the tests wouldn't be helping ensure we only read from the sb db otherwise. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 03/15] doc: Convert MAINTAINERS to rST

2016-10-20 Thread Russell Bryant
hanged, 76 insertions(+), 30 deletions(-) > delete mode 100644 MAINTAINERS.md > create mode 100644 MAINTAINERS.rst > applied to master, thanks. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 02/15] doc: Convert IntegrationGuide to rST

2016-10-20 Thread Russell Bryant
ide.rst > applied to master, thanks. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Russell Bryant
md @@ -313,5 +313,5 @@ Bug Reporting Please report problems to b...@openvswitch.org. -[CONTRIBUTING.rst]: CONTRIBUTING.rst +[CONTRIBUTING.md]: CONTRIBUTING.md [CodingStyle.rst]: CodingStyle.rst -- Russell Bryant ___ dev mailing list dev@openvswitc

Re: [ovs-dev] [PATCH] fedora: do not restart the service on a pkg upgrade

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 8:48 AM, Flavio Leitner wrote: > On Wed, Oct 19, 2016 at 03:26:10PM -0400, Russell Bryant wrote: > > On Tue, Oct 18, 2016 at 1:04 PM, Flavio Leitner wrote: > > > > > There is no reliable way to restore the previous networking > > > st

Re: [ovs-dev] [PATCH] DocumentationStyle: Clarify what documentation is

2016-10-20 Thread Russell Bryant
-- > DocumentationStyle.rst | 10 -- > 1 file changed, 4 insertions(+), 6 deletions(-) > applied to master, thanks. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-20 Thread Russell Bryant
On Thu, Oct 20, 2016 at 3:07 AM, Stephen Finucane wrote: > On 2016-10-20 01:25, Russell Bryant wrote: > >> On Tue, Oct 18, 2016 at 4:03 PM, Stephen Finucane >> wrote: >> >> Signed-off-by: Stephen Finucane >>> --- >>> CONT

Re: [ovs-dev] [PATCH 01/15] doc: Convert CodingStyle to rST

2016-10-19 Thread Russell Bryant
tyle.rst | 642 ++ > + > Makefile.am | 4 +- > OPENFLOW-1.1+.md | 6 +- > 5 files changed, 649 insertions(+), 585 deletions(-) > delete mode 100644 CodingStyle.md > create mode 100644 CodingStyle.rst > Unfortunately this patch needs a rebase. Can

Re: [ovs-dev] [PATCH] fedora: do not restart the service on a pkg upgrade

2016-10-19 Thread Russell Bryant
> The OVS might be providing the network used for remote > administration, so do not automatically restart the service > during a package upgrade. > > Signed-off-by: Flavio Leitner > Do you think this should be backported, as well? -- Russell Bryant __

Re: [ovs-dev] [PATCH] doc: fix bad link to dpdk advance installation guide

2016-10-19 Thread Russell Bryant
>> >> Signed-off-by: Mauricio Vasquez B >> > > I don't know the custom on the OVS dev mailing list, but... > > Reviewed-by: Stephen Finucane Acked-by in this case. We have a document for it. :-) https://github.com/openvswitch/ovs/blob/master/CONTRIBU

Re: [ovs-dev] [PATCH] Makefile.am: Add DocumentationStyle.rst to distribution.

2016-10-19 Thread Russell Bryant
On Tue, Oct 18, 2016 at 7:59 PM, Jarno Rajahalme wrote: > Compile fails otherwise. > > Signed-off-by: Jarno Rajahalme > --- > Makefile.am | 1 + > 1 file changed, 1 insertion(+) > Sorry about that. I went ahead and applied this to unbreak master

Re: [ovs-dev] [PATCH 6/9] doc: Convert INSTALL.Windows to rST

2016-10-18 Thread Russell Bryant
On Tue, Oct 18, 2016 at 2:55 PM, Stephen Fincane wrote: > On Tue, 2016-10-18 at 14:18 -0400, Russell Bryant wrote: > > The old doc used "% command" as the prompt prefix. I see that in the > > new doc it is "$ command". I'm not a Windows person, but f

Re: [ovs-dev] [PATCH 1/9] dist-docs: Add support for rST

2016-10-18 Thread Russell Bryant
7;rst2html'. > This tool is packaged on Ubuntu, Fedora (via 'python-docutils'), etc. > and can be installed from pip using the 'docsutils' package. > > Signed-off-by: Stephen Finucane > I've applied this whole series to master with minor tweaks mentione

Re: [ovs-dev] [PATCH 1/2] docs: Add documentation style guide

2016-10-18 Thread Russell Bryant
es a change in content > nesting, > + by either closing the existing level or introducing a new level. > + > +- Avoid trailing spaces on lines. > + > +- Use the following headers levels. > + > + | === Heading 0 (reserved for the title in a document) > + | --- Heading 1 >

Re: [ovs-dev] [PATCH 8/9] doc: Convert INSTALL.XenServer to rST

2016-10-18 Thread Russell Bryant
| 4 +- > 4 files changed, 228 insertions(+), 4 deletions(-) > create mode 100644 INSTALL.XenServer.rst > This one was just missing the removal of INSTALL.XenServer.md. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://op

Re: [ovs-dev] [PATCH 6/9] doc: Convert INSTALL.Windows to rST

2016-10-18 Thread Russell Bryant
shot, then run boot.sh in the top source directory to build > +the "configure" script::: > + > +$ ./boot.sh > The old doc used "% command" as the prompt prefix. I see that in the new doc it is "$ command". I'm not a Windows person, but from a qui

Re: [ovs-dev] OVS 2.6.1 release?

2016-10-18 Thread Russell Bryant
On Tue, Oct 18, 2016 at 4:54 AM, Justin Pettit wrote: > > On Oct 17, 2016, at 5:54 PM, Ben Pfaff wrote: > > > >> On Mon, Oct 17, 2016 at 09:14:33AM -0400, Russell Bryant wrote: > >> Can we do a 2.6.1 release in the near future? The latest Python commit

Re: [ovs-dev] [PATCH 2/9] doc: Convert INSTALL to rST

2016-10-17 Thread Russell Bryant
On Mon, Oct 17, 2016 at 8:19 AM, Stephen Finucane wrote: > On 15 Oct 22:01, Russell Bryant wrote: > > On Sat, Oct 15, 2016 at 9:57 PM, Russell Bryant wrote: > > Very minor nit ... I think I would find it easier to review these if > > content changes were separate from the

Re: [ovs-dev] [PATCH 1/9] dist-docs: Add support for rST

2016-10-17 Thread Russell Bryant
On Mon, Oct 17, 2016 at 8:21 AM, Stephen Finucane wrote: > On 15 Oct 21:20, Russell Bryant wrote: > > On Sat, Oct 8, 2016 at 12:30 PM, Stephen Finucane > wrote: > > I added this line here: > > > > +title=`grep -A 1 -e "^=" $srcdir/$file | sed

[ovs-dev] OVS 2.6.1 release?

2016-10-17 Thread Russell Bryant
ress. 578d385 ovn: Replace tabs with spaces and clean up alignment in unit tests. c37c4a7 Prepare for 2.6.1. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/9] doc: Convert INSTALL to rST

2016-10-15 Thread Russell Bryant
On Sat, Oct 15, 2016 at 9:57 PM, Russell Bryant wrote: > > On Sat, Oct 8, 2016 at 12:30 PM, Stephen Finucane > wrote: > >> Signed-off-by: Stephen Finucane >> --- >> CONTRIBUTING.md | 6 +- >> FAQ.md | 10 +

Re: [ovs-dev] [PATCH 2/9] doc: Convert INSTALL to rST

2016-10-15 Thread Russell Bryant
On Sat, Oct 8, 2016 at 12:30 PM, Stephen Finucane wrote: > Signed-off-by: Stephen Finucane > --- > CONTRIBUTING.md | 6 +- > FAQ.md | 10 +- > INSTALL.DPDK-ADVANCED.md | 2 +- > INSTALL.DPDK.md | 6 +- > INSTALL.Debian.md| 4 +- > INSTALL.Doc

Re: [ovs-dev] [PATCH 1/9] dist-docs: Add support for rST

2016-10-15 Thread Russell Bryant
r/$file.txt" --stylesheet-path="style.css" \ > +--link-stylesheet --title="$file (Open vSwitch $VERSION)" > \ > + > "$distdir/$file.html" > +cat < + > + $file > + $title > + HTML, plain > text > + > +EOF > +;; > + > *) > cp "$srcdir/$file" "$distdir/$file" > cat < -- > 2.7.4 > > ___ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v1] Python-IDL: getattr after mutate fix

2016-10-14 Thread Russell Bryant
: http://openvswitch.org/pipermail/dev/2016-September/ > 080120.html > > Fixes: a59912a0ee8e ("python: Add support for partial map and set > updates") > Thanks! I applied this to master and branch-2.6. -- Russell Bryant

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

2016-10-14 Thread Russell Bryant
On Wed, Oct 12, 2016 at 12:03 PM, Russell Bryant wrote: > > > On Fri, Oct 7, 2016 at 1:48 PM, Flavio Leitner wrote: > >> On Thu, Sep 08, 2016 at 07:58:21PM +0100, Markos Chandras wrote: >> > On 09/08/2016 05:50 PM, Aaron Conole wrote: >> > > >> >

Re: [ovs-dev] [PATCH] rhel-systemd: Delay shutting down the services

2016-10-14 Thread Russell Bryant
On Fri, Oct 14, 2016 at 9:36 PM, Russell Bryant wrote: > > On Thu, Oct 13, 2016 at 9:44 AM, Flavio Leitner wrote: > >> On Wed, Oct 12, 2016 at 11:56:51AM -0400, Russell Bryant wrote: >> > On Fri, Oct 7, 2016 at 1:36 PM, Aaron Conole >> wrote: >> >

Re: [ovs-dev] [PATCH] rhel-systemd: Delay shutting down the services

2016-10-14 Thread Russell Bryant
On Thu, Oct 13, 2016 at 9:44 AM, Flavio Leitner wrote: > On Wed, Oct 12, 2016 at 11:56:51AM -0400, Russell Bryant wrote: > > On Fri, Oct 7, 2016 at 1:36 PM, Aaron Conole wrote: > > > > > During testing it was found that systemd would consider the openvswitch &g

Re: [ovs-dev] [PATCH 0/9 md->rst] Convert most INSTALL guides

2016-10-13 Thread Russell Bryant
there's a bunch I haven't looked at. Thanks a lot for working on this!! -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 0/9 md->rst] Convert most INSTALL guides

2016-10-13 Thread Russell Bryant
On Thu, Oct 13, 2016 at 9:32 AM, Ben Pfaff wrote: > On Thu, Oct 13, 2016 at 09:10:58AM -0400, Russell Bryant wrote: > > On Thu, Oct 13, 2016 at 6:00 AM, Stephen Finucane > wrote: > > > > > On 2016-10-12 20:20, Russell Bryant wrote: > > > > > >> O

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

2016-10-13 Thread Russell Bryant
I should have also noted who was already looking at each of these items ... On Thu, Oct 13, 2016 at 10:02 AM, Numan Siddique wrote: > We may have to add one more item in the task breakdown list. Please see > below > > > On Wed, Oct 12, 2016 at 11:21 PM, Russell Bryant wrote:

Re: [ovs-dev] [PATCH 0/9 md->rst] Convert most INSTALL guides

2016-10-13 Thread Russell Bryant
On Thu, Oct 13, 2016 at 6:00 AM, Stephen Finucane wrote: > On 2016-10-12 20:20, Russell Bryant wrote: > >> On Sat, Oct 8, 2016 at 12:30 PM, Stephen Finucane >> wrote: >> >> I'm going to dripfeed the conversion patches to avoid overloading >>> peoples

Re: [ovs-dev] [PATCH 0/9 md->rst] Convert most INSTALL guides

2016-10-12 Thread Russell Bryant
ALL.KVM to rST > Do you have the end result published anywhere? It would be nice to see the before and after before diving into the detailed reviews. Thanks, -- Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v1] Python-IDL: getattr after mutate fix

2016-10-12 Thread Russell Bryant
et updates") Signed-off-by: Amitabha Biswas > Your Signed-off-by doesn't match the commit author (your gmail address). Can you re-submit with the two matching? > Reported-by: Richard Theis Other than the Signed-off-by issue, I think this is r

[ovs-dev] ovn: Improving southbound database security

2016-10-12 Thread Russell Bryant
pping the table seems not backwards compatible and would break a rolling upgrade scenario. Should we leave it as unused for one release, and then remove it in the next release? More generally, I think we need to document our upgrade strategy and related rules for database

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

2016-10-12 Thread Russell Bryant
that depends on network. When OVS > terminates, it breaks networking connectivity causing issues. The patchset > intends to fix the issue as well, though it needs to include the follow up > fix http://openvswitch.org/pipermail/dev/2016-October/

Re: [ovs-dev] [PATCH] rhel-systemd: Delay shutting down the services

2016-10-12 Thread Russell Bryant
e a Signed-off-by for this patch, please? -- Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-10-03 Thread Russell Bryant
s allocation. > > Example: >ovn-nbctl lsp-set-addresses p1 "00:01:02:03:04:05 dynamic" > > Signed-off-by: Lance Richardson > Acked-by: Russell Bryant This looks good to me. It's probably worth adding a line to NEWS to document the improvement. We could ju

Re: [ovs-dev] [PATCH] ovn: Avoid using bash "+=" extension in tests.

2016-10-03 Thread Russell Bryant
On Fri, Sep 30, 2016 at 1:12 PM, Ben Pfaff wrote: > This fixes test failures when the tests are run under a shell like ash or > dash. > > Signed-off-by: Ben Pfaff > Acked-by: Russell Bryant -- Russell Bryant ___ dev m

Re: [ovs-dev] Status on etcd?

2016-10-03 Thread Russell Bryant
On Sun, Oct 2, 2016 at 3:34 AM, Frederick Kautz wrote: > I recall hearing that someone was adding etcd v3 support to OVN. Does > anyone have any info on whether this is still happening? > I believe Andy Zhou was the one most actively researching approaches. CC'd. --

Re: [ovs-dev] [PATCH] ovn: update docs for l2gateway port vlan tag

2016-09-30 Thread Russell Bryant
On Fri, Sep 30, 2016 at 1:29 AM, Ben Pfaff wrote: > What a nice introduction! > Agreed! > Russell, do you want to review this? Yes. The patch looks good. I trimmed the introduction from the commit message, added Gabe to the AUTHORS file, and applied this to master and branch-2.6

Re: [ovs-dev] OVS 2.6 release proposal

2016-09-21 Thread Russell Bryant
On Mon, Sep 19, 2016 at 11:50 AM, Ben Pfaff wrote: > I suggest that we plan to release OVS 2.6.0 this Thursday, Sept. 22, > unless significant new bugs appear before then. > > Comments? > +1 -- Russell Bryant ___ dev mailing list dev@

Re: [ovs-dev] [PATCH v2 0/4] OCF script for OVN DB servers

2016-09-21 Thread Russell Bryant
fedora packager > ovn: Add ovndb servers ocf script in debian packager > Based on our discussion earlier today, I'm expecting a v3 before I should review this, right? -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn: Change ct_label[0] to ct_label.blocked

2016-09-21 Thread Russell Bryant
ted-by: Justin Pettit > Suggested-at: http://openvswitch.org/pipermail/dev/2016-July/075773.html > Signed-off-by: Babu Shanmugam > Thanks! I applied this to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-09-19 Thread Russell Bryant
to run kernel datapath test when building rpms > > Russell, will you please consider this? I'm fine with this. I have backported it to branch-2.6. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-09-09 Thread Russell Bryant
ested-by: Flavio Leitner > > Signed-off-by: Lance Richardson > > --- > > Acked-by: Flavio Leitner Thanks, applied to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-09-09 Thread Russell Bryant
> Acked-by: Flavio Leitner Thanks, applied to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] ovn: ensure datapath removal is proper

2016-09-08 Thread Russell Bryant
ace condition between deleting the logical switch and your check here. One solution would be to just use OVS_WAIT_UNTIL() to wait until the test passes (or we eventually give up that it's never going to pass, because we've given it enough time). > + > +OVN_CLEANUP([hv1]) > +

Re: [ovs-dev] [PATCH] ovn-nbctl: Update man page for missing tables.

2016-09-08 Thread Russell Bryant
tions(+), 4 deletions(-) > Thanks, I applied this to master and branch-2.6. -- Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn-controller: Fix match crieria for dynamic mac binding flows

2016-09-08 Thread Russell Bryant
appears to be the only place consider_neighbor_flow() is used, so it's not clear there is any value for having match passed in as an argument at all. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-09-08 Thread Russell Bryant
would result in a roughly 1/2 hour wait time > to stop). > > Apologies if this is considered in-appropriate for the 2.5 branch, or if > no new backport requests are accepted on the 2.5 branch. > What's the upgrade impact of this for someone using 2.5.0 and then moving to a ver

Re: [ovs-dev] [PATCH] ovn: fix ovn-northd leaks in build_acl

2016-09-08 Thread Russell Bryant
lows.constprop.36 (ovn-northd.c:2472) >by 0x4072D9: build_lflows (ovn-northd.c:3845) >by 0x4072D9: ovnnb_db_run (ovn-northd.c:3971) >by 0x4072D9: main (ovn-northd.c:4375) > > Signed-off-by: Ramu Ramamurthy > Acked-by: Ryan Moats > I've applied this to master

Re: [ovs-dev] [PATCH] ovn: fix ovn-northd leaks in build_acl

2016-09-01 Thread Russell Bryant
+) > I was about to apply this, but it's missing your Signed-off-by. Can you please re-submit with Signed-off-by included? Thanks, -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn-controller: Fix leak in patched_datapaths processing.

2016-09-01 Thread Russell Bryant
On Thu, Sep 1, 2016 at 12:42 PM, Ben Pfaff wrote: > Nothing freed 'key', which was dynamically allocated. This commit changes > 'key' so that it is no longer dynamically allocated. > > Reported-by: Ryan Moats > Signed-off-by: Ben

Re: [ovs-dev] [PATCH 2/2] ovn-controller: Drop incremental processing from encapsulation code.

2016-08-31 Thread Russell Bryant
functions > now available in the IDL to simplify some code. > > Signed-off-by: Ben Pfaff > Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/2] ovsdb-idlc: Make set and map update operations take const arguments.

2016-08-31 Thread Russell Bryant
ist of ports in the > bridge for that particular port and, if it finds it, removes it. > > Signed-off-by: Ben Pfaff > Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v4] ovn-controller: Back out incremental processing

2016-08-25 Thread Russell Bryant
On Wed, Aug 24, 2016 at 9:30 PM, Ryan Moats wrote: > As [1] indicates, incremental processing hasn't resulted > in an improvement worth the complexity it has added. > This patch backs out all of the code specific to incremental > processing, along with the persisting of OF flows, > logical ports,

Re: [ovs-dev] [PATCH] rhel: Include ovn-trace in ovn-common package

2016-08-23 Thread Russell Bryant
s to master and branch-2.6. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-08-23 Thread Russell Bryant
man8/ovn-nbctl.8* > %{_mandir}/man7/ovn-architecture.7* > %{_mandir}/man8/ovn-sbctl.8* > +%{_mandir}/man8/ovn-trace.8* > %{_mandir}/man5/ovn-nb.5* > %{_mandir}/man5/ovn-sb.5* > Thanks for the fix! Babu submitted the same patch a little while before you did

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

2016-08-23 Thread Russell Bryant
On Tue, Aug 23, 2016 at 5:05 PM, Darrell Ball wrote: > > > On Mon, Aug 22, 2016 at 1:08 PM, Lance Richardson > wrote: > >> > From: "Ben Pfaff" >> > To: "Russell Bryant" >> > Cc: "Lance Richardson" , "ovs dev" &l

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

2016-08-22 Thread Russell Bryant
#2. The only OVSDB feature required in that case is to accept read-only connections, which could be on a per-socket basis. This seems much simpler all around, as long as we can all get on board with ovn-controller as a read-only client. Are you interested in looking closer at what #1 would look like, with details of what the access control policy would look like? -- Russell Bryant ___ 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 Russell Bryant
they wouldn't be transferred from home > > host to another. Each chassis would be responsible for its own mac > learning.​ > > That's what I'm not comfortable with... Why is that? Isn't that how a network would typically work anyway? -- Russell Bryant __

Re: [ovs-dev] [PATCH 2/2] ovn/TODO: Add items proposed for 2.7 in OVN IRC meeting.

2016-08-22 Thread Russell Bryant
+ > 1 file changed, 57 insertions(+) > > diff --git a/ovn/TODO b/ovn/TODO > index b3c4831..91600f7 100644 > --- a/ovn/TODO > +++ b/ovn/TODO > @@ -1,5 +1,62 @@ > -*- outline -*- > > +* Work out database for clustering or HA properly.

  1   2   3   4   5   6   7   8   9   10   >