Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-06-03 Thread Ilya Maximets
On 5/27/22 14:45, Van Haaren, Harry wrote: >> -Original Message- >> From: Ilya Maximets >> Sent: Friday, May 27, 2022 1:10 PM >> To: Van Haaren, Harry ; Eelco Chaudron >> >> Cc: i.maxim...@ovn.org; ovs-dev@openvswitch.org; Stokes, Ian >> ; Amber, Kumar >> Subject: Re: [PATCH] dpcls:

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-27 Thread Van Haaren, Harry
> -Original Message- > From: Ilya Maximets > Sent: Friday, May 27, 2022 1:10 PM > To: Van Haaren, Harry ; Eelco Chaudron > > Cc: i.maxim...@ovn.org; ovs-dev@openvswitch.org; Stokes, Ian > ; Amber, Kumar > Subject: Re: [PATCH] dpcls: revert subtable-lookup-prio-get name change > > On

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-27 Thread Ilya Maximets
On 5/27/22 14:09, Ilya Maximets wrote: > On 5/25/22 18:15, Van Haaren, Harry wrote: >>> -Original Message- >>> From: Ilya Maximets >>> Sent: Wednesday, May 25, 2022 4:33 PM >>> To: Eelco Chaudron ; Van Haaren, Harry >>> >>> Cc: i.maxim...@ovn.org; ovs-dev@openvswitch.org; Stokes, Ian >>>

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-27 Thread Ilya Maximets
On 5/25/22 18:15, Van Haaren, Harry wrote: >> -Original Message- >> From: Ilya Maximets >> Sent: Wednesday, May 25, 2022 4:33 PM >> To: Eelco Chaudron ; Van Haaren, Harry >> >> Cc: i.maxim...@ovn.org; ovs-dev@openvswitch.org; Stokes, Ian >> ; Amber, Kumar >> Subject: Re: [PATCH] dpcls:

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Van Haaren, Harry
> -Original Message- > From: Ilya Maximets > Sent: Wednesday, May 25, 2022 4:33 PM > To: Eelco Chaudron ; Van Haaren, Harry > > Cc: i.maxim...@ovn.org; ovs-dev@openvswitch.org; Stokes, Ian > ; Amber, Kumar > Subject: Re: [PATCH] dpcls: revert subtable-lookup-prio-get name change > > On

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Ilya Maximets
On 5/25/22 16:32, Eelco Chaudron wrote: > > > On 25 May 2022, at 16:10, Harry van Haaren wrote: > >> This commit reverts the name-change that was done (prio->info). >> The change breaks a user visible ovs-appctl command, resulting in >> breakage of tools/scripts/user-expectation outside of the

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Stokes, Ian
> On 25 May 2022, at 16:35, Stokes, Ian wrote: > > >> This commit reverts the name-change that was done (prio->info). > >> The change breaks a user visible ovs-appctl command, resulting in > >> breakage of tools/scripts/user-expectation outside of the OVS repo. > >> > >> This commit changes the

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Eelco Chaudron
On 25 May 2022, at 16:35, Stokes, Ian wrote: >> This commit reverts the name-change that was done (prio->info). >> The change breaks a user visible ovs-appctl command, resulting in >> breakage of tools/scripts/user-expectation outside of the OVS repo. >> >> This commit changes the

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Stokes, Ian
> This commit reverts the name-change that was done (prio->info). > The change breaks a user visible ovs-appctl command, resulting in > breakage of tools/scripts/user-expectation outside of the OVS repo. > > This commit changes the documentation, command string, and unit tests > back to the

Re: [ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Eelco Chaudron
On 25 May 2022, at 16:10, Harry van Haaren wrote: > This commit reverts the name-change that was done (prio->info). > The change breaks a user visible ovs-appctl command, resulting in > breakage of tools/scripts/user-expectation outside of the OVS repo. > > This commit changes the

[ovs-dev] [PATCH] dpcls: revert subtable-lookup-prio-get name change

2022-05-25 Thread Harry van Haaren
This commit reverts the name-change that was done (prio->info). The change breaks a user visible ovs-appctl command, resulting in breakage of tools/scripts/user-expectation outside of the OVS repo. This commit changes the documentation, command string, and unit tests back to the expected "prio"