Re: [ovs-dev] [PATCH] Documentation: Update Pacemaker main page link.

2024-05-03 Thread Simon Horman
On Wed, May 01, 2024 at 01:39:58PM +0200, Eelco Chaudron wrote: > > > On 1 May 2024, at 12:54, Simon Horman wrote: > > > Update link to pacemaker main page as the existing link is broken. > > Also, use HTTPS. > > > > Broken link flagged by make check-docs > > > > Signed-off-by: Simon Horman >

Re: [ovs-dev] [PATCH] Documentation: Update Pacemaker main page link.

2024-05-01 Thread Eelco Chaudron
On 1 May 2024, at 12:54, Simon Horman wrote: > Update link to pacemaker main page as the existing link is broken. > Also, use HTTPS. > > Broken link flagged by make check-docs > > Signed-off-by: Simon Horman > --- Thanks for fixing the link. Acked-by: Eelco Chaudron

Re: [ovs-dev] [PATCH] Documentation: Remove references to kernel datapath in OVS tree.

2024-03-22 Thread Simon Horman
On Thu, Mar 21, 2024 at 06:23:43PM +0100, Ilya Maximets wrote: > On 3/21/24 11:03, Simon Horman wrote: > > The Kernel datapath is no longer present in the primary development > > branch of the OVS tree. Remove references to this in the documentation. > > > > Signed-off-by: Simon Horman > > --- >

Re: [ovs-dev] [PATCH] Documentation: Remove references to kernel datapath in OVS tree.

2024-03-21 Thread Ilya Maximets
On 3/21/24 11:03, Simon Horman wrote: > The Kernel datapath is no longer present in the primary development > branch of the OVS tree. Remove references to this in the documentation. > > Signed-off-by: Simon Horman > --- > Documentation/faq/releases.rst | 3 --- >

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2024-01-03 Thread Aaron Conole
Ilya Maximets writes: > On 12/21/23 14:50, Simon Horman wrote: >> On Thu, Dec 21, 2023 at 09:12:52AM +0100, Eelco Chaudron wrote: >>> >>> >>> On 14 Dec 2023, at 15:19, Simon Horman wrote: >>> As a community we should strive to be inclusive. As such it seems appropriate to adopt an word

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2024-01-02 Thread Ilya Maximets
On 12/21/23 14:50, Simon Horman wrote: > On Thu, Dec 21, 2023 at 09:12:52AM +0100, Eelco Chaudron wrote: >> >> >> On 14 Dec 2023, at 15:19, Simon Horman wrote: >> >>> As a community we should strive to be inclusive. >>> As such it seems appropriate to adopt an word list, >>> to help guide the use

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-21 Thread Simon Horman
On Thu, Dec 21, 2023 at 09:12:52AM +0100, Eelco Chaudron wrote: > > > On 14 Dec 2023, at 15:19, Simon Horman wrote: > > > As a community we should strive to be inclusive. > > As such it seems appropriate to adopt an word list, > > to help guide the use of inclusive language. > > > > This patch

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-21 Thread Simon Horman
On Thu, Dec 14, 2023 at 05:06:18PM -0500, Aaron Conole wrote: > Simon Horman writes: > > > As a community we should strive to be inclusive. > > As such it seems appropriate to adopt an word list, > > to help guide the use of inclusive language. > > > > This patch proposes use of the Inclusive

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-21 Thread Simon Horman
On Thu, Dec 14, 2023 at 02:19:14PM +, Simon Horman wrote: > As a community we should strive to be inclusive. > As such it seems appropriate to adopt an word list, > to help guide the use of inclusive language. > > This patch proposes use of the Inclusive Naming Word List v1.0. > > Link:

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-21 Thread Eelco Chaudron
On 14 Dec 2023, at 15:19, Simon Horman wrote: > As a community we should strive to be inclusive. > As such it seems appropriate to adopt an word list, > to help guide the use of inclusive language. > > This patch proposes use of the Inclusive Naming Word List v1.0. > > Link:

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-20 Thread Jakob Meng
On 14.12.23 23:06, Aaron Conole wrote: > Simon Horman writes: > >> As a community we should strive to be inclusive. >> As such it seems appropriate to adopt an word list, >> to help guide the use of inclusive language. >> >> This patch proposes use of the Inclusive Naming Word List v1.0. >> >>

Re: [ovs-dev] [PATCH] Documentation: Add section on inclusive language.

2023-12-14 Thread Aaron Conole
Simon Horman writes: > As a community we should strive to be inclusive. > As such it seems appropriate to adopt an word list, > to help guide the use of inclusive language. > > This patch proposes use of the Inclusive Naming Word List v1.0. > > Link: https://inclusivenaming.org/word-lists/ >

Re: [ovs-dev] [PATCH] Documentation: Add CVE-2022-40982, aka Downfall reference.

2023-10-05 Thread Eelco Chaudron
On 5 Oct 2023, at 9:51, Simon Horman wrote: > On Wed, Oct 04, 2023 at 02:59:48PM +0200, Eelco Chaudron wrote: >> Added a reference to the DPDK documentation as a result of >> analyzing the OVS code for potential performance impacts due >> to the Downfall mitigation. >> >> Signed-off-by: Eelco

Re: [ovs-dev] [PATCH] Documentation: Add CVE-2022-40982, aka Downfall reference.

2023-10-05 Thread Simon Horman
On Wed, Oct 04, 2023 at 02:59:48PM +0200, Eelco Chaudron wrote: > Added a reference to the DPDK documentation as a result of > analyzing the OVS code for potential performance impacts due > to the Downfall mitigation. > > Signed-off-by: Eelco Chaudron Acked-by: Simon Horman

Re: [ovs-dev] [PATCH] Documentation: Add CVE-2022-40982, aka Downfall reference.

2023-10-04 Thread Aaron Conole
Eelco Chaudron writes: > Added a reference to the DPDK documentation as a result of > analyzing the OVS code for potential performance impacts due > to the Downfall mitigation. > > Signed-off-by: Eelco Chaudron > --- Acked-by: Aaron Conole ___ dev

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread 0-day Robot
Bleep bloop. Greetings Wei Li, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (ovn-architecture.7.xml). error: could not build fake

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread 0-day Robot
Bleep bloop. Greetings Wei Li, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (ovn-architecture.7.xml). error: could not build fake

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread Ilya Maximets
On 5/5/23 10:57, Wei Li wrote: > Signed-off-by: Wei Li > --- > ovn-architecture.7.xml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ovn-architecture.7.xml b/ovn-architecture.7.xml > index cb1064f71..86c6258e0 100644 > --- a/ovn-architecture.7.xml > +++

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread 0-day Robot
Bleep bloop. Greetings Wei Li, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (ovn-architecture.7.xml). error: could not build fake

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread 0-day Robot
Bleep bloop. Greetings Wei Li, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (ovn-architecture.7.xml). error: could not build fake

Re: [ovs-dev] [PATCH] documentation: packets that arrive from other chassis resubmit to table 38

2023-05-05 Thread 0-day Robot
Bleep bloop. Greetings Wei Li, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (ovn-architecture.7.xml). error: could not build fake

Re: [ovs-dev] [PATCH] Documentation: Fix links in the DPDK guide on physical ports.

2023-01-09 Thread Ilya Maximets
On 1/9/23 16:41, David Marchand wrote: > On Fri, Jan 6, 2023 at 6:31 PM Ilya Maximets wrote: >> >> On 1/6/23 08:59, David Marchand wrote: >>> On Thu, Jan 5, 2023 at 8:34 PM Ilya Maximets wrote: The text enclosed in '<...>' supposed to be an actual link and not the name of the

Re: [ovs-dev] [PATCH] Documentation: Fix links in the DPDK guide on physical ports.

2023-01-09 Thread David Marchand
On Fri, Jan 6, 2023 at 6:31 PM Ilya Maximets wrote: > > On 1/6/23 08:59, David Marchand wrote: > > On Thu, Jan 5, 2023 at 8:34 PM Ilya Maximets wrote: > >> > >> The text enclosed in '<...>' supposed to be an actual link and not the > >> name of the link. This generates incorrect links that lead

Re: [ovs-dev] [PATCH] Documentation: Fix links in maintainers.rst.

2023-01-06 Thread Ilya Maximets
On 1/6/23 10:51, Eelco Chaudron wrote: > > > On 5 Jan 2023, at 20:42, Ilya Maximets wrote: > >> GitHub and Sphinx are parsing links differently. Sphinx knows about >> the overall documentation structure and all the sections defined in >> other docs, while GitHub is using direct rst 2 html

Re: [ovs-dev] [PATCH] Documentation: Fix links in the DPDK guide on physical ports.

2023-01-06 Thread Ilya Maximets
On 1/6/23 08:59, David Marchand wrote: > On Thu, Jan 5, 2023 at 8:34 PM Ilya Maximets wrote: >> >> The text enclosed in '<...>' supposed to be an actual link and not the >> name of the link. This generates incorrect links that lead nowhere. > > Indeed. > I was wondering if you have access to

Re: [ovs-dev] [PATCH] Documentation: Fix links in maintainers.rst.

2023-01-06 Thread Ilya Maximets
On 1/5/23 21:12, Han Zhou wrote: > > > On Thu, Jan 5, 2023 at 11:42 AM Ilya Maximets > wrote: >> >> GitHub and Sphinx are parsing links differently.  Sphinx knows about >> the overall documentation structure and all the sections defined in >> other docs, while GitHub

Re: [ovs-dev] [PATCH] Documentation: Fix links in maintainers.rst.

2023-01-06 Thread Eelco Chaudron
On 5 Jan 2023, at 20:42, Ilya Maximets wrote: > GitHub and Sphinx are parsing links differently. Sphinx knows about > the overall documentation structure and all the sections defined in > other docs, while GitHub is using direct rst 2 html conversion and > doesn't know any of that. Sphinx

Re: [ovs-dev] [PATCH] Documentation: Fix links in the DPDK guide on physical ports.

2023-01-06 Thread David Marchand
On Thu, Jan 5, 2023 at 8:34 PM Ilya Maximets wrote: > > The text enclosed in '<...>' supposed to be an actual link and not the > name of the link. This generates incorrect links that lead nowhere. Indeed. I was wondering if you have access to openvswitch.org http logs, to catch other dead

Re: [ovs-dev] [PATCH] Documentation: Fix links in maintainers.rst.

2023-01-05 Thread Han Zhou
On Thu, Jan 5, 2023 at 11:42 AM Ilya Maximets wrote: > > GitHub and Sphinx are parsing links differently. Sphinx knows about > the overall documentation structure and all the sections defined in > other docs, while GitHub is using direct rst 2 html conversion and > doesn't know any of that.

Re: [ovs-dev] [PATCH] Documentation: Fix links in the DPDK guide on physical ports.

2023-01-05 Thread 0-day Robot
Bleep bloop. Greetings Ilya Maximets, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 94 characters long (recommended limit is 79) #39 FILE:

Re: [ovs-dev] [PATCH] Documentation: use new syntax for dpdk port representors

2022-11-02 Thread Ilya Maximets
On 4/27/22 10:15, Robin Jarry wrote: > Since DPDK 21.05, the representor identifier now handles a relative VF > offset. The legacy representor ID seems only valid in certain cases > (first dpdk port). > > Link: https://github.com/DPDK/dpdk/commit/cebf7f17159a8 > Signed-off-by: Robin Jarry > ---

Re: [ovs-dev] [PATCH] Documentation: Clarify QEMU version requirement.

2022-05-05 Thread Ferriter, Cian
> -Original Message- > From: Ilya Maximets > Sent: Wednesday 4 May 2022 22:31 > To: Ferriter, Cian ; ovs-dev@openvswitch.org > Cc: i.maxim...@ovn.org > Subject: Re: [ovs-dev] [PATCH] Documentation: Clarify QEMU version > requirement. > > On 4/26/22

Re: [ovs-dev] [PATCH] Documentation: Clarify QEMU version requirement.

2022-05-04 Thread Ilya Maximets
On 4/26/22 18:25, Cian Ferriter wrote: > The QEMU version requirement of >= 2.7 is for vhost-user-client ports > specifically. > > Signed-off-by: Cian Ferriter > --- > Documentation/topics/dpdk/vhost-user.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Applied. Thanks! Best

Re: [ovs-dev] [PATCH] Documentation: use new syntax for dpdk port representors

2022-04-27 Thread 0-day Robot
Bleep bloop. Greetings Robin Jarry, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 92 characters long (recommended limit is 79) #26 FILE:

Re: [ovs-dev] [PATCH] Documentation: Update USDT documentation to include systemtap dependency

2022-02-16 Thread Ilya Maximets
On 2/7/22 12:32, Eelco Chaudron wrote: > Update the documentation to include details on SystemTap dependency > when enabling USDT probes. > > Suggested-by: Adrian Moreno > Signed-off-by: Eelco Chaudron > --- > Documentation/topics/usdt-probes.rst |5 + > 1 file changed, 5 insertions(+)

Re: [ovs-dev] [PATCH] Documentation: Update USDT documentation to include systemtap dependency

2022-02-16 Thread Adrian Moreno
On 2/7/22 12:32, Eelco Chaudron wrote: Update the documentation to include details on SystemTap dependency when enabling USDT probes. Suggested-by: Adrian Moreno Signed-off-by: Eelco Chaudron --- Documentation/topics/usdt-probes.rst |5 + 1 file changed, 5 insertions(+) diff

Re: [ovs-dev] [PATCH] Documentation: Update USDT documentation to include systemtap dependency

2022-02-11 Thread Paolo Valerio
Eelco Chaudron writes: > Update the documentation to include details on SystemTap dependency > when enabling USDT probes. > > Suggested-by: Adrian Moreno > Signed-off-by: Eelco Chaudron > --- > Documentation/topics/usdt-probes.rst |5 + > 1 file changed, 5 insertions(+) > > diff --git

Re: [ovs-dev] [PATCH] Documentation: Change the address in userspace-tunneling.rst

2021-09-30 Thread Paolo Valerio
"Ferriter, Cian" writes: >> -Original Message- >> From: dev On Behalf Of Paolo Valerio >> Sent: Wednesday 29 September 2021 21:43 >> To: d...@openvswitch.org >> Cc: f...@redhat.com; i.maxim...@ovn.org >> Subject: [ovs-dev] [PATCH] Documentation: Change the address in >>

Re: [ovs-dev] [PATCH] Documentation: Change the address in userspace-tunneling.rst

2021-09-30 Thread Ferriter, Cian
> -Original Message- > From: dev On Behalf Of Paolo Valerio > Sent: Wednesday 29 September 2021 21:43 > To: d...@openvswitch.org > Cc: f...@redhat.com; i.maxim...@ovn.org > Subject: [ovs-dev] [PATCH] Documentation: Change the address in > userspace-tunneling.rst > > Fixes:

Re: [ovs-dev] [PATCH] Documentation: Cleanup PMD information.

2021-08-10 Thread Kevin Traynor
On 05/08/2021 09:32, Adrian Moreno wrote: > > On 7/22/21 3:25 PM, Kevin Traynor wrote: >> The 'Port/Rx Queue Assigment to PMD Threads' section has >> expanded over time and now includes info about stats/commands, >> manual pinning and different options for OVS assigning Rxqs to >> PMDs. >> >>

Re: [ovs-dev] [PATCH] Documentation: Cleanup PMD information.

2021-08-05 Thread Adrian Moreno
On 7/22/21 3:25 PM, Kevin Traynor wrote: > The 'Port/Rx Queue Assigment to PMD Threads' section has > expanded over time and now includes info about stats/commands, > manual pinning and different options for OVS assigning Rxqs to > PMDs. > > Split them into different sections with sub-headings

Re: [ovs-dev] [PATCH] Documentation: Remove duplicate words.

2021-07-19 Thread Ben Pfaff
On Mon, Jul 19, 2021 at 01:27:16PM +0200, David Marchand wrote: > This is a simple cleanup with a script of mine. > > Signed-off-by: David Marchand > --- > This script is a bit too silly/simplistic and requires manual review > (too many false positives), but I don't think it is worth investing

Re: [ovs-dev] [PATCH] documentation: add missing sudo to intro install

2021-04-08 Thread Ben Pfaff
On Fri, Apr 02, 2021 at 07:34:43PM +, hunchback wrote: > the current documentation assumes everything is run from within root > what is not correct in the general case, the revised install > documentation makes use of `sudo` for specific commands which require to > be run under root

Re: [ovs-dev] [PATCH] documentation: add missing sudo to intro install

2021-04-02 Thread Ben Pfaff
On Fri, Apr 02, 2021 at 07:11:22PM +, hunchback wrote: > the current documentation assumes everything is run from within root > what is not correct in the general case, the revised install > documentation makes use of `sudo` for specific commands which require to > be run under root

Re: [ovs-dev] [PATCH] documentation: add missing sudo to intro install

2021-03-30 Thread Ben Pfaff
On Tue, Mar 30, 2021 at 11:55:11AM +0100, Aidan Shribman wrote: > the current documentation assumes everything is run from within root > what is not correct in the general case, the revised install > documentation makes use of `sudo` for specific commands which require to > be run under root

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-18 Thread William Tu
On Thu, Feb 18, 2021 at 6:55 AM Eelco Chaudron wrote: > > > > On 18 Feb 2021, at 15:40, William Tu wrote: > > > On Thu, Feb 18, 2021 at 1:23 AM Eelco Chaudron > > wrote: > >> > >> > >> > >> On 17 Feb 2021, at 18:41, William Tu wrote: > >> > > >> Information Rate). High priority traffic

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-18 Thread Eelco Chaudron
On 18 Feb 2021, at 15:40, William Tu wrote: On Thu, Feb 18, 2021 at 1:23 AM Eelco Chaudron wrote: On 17 Feb 2021, at 18:41, William Tu wrote: Information Rate). High priority traffic is routed to queue 10, which marks all traffic as CIR, i.e. Green. All low priority traffic, queue

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-18 Thread William Tu
On Thu, Feb 18, 2021 at 1:23 AM Eelco Chaudron wrote: > > > > On 17 Feb 2021, at 18:41, William Tu wrote: > > >> > Information Rate). High priority traffic is routed to queue 10, > which marks > all traffic as CIR, i.e. Green. All low priority traffic, queue 20, > is >

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-18 Thread Eelco Chaudron
On 17 Feb 2021, at 18:41, William Tu wrote: >> Information Rate). High priority traffic is routed to queue 10, which marks all traffic as CIR, i.e. Green. All low priority traffic, queue 20, is marked as EIR, i.e. Yellow:: $ ovs-vsctl --timeout=5 set

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-17 Thread William Tu
> > >> Information Rate). High priority traffic is routed to queue 10, > >> which marks > >> all traffic as CIR, i.e. Green. All low priority traffic, queue 20, > >> is > >> marked as EIR, i.e. Yellow:: > >> > >> $ ovs-vsctl --timeout=5 set port dpdk1 qos=@myqos -- \ > >>

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-17 Thread Eelco Chaudron
On 17 Feb 2021, at 18:17, William Tu wrote: On Wed, Feb 17, 2021 at 8:21 AM Eelco Chaudron wrote: On 17 Feb 2021, at 9:23, Eelco Chaudron wrote: On 17 Feb 2021, at 4:39, William Tu wrote: RFC4115 says "The CIR and EIR are both measured in bits/s." Fix the example use case based on

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-17 Thread William Tu
On Wed, Feb 17, 2021 at 8:21 AM Eelco Chaudron wrote: > > > > On 17 Feb 2021, at 9:23, Eelco Chaudron wrote: > > > On 17 Feb 2021, at 4:39, William Tu wrote: > > > >> RFC4115 says "The CIR and EIR are both measured in bits/s." > >> Fix the example use case based on the decription. > >> 64-Byte

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-17 Thread Eelco Chaudron
On 17 Feb 2021, at 9:23, Eelco Chaudron wrote: On 17 Feb 2021, at 4:39, William Tu wrote: RFC4115 says "The CIR and EIR are both measured in bits/s." Fix the example use case based on the decription. 64-Byte packet * 8 * 1000 pps = 512000 Did you run some tests to verify the changes you

Re: [ovs-dev] [PATCH] Documentation: Fix DPDK qos example.

2021-02-17 Thread Eelco Chaudron
On 17 Feb 2021, at 4:39, William Tu wrote: RFC4115 says "The CIR and EIR are both measured in bits/s." Fix the example use case based on the decription. 64-Byte packet * 8 * 1000 pps = 512000 Did you run some tests to verify the changes you made? Fixes: e61bdffc2a98 ("netdev-dpdk: Add new

Re: [ovs-dev] [PATCH] Documentation: Simplify the website main page.

2021-01-09 Thread Flavio Leitner
On Thu, Jan 07, 2021 at 12:15:10PM -0800, Ben Pfaff wrote: > On Mon, Dec 28, 2020 at 07:02:55PM -0300, Flavio Leitner wrote: > > The initial website page is difficult to read because of > > the large amount of links from different parts of the whole > > documentation. Most of all those links come

Re: [ovs-dev] [PATCH] Documentation: Simplify the website main page.

2021-01-07 Thread Ben Pfaff
On Mon, Dec 28, 2020 at 07:02:55PM -0300, Flavio Leitner wrote: > The initial website page is difficult to read because of > the large amount of links from different parts of the whole > documentation. Most of all those links come from their > index page referenced in the section 'Contents' on the

Re: [ovs-dev] [PATCH] Documentation: Fix rendering of extra repo info for RHEL 8

2020-11-05 Thread Ilya Maximets
On 11/3/20 5:36 PM, Gregory Rose wrote: > > > On 11/3/2020 2:12 AM, Timothy Redaelli wrote: >> In commit a82083ee3091 ("Documentation: Add extra repo info for RHEL 8") >> a newline was missing to correctly generate the code block to add >> codeready-builder repository. >> >> This commit adds the

Re: [ovs-dev] [PATCH] Documentation: Fix rendering of extra repo info for RHEL 8

2020-11-03 Thread Gregory Rose
On 11/3/2020 2:12 AM, Timothy Redaelli wrote: In commit a82083ee3091 ("Documentation: Add extra repo info for RHEL 8") a newline was missing to correctly generate the code block to add codeready-builder repository. This commit adds the missing newline to correctly generate the code block

Re: [ovs-dev] [PATCH] Documentation: Correct claims about Debian packaging.

2020-09-16 Thread Ilya Maximets
On 7/22/19 7:35 PM, Ben Pfaff wrote: > The documentation reported the union of all possible Debian- and > Debian-derived packaging. This isn't realistic: there are differences > between OVS upstream, Debian downstream, and Ubuntu downstream. This > commit distinguishes them. > > Reported-by:

Re: [ovs-dev] [PATCH] Documentation, windows: Document how to generate the Windows installer

2020-08-24 Thread 0-day Robot
Bleep bloop. Greetings Alin Gabriel Serdean, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 80 characters long (recommended limit is 79) #26 FILE:

Re: [ovs-dev] [PATCH] Documentation: Fix kernel support matrix

2020-05-20 Thread William Tu
On Tue, May 19, 2020 at 04:00:07PM -0700, Han Zhou wrote: > On Tue, May 19, 2020 at 3:01 PM Greg Rose wrote: > > > > The documentation matrix for OVS branches and which kernels they support > > is out of date. Update it to show that since 2.10 the lowest kernel > > that we test and support is

Re: [ovs-dev] [PATCH] Documentation: Fix kernel support matrix

2020-05-19 Thread Han Zhou
On Tue, May 19, 2020 at 3:01 PM Greg Rose wrote: > > The documentation matrix for OVS branches and which kernels they support > is out of date. Update it to show that since 2.10 the lowest kernel > that we test and support is Linux 3.16. > > RHEL and CentOS kernels based upon the original 3.10

Re: [ovs-dev] [PATCH] Documentation: Add note about iproute2 requirements for check-kmod

2020-03-19 Thread William Tu
On Wed, Mar 11, 2020 at 10:49:17AM -0700, Greg Rose wrote: > On many systems the check-kmod and check-kernel test suites have > many failures due to the lack of feature support in the older > iproute2 utility packages shipped with those systems. Add a > note indicating that it might be necessary

Re: [ovs-dev] [PATCH] Documentation: Add note about iproute2 requirements for check-kmod

2020-03-19 Thread Gregory Rose
On 3/11/2020 10:52 AM, William Tu wrote: On Wed, Mar 11, 2020 at 10:49 AM Greg Rose wrote: On many systems the check-kmod and check-kernel test suites have many failures due to the lack of feature support in the older iproute2 utility packages shipped with those systems. Add a note

Re: [ovs-dev] [PATCH] Documentation: Add note about iproute2 requirements for check-kmod

2020-03-11 Thread William Tu
On Wed, Mar 11, 2020 at 10:49 AM Greg Rose wrote: > > On many systems the check-kmod and check-kernel test suites have > many failures due to the lack of feature support in the older > iproute2 utility packages shipped with those systems. Add a > note indicating that it might be necessary to

Re: [ovs-dev] [PATCH] Documentation: Fix literal blocks formating

2020-02-13 Thread Flavio Leitner
On Wed, Feb 12, 2020 at 11:46:47AM -0800, Yi-Hung Wei wrote: > Signed-off-by: Yi-Hung Wei > --- Thanks the patch, looks much better with it. Acked-by: Flavio Leitner ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH] Documentation: Fix building with Python 3.9

2020-01-16 Thread Ben Pfaff
On Thu, Jan 16, 2020 at 11:20:01AM -0300, Flavio Leitner wrote: > On Thu, Jan 16, 2020 at 02:21:47PM +0100, Timothy Redaelli wrote: > > open(), io.open(), codecs.open() and fileinput.FileInput no longer accept > > 'U' > > ("universal newline") in the file mode. > > This flag was deprecated since

Re: [ovs-dev] [PATCH] Documentation: Fix building with Python 3.9

2020-01-16 Thread Flavio Leitner
On Thu, Jan 16, 2020 at 02:21:47PM +0100, Timothy Redaelli wrote: > open(), io.open(), codecs.open() and fileinput.FileInput no longer accept 'U' > ("universal newline") in the file mode. > This flag was deprecated since Python 3.3. > In Python 3, the "universal newline" is used by default when a

Re: [ovs-dev] [PATCH] Documentation: add notes for TSO & i40e

2020-01-14 Thread Loftus, Ciara
> -Original Message- > From: David Marchand > Sent: Monday 13 January 2020 15:58 > To: Loftus, Ciara > Cc: ovs dev ; Flavio Leitner ; > Stokes, Ian > Subject: Re: [ovs-dev] [PATCH] Documentation: add notes for TSO & i40e > > On Mon, Jan 13, 2020

Re: [ovs-dev] [PATCH] Documentation: add notes for TSO & i40e

2020-01-13 Thread Flavio Leitner
On Mon, Jan 13, 2020 at 03:48:29PM +, Ciara Loftus wrote: > When using TSO in OVS-DPDK with an i40e device, the following > commit is required for DPDK, which fixes an issue on the TSO path: > http://git.dpdk.org/next/dpdk-next-net/commit/?id=b2a4dc260139409c539fb8e7f1b9d0a5182cfd2b > Document

Re: [ovs-dev] [PATCH] Documentation: add notes for TSO & i40e

2020-01-13 Thread 0-day Robot
Bleep bloop. Greetings Ciara Loftus, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: Failed to merge in the changes. Patch failed at 0001 Documentation: add notes for TSO & i40e The

Re: [ovs-dev] [PATCH] Documentation: add notes for TSO & i40e

2020-01-13 Thread David Marchand
On Mon, Jan 13, 2020 at 4:49 PM Ciara Loftus wrote: > > When using TSO in OVS-DPDK with an i40e device, the following > commit is required for DPDK, which fixes an issue on the TSO path: > http://git.dpdk.org/next/dpdk-next-net/commit/?id=b2a4dc260139409c539fb8e7f1b9d0a5182cfd2b Please avoid

Re: [ovs-dev] [PATCH] Documentation: Fix ovs-tcpdump options.

2019-12-18 Thread Simon Horman
On Mon, Dec 16, 2019 at 09:24:29AM -0800, Gregory Rose wrote: > On 12/16/2019 6:18 AM, William Tu wrote: > > Signed-off-by: William Tu > > --- > > Documentation/ref/ovs-tcpdump.8.rst | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git

Re: [ovs-dev] [PATCH] Documentation: Fix ovs-tcpdump options.

2019-12-16 Thread Gregory Rose
On 12/16/2019 6:18 AM, William Tu wrote: Signed-off-by: William Tu --- Documentation/ref/ovs-tcpdump.8.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/ref/ovs-tcpdump.8.rst b/Documentation/ref/ovs-tcpdump.8.rst index 048b70f23daf..b9f8cdf6f786 100644

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-12-02 Thread Ben Pfaff
On Thu, Nov 28, 2019 at 12:57:16PM +0530, Numan Siddique wrote: > On Thu, Nov 28, 2019 at 12:49 PM Numan Siddique wrote: > > > > On Wed, Nov 27, 2019 at 4:10 AM Ben Pfaff wrote: > > > > > > On Thu, Oct 10, 2019 at 02:29:42PM -0700, Ben Pfaff wrote: > > > > Signed-off-by: Ben Pfaff > > > > > >

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-11-27 Thread Numan Siddique
On Thu, Nov 28, 2019 at 12:49 PM Numan Siddique wrote: > > On Wed, Nov 27, 2019 at 4:10 AM Ben Pfaff wrote: > > > > On Thu, Oct 10, 2019 at 02:29:42PM -0700, Ben Pfaff wrote: > > > Signed-off-by: Ben Pfaff > > > > Still needs review. > > Hi Ben, > > Can you please rebase this patch. It doesn't

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-11-27 Thread Numan Siddique
On Wed, Nov 27, 2019 at 4:10 AM Ben Pfaff wrote: > > On Thu, Oct 10, 2019 at 02:29:42PM -0700, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > > Still needs review. Hi Ben, Can you please rebase this patch. It doesn't apply. Thanks Numan > ___ >

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-11-26 Thread Ben Pfaff
On Thu, Oct 10, 2019 at 02:29:42PM -0700, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff Still needs review. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-10-15 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Oct 10, 2019 at 07:58:02PM -0400, 0-day Robot wrote: >> WARNING: New doc ovs-appctl.8.rst not listed in Documentation/automake.mk >> WARNING: New doc ovs-ctl.8.rst not listed in Documentation/automake.mk >> WARNING: New doc ovs-l3ping.8.rst not listed in

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-10-14 Thread Ben Pfaff
On Thu, Oct 10, 2019 at 07:58:02PM -0400, 0-day Robot wrote: > WARNING: New doc ovs-appctl.8.rst not listed in Documentation/automake.mk > WARNING: New doc ovs-ctl.8.rst not listed in Documentation/automake.mk > WARNING: New doc ovs-l3ping.8.rst not listed in Documentation/automake.mk > WARNING:

Re: [ovs-dev] [PATCH] Documentation: Convert multiple manpages to ReST.

2019-10-10 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 109 characters long (recommended limit is 79) #1018 FILE:

Re: [ovs-dev] [PATCH] Documentation: Document a useful pre-push hook for committers.

2019-10-01 Thread Ben Pfaff
On Tue, Oct 01, 2019 at 01:52:54PM +0200, Simon Horman wrote: > On Wed, Sep 25, 2019 at 01:02:31PM -0700, Ben Pfaff wrote: > > Someone else wrote this script originally, I think, but I've extended > > it quite a bit. > > > > Signed-off-by: Ben Pfaff > > Reviewed-by: Simon Horman Thanks! I

Re: [ovs-dev] [PATCH] Documentation: Document a useful pre-push hook for committers.

2019-10-01 Thread Simon Horman
On Wed, Sep 25, 2019 at 01:02:31PM -0700, Ben Pfaff wrote: > Someone else wrote this script originally, I think, but I've extended > it quite a bit. > > Signed-off-by: Ben Pfaff Reviewed-by: Simon Horman ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH] Documentation: Fix security mailing list address.

2019-09-23 Thread Ben Pfaff
On Mon, Sep 23, 2019 at 02:51:35PM -0700, Gregory Rose wrote: > > On 9/23/2019 12:38 PM, Ben Pfaff wrote: > > We don't own ovs.org, and I doubt Ojai Valley School would enjoy > > receiving our email. > > > > Signed-off-by: Ben Pfaff > > --- > > Documentation/internals/mailing-lists.rst | 2 +-

Re: [ovs-dev] [PATCH] Documentation: Fix security mailing list address.

2019-09-23 Thread Gregory Rose
On 9/23/2019 12:38 PM, Ben Pfaff wrote: We don't own ovs.org, and I doubt Ojai Valley School would enjoy receiving our email. Signed-off-by: Ben Pfaff --- Documentation/internals/mailing-lists.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [ovs-dev] [PATCH] Documentation: Clarify connection tracking tutorial

2019-06-17 Thread Gregory Rose
On 6/14/2019 3:57 PM, Darrell Ball wrote: Hi Greg few comments inline. Thanks for the suggestions Darrell, let me work up a V2 and I'll repost then. - Greg On Fri, Jun 14, 2019 at 1:44 PM Greg Rose > wrote: The current documentation states that "all

Re: [ovs-dev] [PATCH] Documentation: Clarify connection tracking tutorial

2019-06-14 Thread Darrell Ball
Hi Greg few comments inline. On Fri, Jun 14, 2019 at 1:44 PM Greg Rose wrote: > The current documentation states that "all packets entering OVS for > the first time are "untracked"". However there is a minor exception > to this in the case where a packet (re)enters the same datapath and > the

Re: [ovs-dev] [PATCH] Documentation: Update documentation for OpenFlow support.

2019-04-24 Thread Ben Pfaff
On Wed, Apr 24, 2019 at 03:13:22PM -0700, Yi-Hung Wei wrote: > On Wed, Apr 24, 2019 at 9:42 AM Ben Pfaff wrote: > > > > The commits that implemented these features forgot to update the > > documentation. > > > > Signed-off-by: Ben Pfaff > > --- > > Documentation/topics/openflow.rst | 26

Re: [ovs-dev] [PATCH] Documentation: Update documentation for OpenFlow support.

2019-04-24 Thread Yi-Hung Wei
On Wed, Apr 24, 2019 at 9:42 AM Ben Pfaff wrote: > > The commits that implemented these features forgot to update the > documentation. > > Signed-off-by: Ben Pfaff > --- > Documentation/topics/openflow.rst | 26 +- > 1 file changed, 1 insertion(+), 25 deletions(-) > >

Re: [ovs-dev] [PATCH] documentation: man vswitchd.conf.db(5) updated flow-restore-wait

2018-11-02 Thread Ben Pfaff
On Fri, Nov 02, 2018 at 03:25:29PM -0700, Zak Whittington wrote: > Commit 7ed73428a changed the behavior of flow-restore-wait to > also prevent the switch from connecting to controllers in the > controller table, but failed to update the man page documentation > generated by vswitchd/vswitch.xml

Re: [ovs-dev] [PATCH] Documentation:

2018-07-18 Thread Aaron Conole
Gregory Rose writes: > On 7/18/2018 8:03 AM, Aaron Conole wrote: >> Hi Greg, >> >> Greg Rose writes: >> >>> Add netstat when mentioning testing. Many check-kmod failures result >>> when it is not present. >>> >>> Signed-off-by: Greg Rose >>> --- >> Just wanted to point out that the subject

Re: [ovs-dev] [PATCH] Documentation:

2018-07-18 Thread Gregory Rose
On 7/18/2018 8:03 AM, Aaron Conole wrote: Hi Greg, Greg Rose writes: Add netstat when mentioning testing. Many check-kmod failures result when it is not present. Signed-off-by: Greg Rose --- Just wanted to point out that the subject looks incomplete. Otherwise: Acked-by: Aaron Conole

Re: [ovs-dev] [PATCH] Documentation:

2018-07-18 Thread Aaron Conole
Hi Greg, Greg Rose writes: > Add netstat when mentioning testing. Many check-kmod failures result > when it is not present. > > Signed-off-by: Greg Rose > --- Just wanted to point out that the subject looks incomplete. Otherwise: Acked-by: Aaron Conole

Re: [ovs-dev] [PATCH] Documentation:

2018-07-13 Thread Gregory Rose
On 7/13/2018 3:44 PM, Greg Rose wrote: Add netstat when mentioning testing. Many check-kmod failures result when it is not present. Signed-off-by: Greg Rose I fat fingered the title - should be "Documentation: Add netstat to testing I can send V2 or it can be fixed on push, either way

Re: [ovs-dev] [PATCH] Documentation: Fix binary representation of 80 and 8080 in Faucet tutorial

2018-04-27 Thread axel
On 2018-04-26 15:39, a...@tripier.fr wrote: > The binary representation of 80 and 8080 are switched in the Faucet tutorial. > > Signed-off-by: Axel Tripier > --- > Documentation/tutorials/faucet.rst | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Please disregard

Re: [ovs-dev] [PATCH] Documentation: Fix commands not being shown in code blocks

2018-04-27 Thread axel
On 2018-04-27 16:12, a...@tripier.fr wrote: > Some commands are not shown in code blocks in the Advances Features > tutorial, they are shown as variable width text because of a missing ":" > to designate them as code blocks. > > Signed-off-by: Axel Tripier > --- >

Re: [ovs-dev] [PATCH] Documentation: Add note about dpdkvhostuser and IOMMU.

2018-03-06 Thread Stokes, Ian
> -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Kevin Traynor > Sent: Tuesday, March 6, 2018 12:07 PM > To: d...@openvswitch.org > Cc: Maxime Coquelin > Subject: [ovs-dev] [PATCH]

Re: [ovs-dev] [PATCH] Documentation: Document optional RHEL7 repositories

2018-01-17 Thread Gregory Rose
On 1/17/2018 4:15 PM, Flavio Leitner wrote: On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote: On minimal install RHEL 7 servers (and perhaps other types of installs) you need to enable a couple of optional repositories for the yum-builddep utility to work correctly. This patch

Re: [ovs-dev] [PATCH] Documentation: Document optional RHEL7 repositories

2018-01-17 Thread Flavio Leitner
On Wed, Jan 17, 2018 at 10:15:31PM -0200, Flavio Leitner wrote: > On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote: > > On minimal install RHEL 7 servers (and perhaps other types of installs) > > you need to enable a couple of optional repositories for the yum-builddep > > utility to work

Re: [ovs-dev] [PATCH] Documentation: Document optional RHEL7 repositories

2018-01-17 Thread Flavio Leitner
On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote: > On minimal install RHEL 7 servers (and perhaps other types of installs) > you need to enable a couple of optional repositories for the yum-builddep > utility to work correctly. This patch documents those two optional > repositories. >

  1   2   >