[ovs-dev] [PATCH v2 1/1] doc: Remove "experimental" warning for userspace.

[Stokes, Ian]

> On 02/02/2017 04:44 PM, Ian Stokes wrote:
> > Remove the experimental warning tag in documentation regarding OVS
> > deployed via userspace.
> >
> > Signed-off-by: Ian Stokes <ian.stokes at intel.com>
> > ---
> >  Documentation/intro/install/dpdk.rst      |    3 ---
> >  Documentation/intro/install/userspace.rst |    4 ----
> >  NEWS                                      |    2 ++
> >  README.rst                                |    6 +++---
> >  4 files changed, 5 insertions(+), 10 deletions(-)
> >
> > diff --git a/Documentation/intro/install/dpdk.rst
> > b/Documentation/intro/install/dpdk.rst
> > index fff0a1a..3018590 100644
> > --- a/Documentation/intro/install/dpdk.rst
> > +++ b/Documentation/intro/install/dpdk.rst
> > @@ -29,9 +29,6 @@ This document describes how to build and install
> > Open vSwitch using a DPDK  datapath. Open vSwitch can use the DPDK
> > library to operate entirely in  userspace.
> >
> > -.. warning::
> > -  The DPDK support of Open vSwitch is considered 'experimental'.
> > -
> >  Build requirements
> >  ------------------
> >
> > diff --git a/Documentation/intro/install/userspace.rst
> > b/Documentation/intro/install/userspace.rst
> > index 0368527..ebd0c12 100644
> > --- a/Documentation/intro/install/userspace.rst
> > +++ b/Documentation/intro/install/userspace.rst
> > @@ -34,10 +34,6 @@ This version of Open vSwitch should be built
> > manually with ``configure`` and  been recently tested, and so Debian
> > packages are not a recommended way to use  this version of Open vSwitch.
> >
> > -.. warning::
> > -  The userspace-only mode of Open vSwitch is considered experimental.
> > It has
> > -  not been thoroughly tested.
> > -
> >  Building and Installing
> >  -----------------------
> >
> > diff --git a/NEWS b/NEWS
> > index 5efcce2..8600f0e 100644
> > --- a/NEWS
> > +++ b/NEWS
> > @@ -3,6 +3,8 @@ Post-v2.7.0
> >     - Tunnels:
> >       * Added support to set packet mark for tunnel endpoint using
> >         `egress_pkt_mark` OVSDB option.
> > +   - Documentation:
> > +     * OVS deployed in userspace mode no longer tagged as experimental.
> 
> I think this would be a bit clearer. What do you think?
> 
> --- a/NEWS
> +++ b/NEWS
> @@ -59,4 +59,5 @@ v2.7.0 - xx xxx xxxx
>       * Removed support for IPsec tunnels.
>     - DPDK:
> +     * Removal of experimental tag.

We can put it under a DPDK header like you have here, I wonder though is the removal of experimental only relevant to DPDK or do we need to consider OVS userspace without DPDK also? I would think both are well tested at this stage so experimental could be removed for both. Thoughts?

> 
> 
> >
> >  v2.7.0 - xx xxx xxxx
> >  ---------------------
> > diff --git a/README.rst b/README.rst
> > index f5cdaa5..90050e3 100644
> > --- a/README.rst
> > +++ b/README.rst
> > @@ -38,9 +38,9 @@ following features:
> >
> >  The included Linux kernel module supports Linux 3.10 and up.
> >
> > -Open vSwitch can also operate, at a cost in performance, entirely in
> > userspace, -without assistance from a kernel module.  This userspace
> > implementation should -be easier to port than the kernel-based switch.
> It is considered experimental.
> > +Open vSwitch can also operate entirely in userspace without
> > +assistance from a kernel module.  This userspace implementation
> > +should be easier to port than the kernel-based switch.
> >
> >  What's here?
> >  ------------
> >