diff mbox series

[ovs-dev] northd: Fix "Destination unknown" table number in man pages.

Message ID 20210405155331.399752-1-aroytman@gmail.com
State Accepted
Headers show
Series [ovs-dev] northd: Fix "Destination unknown" table number in man pages. | expand

Commit Message

Alexey Roytman April 5, 2021, 3:53 p.m. UTC
From: Alexey Roytman <roytman@il.ibm.com>

The actual number of the "Destination unknown" table, is 24 and is not 23
as mentioned.

Signed-off-by: Alexey Roytman <roytman@il.ibm.com>
---
 northd/ovn-northd.8.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Gray April 8, 2021, 9:54 a.m. UTC | #1
On 05/04/2021 16:53, Alexey Roytman wrote:
> From: Alexey Roytman <roytman@il.ibm.com>
> 
> The actual number of the "Destination unknown" table, is 24 and is not 23
> as mentioned.
> 
> Signed-off-by: Alexey Roytman <roytman@il.ibm.com>
> ---
>  northd/ovn-northd.8.xml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml
> index a62f5c057..e42a2249e 100644
> --- a/northd/ovn-northd.8.xml
> +++ b/northd/ovn-northd.8.xml
> @@ -1619,7 +1619,7 @@ output;
>        </li>
>      </ul>
>  
> -    <h3>Ingress Table 23 Destination unknown</h3>
> +    <h3>Ingress Table 24 Destination unknown</h3>
>  
>      <p>
>        This table handles the packets whose destination was not found or
> 

Quick look in the sandbox confirms this

$ ovn-sbctl lflow-list | grep table=24
  table=24(ls_in_l2_unknown   ), priority=50   , match=(outport ==
"none"), action=(drop;)
  table=24(ls_in_l2_unknown   ), priority=0    , match=(1), action=(output;)
  table=24(ls_in_l2_unknown   ), priority=50   , match=(outport ==
"none"), action=(drop;)
  table=24(ls_in_l2_unknown   ), priority=0    , match=(1), action=(output;)


Acked-by: Mark D. Gray <mark.d.gray@redhat.com>
Numan Siddique April 19, 2021, 2:43 p.m. UTC | #2
On Thu, Apr 8, 2021 at 5:54 AM Mark Gray <mark.d.gray@redhat.com> wrote:
>
> On 05/04/2021 16:53, Alexey Roytman wrote:
> > From: Alexey Roytman <roytman@il.ibm.com>
> >
> > The actual number of the "Destination unknown" table, is 24 and is not 23
> > as mentioned.
> >
> > Signed-off-by: Alexey Roytman <roytman@il.ibm.com>
> > ---
> >  northd/ovn-northd.8.xml | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml
> > index a62f5c057..e42a2249e 100644
> > --- a/northd/ovn-northd.8.xml
> > +++ b/northd/ovn-northd.8.xml
> > @@ -1619,7 +1619,7 @@ output;
> >        </li>
> >      </ul>
> >
> > -    <h3>Ingress Table 23 Destination unknown</h3>
> > +    <h3>Ingress Table 24 Destination unknown</h3>
> >
> >      <p>
> >        This table handles the packets whose destination was not found or
> >
>
> Quick look in the sandbox confirms this
>
> $ ovn-sbctl lflow-list | grep table=24
>   table=24(ls_in_l2_unknown   ), priority=50   , match=(outport ==
> "none"), action=(drop;)
>   table=24(ls_in_l2_unknown   ), priority=0    , match=(1), action=(output;)
>   table=24(ls_in_l2_unknown   ), priority=50   , match=(outport ==
> "none"), action=(drop;)
>   table=24(ls_in_l2_unknown   ), priority=0    , match=(1), action=(output;)
>
>
> Acked-by: Mark D. Gray <mark.d.gray@redhat.com>

Thanks. I applied this patch to the main branch.

Numan

>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
diff mbox series

Patch

diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml
index a62f5c057..e42a2249e 100644
--- a/northd/ovn-northd.8.xml
+++ b/northd/ovn-northd.8.xml
@@ -1619,7 +1619,7 @@  output;
       </li>
     </ul>
 
-    <h3>Ingress Table 23 Destination unknown</h3>
+    <h3>Ingress Table 24 Destination unknown</h3>
 
     <p>
       This table handles the packets whose destination was not found or