Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-09 Thread Eelco Chaudron
On 9 Oct 2023, at 13:40, Roi Dayan wrote: > On 09/10/2023 12:56, Eelco Chaudron wrote: >> >> >> On 8 Oct 2023, at 9:08, Roi Dayan wrote: >> >>> On 05/10/2023 15:36, Eelco Chaudron wrote: On 4 Oct 2023, at 12:09, Roi Dayan via dev wrote: > Test geneve options mirror flow

Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-09 Thread Roi Dayan via dev
On 09/10/2023 12:56, Eelco Chaudron wrote: > > > On 8 Oct 2023, at 9:08, Roi Dayan wrote: > >> On 05/10/2023 15:36, Eelco Chaudron wrote: >>> >>> >>> On 4 Oct 2023, at 12:09, Roi Dayan via dev wrote: >>> Test geneve options mirror flow doesn't add redundant mirror. Signed-off-by

Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-09 Thread Eelco Chaudron
On 8 Oct 2023, at 9:08, Roi Dayan wrote: > On 05/10/2023 15:36, Eelco Chaudron wrote: >> >> >> On 4 Oct 2023, at 12:09, Roi Dayan via dev wrote: >> >>> Test geneve options mirror flow doesn't add redundant mirror. >>> >>> Signed-off-by: Roi Dayan >> >> In general this patch looks good, some sm

Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-08 Thread Roi Dayan via dev
On 05/10/2023 15:36, Eelco Chaudron wrote: > > > On 4 Oct 2023, at 12:09, Roi Dayan via dev wrote: > >> Test geneve options mirror flow doesn't add redundant mirror. >> >> Signed-off-by: Roi Dayan > > In general this patch looks good, some small nit below. > > //Eelco > >> --- >> >> Notes:

Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-05 Thread Eelco Chaudron
On 4 Oct 2023, at 12:09, Roi Dayan via dev wrote: > Test geneve options mirror flow doesn't add redundant mirror. > > Signed-off-by: Roi Dayan In general this patch looks good, some small nit below. //Eelco > --- > > Notes: > v2: > - add dot in title. > > tests/tunnel.at | 29 +

Re: [ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-05 Thread Simon Horman
On Wed, Oct 04, 2023 at 01:09:43PM +0300, Roi Dayan wrote: > Test geneve options mirror flow doesn't add redundant mirror. > > Signed-off-by: Roi Dayan Acked-by: Simon Horman ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mail

[ovs-dev] [PATCH v2 2/3] tests/tunnel.at: Add geneve options mirror test.

2023-10-04 Thread Roi Dayan via dev
Test geneve options mirror flow doesn't add redundant mirror. Signed-off-by: Roi Dayan --- Notes: v2: - add dot in title. tests/tunnel.at | 29 + 1 file changed, 29 insertions(+) diff --git a/tests/tunnel.at b/tests/tunnel.at index ddeb66bc9fb7..c0b93211044