Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.
On 6/6/24 19:53, Adrián Moreno wrote:
> On Thu, Jun 06, 2024 at 06:00:26PM GMT, Ilya Maximets wrote:
>> On 6/6/24 17:15, Adrian Moreno wrote:
>>> Add check_pkt_len action to the decoder list that it, itself, uses.
>>>
>>> This makes nested check_pkt_len (i.e:a check_pkt_len inside another)
>>> work.
>>>
>>> Signed-off-by: Adrian Moreno
>>> ---
>>> python/ovs/flow/odp.py | 43 ++--
>>> python/ovs/tests/test_odp.py | 29
>>> 2 files changed, 51 insertions(+), 21 deletions(-)
>>
>> Hi, Adrian.
>>
>> Could you, please, provide a Fixes tag for this?
>> No need to send v2 just for this, just reply with it to this thread.
>> (Tags should start from the beginning of the line for patchwork to
>> recognize them.)
>>
>
> Sure, how about this:
>
> Reported-by: Ilya Maximets
> Fixes: 076663b31edc ("python: Add ovs datapath flow parsing.")
>
> Thanks
> Adrián
>
Thanks, Adrian and Eelco!
Applied and backported down to 3.0.
Best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.
On 6 Jun 2024, at 17:15, Adrian Moreno wrote: > Add check_pkt_len action to the decoder list that it, itself, uses. > > This makes nested check_pkt_len (i.e:a check_pkt_len inside another) > work. > > Signed-off-by: Adrian Moreno Thanks for fixing this, changes look good to me. Guess only a fixes tag is missing as Ilya already mentioned. Acked-by: Eelco Chaudron ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.
On Thu, Jun 06, 2024 at 06:00:26PM GMT, Ilya Maximets wrote:
> On 6/6/24 17:15, Adrian Moreno wrote:
> > Add check_pkt_len action to the decoder list that it, itself, uses.
> >
> > This makes nested check_pkt_len (i.e:a check_pkt_len inside another)
> > work.
> >
> > Signed-off-by: Adrian Moreno
> > ---
> > python/ovs/flow/odp.py | 43 ++--
> > python/ovs/tests/test_odp.py | 29
> > 2 files changed, 51 insertions(+), 21 deletions(-)
>
> Hi, Adrian.
>
> Could you, please, provide a Fixes tag for this?
> No need to send v2 just for this, just reply with it to this thread.
> (Tags should start from the beginning of the line for patchwork to
> recognize them.)
>
Sure, how about this:
Reported-by: Ilya Maximets
Fixes: 076663b31edc ("python: Add ovs datapath flow parsing.")
Thanks
Adrián
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] [PATCH] python: ovs: flow: Fix nested check_pkt_len acts.
On 6/6/24 17:15, Adrian Moreno wrote: > Add check_pkt_len action to the decoder list that it, itself, uses. > > This makes nested check_pkt_len (i.e:a check_pkt_len inside another) > work. > > Signed-off-by: Adrian Moreno > --- > python/ovs/flow/odp.py | 43 ++-- > python/ovs/tests/test_odp.py | 29 > 2 files changed, 51 insertions(+), 21 deletions(-) Hi, Adrian. Could you, please, provide a Fixes tag for this? No need to send v2 just for this, just reply with it to this thread. (Tags should start from the beginning of the line for patchwork to recognize them.) Best regards, Ilya Maximets. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
