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  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  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 up
> with
> > > > the pace of OVN and is no longer a good current resource.
> > > >
> > > > For a sandbox based tutorial like this, I'd like to start over using
> > > > ovn-trace as the basis.
> > > >
> > > > 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 
> > >
> > > I'm a little sad to see this go, but if it's no longer a good
> > > introduction then it has to happen.  Thank you for all the work on it.
> > >
> >
> > Thanks.  I'm actually happy that OVN has progressed quickly and we have
> > much better tools for working with OVN and analyzing flows now.  :-)
> >
> > If I put energy into this, I'd like to write a tutorial around ovn-trace.
> > Thanks for all the work you put into that!
>
> You're welcome!
>
> It sounds possible that OVS/OVN will have a 3-hour tutorial slot at ONS
> in April.  If so, we'll need to write a tutorial.  Maybe we can
> collaborate.
>

Sure.  The dates aren't great for me to be there in person (conflict with
wedding anniversary), but I'd certainly be happy to collaborate on content.

-- 
Russell Bryant
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

2017-01-23 Thread Ben Pfaff 
On Mon, Jan 23, 2017 at 09:02:07AM -0500, Russell Bryant wrote:
> On Thu, Jan 19, 2017 at 6:01 PM, Ben Pfaff  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 up with
> > > the pace of OVN and is no longer a good current resource.
> > >
> > > For a sandbox based tutorial like this, I'd like to start over using
> > > ovn-trace as the basis.
> > >
> > > 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 
> >
> > I'm a little sad to see this go, but if it's no longer a good
> > introduction then it has to happen.  Thank you for all the work on it.
> >
> 
> Thanks.  I'm actually happy that OVN has progressed quickly and we have
> much better tools for working with OVN and analyzing flows now.  :-)
> 
> If I put energy into this, I'd like to write a tutorial around ovn-trace.
> Thanks for all the work you put into that!

You're welcome!

It sounds possible that OVS/OVN will have a 3-hour tutorial slot at ONS
in April.  If so, we'll need to write a tutorial.  Maybe we can
collaborate.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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  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 up with
> > the pace of OVN and is no longer a good current resource.
> >
> > For a sandbox based tutorial like this, I'd like to start over using
> > ovn-trace as the basis.
> >
> > 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 
>
> I'm a little sad to see this go, but if it's no longer a good
> introduction then it has to happen.  Thank you for all the work on it.
>

Thanks.  I'm actually happy that OVN has progressed quickly and we have
much better tools for working with OVN and analyzing flows now.  :-)

If I put energy into this, I'd like to write a tutorial around ovn-trace.
Thanks for all the work you put into that!


> Acked-by: Ben Pfaff 
>

Thanks, 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] doc: Remove tutorials/ovn-basics.

2017-01-19 Thread Ben Pfaff 
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 up with
> the pace of OVN and is no longer a good current resource.
> 
> For a sandbox based tutorial like this, I'd like to start over using
> ovn-trace as the basis.
> 
> 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 

I'm a little sad to see this go, but if it's no longer a good
introduction then it has to happen.  Thank you for all the work on it.

Acked-by: Ben Pfaff 
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

2017-01-19 Thread Russell Bryant 
The only thing worse than a lack of documentation is incorrect or
out-of-date documentation.  Over time, this document has not kept up with
the pace of OVN and is no longer a good current resource.

For a sandbox based tutorial like this, I'd like to start over using
ovn-trace as the basis.

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 
---
 Documentation/automake.mk  |   1 -
 Documentation/index.rst|   1 -
 Documentation/tutorials/index.rst  |   1 -
 Documentation/tutorials/ovn-basics.rst | 974 -
 tutorial/automake.mk   |  33 +-
 tutorial/ovn/env1/add-security-ip-ports.sh |  25 -
 tutorial/ovn/env1/add-third-port.sh|  21 -
 tutorial/ovn/env1/add-unknown-ports.sh |  25 -
 tutorial/ovn/env1/packet1.sh   |  19 -
 tutorial/ovn/env1/packet2.sh   |  19 -
 tutorial/ovn/env1/packet3.sh   |  19 -
 tutorial/ovn/env1/packet4.sh   |  19 -
 tutorial/ovn/env1/setup.sh |  46 --
 tutorial/ovn/env2/packet1.sh   |  18 -
 tutorial/ovn/env2/packet2.sh   |  18 -
 tutorial/ovn/env2/setup.sh |  36 --
 tutorial/ovn/env3/packet1.sh   |  19 -
 tutorial/ovn/env3/packet2.sh   |  31 -
 tutorial/ovn/env3/setup.sh |  44 --
 tutorial/ovn/env4/packet1.sh   |  21 -
 tutorial/ovn/env4/packet2.sh   |  21 -
 tutorial/ovn/env4/packet3.sh   |  20 -
 tutorial/ovn/env4/packet4.sh   |  20 -
 tutorial/ovn/env4/setup.sh |  50 --
 tutorial/ovn/env5/packet1.sh   |  21 -
 tutorial/ovn/env5/packet2.sh   |  20 -
 tutorial/ovn/env5/setup.sh |  67 --
 tutorial/ovn/env6/add-acls.sh  |  21 -
 tutorial/ovn/env6/setup.sh |  46 --
 tutorial/ovn/env7/add-container-ports.sh   |  60 --
 tutorial/ovn/env7/packet1.sh   |  19 -
 tutorial/ovn/env7/packet2.sh   |  19 -
 tutorial/ovn/env7/setup.sh |  36 --
 tutorial/ovn/env8/packet1.sh   |  21 -
 tutorial/ovn/env8/packet2.sh   |  20 -
 tutorial/ovn/env8/setup.sh |  47 --
 36 files changed, 1 insertion(+), 1897 deletions(-)
 delete mode 100644 Documentation/tutorials/ovn-basics.rst
 delete mode 100755 tutorial/ovn/env1/add-security-ip-ports.sh
 delete mode 100755 tutorial/ovn/env1/add-third-port.sh
 delete mode 100755 tutorial/ovn/env1/add-unknown-ports.sh
 delete mode 100755 tutorial/ovn/env1/packet1.sh
 delete mode 100755 tutorial/ovn/env1/packet2.sh
 delete mode 100755 tutorial/ovn/env1/packet3.sh
 delete mode 100755 tutorial/ovn/env1/packet4.sh
 delete mode 100755 tutorial/ovn/env1/setup.sh
 delete mode 100755 tutorial/ovn/env2/packet1.sh
 delete mode 100755 tutorial/ovn/env2/packet2.sh
 delete mode 100755 tutorial/ovn/env2/setup.sh
 delete mode 100755 tutorial/ovn/env3/packet1.sh
 delete mode 100755 tutorial/ovn/env3/packet2.sh
 delete mode 100755 tutorial/ovn/env3/setup.sh
 delete mode 100755 tutorial/ovn/env4/packet1.sh
 delete mode 100755 tutorial/ovn/env4/packet2.sh
 delete mode 100755 tutorial/ovn/env4/packet3.sh
 delete mode 100755 tutorial/ovn/env4/packet4.sh
 delete mode 100755 tutorial/ovn/env4/setup.sh
 delete mode 100755 tutorial/ovn/env5/packet1.sh
 delete mode 100755 tutorial/ovn/env5/packet2.sh
 delete mode 100755 tutorial/ovn/env5/setup.sh
 delete mode 100755 tutorial/ovn/env6/add-acls.sh
 delete mode 100755 tutorial/ovn/env6/setup.sh
 delete mode 100755 tutorial/ovn/env7/add-container-ports.sh
 delete mode 100755 tutorial/ovn/env7/packet1.sh
 delete mode 100755 tutorial/ovn/env7/packet2.sh
 delete mode 100755 tutorial/ovn/env7/setup.sh
 delete mode 100755 tutorial/ovn/env8/packet1.sh
 delete mode 100755 tutorial/ovn/env8/packet2.sh
 delete mode 100755 tutorial/ovn/env8/setup.sh

diff --git a/Documentation/automake.mk b/Documentation/automake.mk
index 25c14be..18e03a3 100644
--- a/Documentation/automake.mk
+++ b/Documentation/automake.mk
@@ -21,7 +21,6 @@ EXTRA_DIST += \
Documentation/intro/install/windows.rst \
Documentation/intro/install/xenserver.rst \
Documentation/tutorials/index.rst \
-   Documentation/tutorials/ovn-basics.rst \
Documentation/tutorials/ovs-advanced.rst \
Documentation/topics/index.rst \
Documentation/topics/bonding.rst \
diff --git a/Documentation/index.rst b/Documentation/index.rst
index deb44d3..02b376f 100644
--- a/Documentation/index.rst
+++ b/Documentation/index.rst
@@ -61,7 +61,6 @@ vSwitch? Start here.
   :doc:`intro/install/dpdk`
 
 - **Tutorials:** :doc:`tutorials/ovs-advanced` |
-  :doc:`tutorials/ovn-basics`