Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-25 Thread Ilya Maximets
On 8/26/23 00:51, Antonin Bas wrote: > Hi Ilya, > > Thanks for applying the patch. > I just finished testing / investigating output(port=controller,max_len=X), > and there is no need to patch the documentation, for the following reasons: > > The output(port=controller,max_len=X) action is reject

Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-25 Thread Antonin Bas
Hi Ilya, Thanks for applying the patch. I just finished testing / investigating output(port=controller,max_len=X), and there is no need to patch the documentation, for the following reasons: The output(port=controller,max_len=X) action is rejected by OVS: root@ovs-test-node-1:/home/vagrant# ovs-o

Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-25 Thread Ilya Maximets
On 8/25/23 18:38, Antonin Bas wrote: > Hi Ilya, > > Do you think we can merge this patch first as it is specific to the > controller action? > I will check the behavior of output(port=controller,max_len=X) and once I > have confirmation, I can submit a new patch. OK. I applied the change now.

Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-25 Thread Antonin Bas
Hi Ilya, Do you think we can merge this patch first as it is specific to the controller action? I will check the behavior of output(port=controller,max_len=X) and once I have confirmation, I can submit a new patch. Thanks, Antonin Le ven. 25 août 2023 à 07:10, Ilya Maximets a écrit : > On 8/1

Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-25 Thread Ilya Maximets
On 8/17/23 02:30, Antonin Bas wrote: > From: Antonin Bas > > From: Antonin Bas > > Since Open vSwitch 2.7, the max_len option has no effect, and the full > packet is always sent to controllers. This was confirmed with both the > kernel and netdev datapaths. Hi, Antonin. Thanks for the patch!

Re: [ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-16 Thread Simon Horman
On Wed, Aug 16, 2023 at 05:30:59PM -0700, Antonin Bas wrote: > From: Antonin Bas > > From: Antonin Bas > > Since Open vSwitch 2.7, the max_len option has no effect, and the full > packet is always sent to controllers. This was confirmed with both the > kernel and netdev datapaths. > > Reported

[ovs-dev] [PATCH v2] doc: Fix description of max_len for controller action.

2023-08-16 Thread Antonin Bas
From: Antonin Bas From: Antonin Bas Since Open vSwitch 2.7, the max_len option has no effect, and the full packet is always sent to controllers. This was confirmed with both the kernel and netdev datapaths. Reported-by: Antonin Bas Reported-at: https://github.com/openvswitch/ovs-issues/issues