Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-30 Thread David Marchand
On Sun, Jan 29, 2023 at 11:34 AM Simon Horman wrote: > > Probably not important, but: > > Fixes: b020a416e24c ("System Tests: Enhance NAT tests.") > > > Signed-off-by: Ilya Maximets > > > > Reviewed-by: David Marchand > > >>> > > >>> FWIIW, > > >>> > > >>> Reviewed-by:

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-29 Thread Simon Horman
On Fri, Jan 27, 2023 at 05:52:41PM +0100, Ilya Maximets wrote: > On 1/25/23 13:50, Simon Horman wrote: > > On Tue, Jan 24, 2023 at 05:15:53PM +0100, David Marchand wrote: > >> On Mon, Jan 23, 2023 at 3:50 PM Simon Horman > >> wrote: > >>> > >>> On Mon, Jan 23, 2023 at 03:29:50PM +0100, David

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-27 Thread Ilya Maximets
On 1/25/23 13:50, Simon Horman wrote: > On Tue, Jan 24, 2023 at 05:15:53PM +0100, David Marchand wrote: >> On Mon, Jan 23, 2023 at 3:50 PM Simon Horman >> wrote: >>> >>> On Mon, Jan 23, 2023 at 03:29:50PM +0100, David Marchand wrote: On Mon, Jan 23, 2023 at 3:05 PM Ilya Maximets wrote:

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-25 Thread Simon Horman
On Tue, Jan 24, 2023 at 05:15:53PM +0100, David Marchand wrote: > On Mon, Jan 23, 2023 at 3:50 PM Simon Horman > wrote: > > > > On Mon, Jan 23, 2023 at 03:29:50PM +0100, David Marchand wrote: > > > On Mon, Jan 23, 2023 at 3:05 PM Ilya Maximets wrote: > > > > > > > > Test fails is 'nc' is not

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-24 Thread David Marchand
On Mon, Jan 23, 2023 at 3:50 PM Simon Horman wrote: > > On Mon, Jan 23, 2023 at 03:29:50PM +0100, David Marchand wrote: > > On Mon, Jan 23, 2023 at 3:05 PM Ilya Maximets wrote: > > > > > > Test fails is 'nc' is not available, it should be skipped instead. > > > > > > > Probably not important,

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-23 Thread Simon Horman
On Mon, Jan 23, 2023 at 03:29:50PM +0100, David Marchand wrote: > On Mon, Jan 23, 2023 at 3:05 PM Ilya Maximets wrote: > > > > Test fails is 'nc' is not available, it should be skipped instead. > > > > Probably not important, but: > Fixes: b020a416e24c ("System Tests: Enhance NAT tests.") > >

Re: [ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-23 Thread David Marchand
On Mon, Jan 23, 2023 at 3:05 PM Ilya Maximets wrote: > > Test fails is 'nc' is not available, it should be skipped instead. > Probably not important, but: Fixes: b020a416e24c ("System Tests: Enhance NAT tests.") > Signed-off-by: Ilya Maximets Reviewed-by: David Marchand Some notes: - in

[ovs-dev] [PATCH] system-traffic.at: Skip the 'ICMP6 Related' test if nc is missing.

2023-01-23 Thread Ilya Maximets
Test fails is 'nc' is not available, it should be skipped instead. Signed-off-by: Ilya Maximets --- tests/system-traffic.at | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/system-traffic.at b/tests/system-traffic.at index 503455cc6..fa605d16d 100644 --- a/tests/system-traffic.at +++