Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-03-25 Thread Chris Mi via dev
On 6/24/2023 4:18 AM, Ilya Maximets wrote: On 6/19/23 07:05, Chris Mi wrote: Add three sFlow offload test cases: 3: offloads - sflow with sampling=1 - offloads enabled ok 4: offloads - sflow with sampling=2 - offloads enabled ok 5: offloads - ping over vxlan tunnel with sflow -

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-29 Thread Chris Mi via dev
On 2/29/2024 9:04 PM, Ilya Maximets wrote: On 2/29/24 01:56, Chris Mi wrote: On 2/28/2024 8:51 PM, Ilya Maximets wrote: On 2/28/24 13:26, Chris Mi wrote: Hi Ilya, On 2/28/2024 7:56 PM, Ilya Maximets wrote: On 2/28/24 11:03, Eelco Chaudron wrote: On 28 Feb 2024, at 8:06, Chris Mi wrote:

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-29 Thread Ilya Maximets
On 2/29/24 01:56, Chris Mi wrote: > On 2/28/2024 8:51 PM, Ilya Maximets wrote: >> On 2/28/24 13:26, Chris Mi wrote: >>> Hi Ilya, >>> >>> On 2/28/2024 7:56 PM, Ilya Maximets wrote: On 2/28/24 11:03, Eelco Chaudron wrote: > > > On 28 Feb 2024, at 8:06, Chris Mi wrote: > >>

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Chris Mi via dev
On 2/28/2024 8:51 PM, Ilya Maximets wrote: On 2/28/24 13:26, Chris Mi wrote: Hi Ilya, On 2/28/2024 7:56 PM, Ilya Maximets wrote: On 2/28/24 11:03, Eelco Chaudron wrote: On 28 Feb 2024, at 8:06, Chris Mi wrote: Hi Eelco, I did a rebase and pushed to github. Nothing changed, but the

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Ilya Maximets
On 2/28/24 13:26, Chris Mi wrote: > Hi Ilya, > > On 2/28/2024 7:56 PM, Ilya Maximets wrote: >> On 2/28/24 11:03, Eelco Chaudron wrote: >>> >>> >>> On 28 Feb 2024, at 8:06, Chris Mi wrote: >>> Hi Eelco, I did a rebase and pushed to github. Nothing changed, but the actions

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Chris Mi via dev
Hi Ilya, On 2/28/2024 7:56 PM, Ilya Maximets wrote: On 2/28/24 11:03, Eelco Chaudron wrote: On 28 Feb 2024, at 8:06, Chris Mi wrote: Hi Eelco, I did a rebase and pushed to github. Nothing changed, but the actions always reported the following failure: -1000 packets transmitted, 1000

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Ilya Maximets
On 2/28/24 11:03, Eelco Chaudron wrote: > > > On 28 Feb 2024, at 8:06, Chris Mi wrote: > >> Hi Eelco, >> >> I did a rebase and pushed to github. Nothing changed, but the actions always >> reported the following failure: >> >> -1000 packets transmitted, 1000 received, 0% packet loss, time 0ms

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Eelco Chaudron
On 28 Feb 2024, at 8:06, Chris Mi wrote: > Hi Eelco, > > I did a rebase and pushed to github. Nothing changed, but the actions always > reported the following failure: > > -1000 packets transmitted, 1000 received, 0% packet loss, time 0ms > +744 packets transmitted, 744 received, 0% packet

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-28 Thread Eelco Chaudron
On 28 Feb 2024, at 4:38, Chris Mi wrote: > On 2/28/2024 9:20 AM, Chris Mi via dev wrote: >> On 2/27/2024 6:54 PM, Eelco Chaudron wrote: >>> >>> >>> On 9 Jan 2024, at 11:40, Eelco Chaudron wrote: >>> On 9 Jan 2024, at 10:22, Chris Mi wrote: > Hi Roi and Eelco, > > Sorry for

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-27 Thread Chris Mi via dev
Hi Eelco, I did a rebase and pushed to github. Nothing changed, but the actions always reported the following failure: -1000 packets transmitted, 1000 received, 0% packet loss, time 0ms +744 packets transmitted, 744 received, 0% packet loss, time 0ms I can't reproduce the failure locally.

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-27 Thread Chris Mi via dev
On 2/28/2024 9:20 AM, Chris Mi via dev wrote: On 2/27/2024 6:54 PM, Eelco Chaudron wrote: On 9 Jan 2024, at 11:40, Eelco Chaudron wrote: On 9 Jan 2024, at 10:22, Chris Mi wrote: Hi Roi and Eelco, Sorry for the late reply. We are still busy with other task. So maybe we'll have to

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-27 Thread Chris Mi via dev
On 2/27/2024 6:54 PM, Eelco Chaudron wrote: On 9 Jan 2024, at 11:40, Eelco Chaudron wrote: On 9 Jan 2024, at 10:22, Chris Mi wrote: Hi Roi and Eelco, Sorry for the late reply. We are still busy with other task. So maybe we'll have to postpone it. Thanks for the update and I guess we

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-02-27 Thread Eelco Chaudron
On 9 Jan 2024, at 11:40, Eelco Chaudron wrote: > On 9 Jan 2024, at 10:22, Chris Mi wrote: > >> Hi Roi and Eelco, >> >> Sorry for the late reply. We are still busy with other task. So maybe we'll >> have to postpone it. > > Thanks for the update and I guess we are too late for the upcoming

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-01-09 Thread Eelco Chaudron
On 9 Jan 2024, at 10:22, Chris Mi wrote: > Hi Roi and Eelco, > > Sorry for the late reply. We are still busy with other task. So maybe we'll > have to postpone it. Thanks for the update and I guess we are too late for the upcoming release unless you can submit it this week. //Eelco FYI,

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-01-09 Thread Chris Mi via dev
Hi Roi and Eelco, Sorry for the late reply. We are still busy with other task. So maybe we'll have to postpone it. Regards, Chris On 1/9/2024 4:52 PM, Roi Dayan wrote: On 02/11/2023 15:24, Eelco Chaudron wrote: On 3 Oct 2023, at 17:03, Eelco Chaudron wrote: On 19 Jun 2023, at 12:18,

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2024-01-09 Thread Roi Dayan via dev
On 02/11/2023 15:24, Eelco Chaudron wrote: > > > On 3 Oct 2023, at 17:03, Eelco Chaudron wrote: > >> On 19 Jun 2023, at 12:18, Chris Mi wrote: >> >>> On 6/19/2023 6:04 PM, Eelco Chaudron wrote: On 19 Jun 2023, at 7:05, Chris Mi wrote: > Add three sFlow offload test cases:

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-11-02 Thread Eelco Chaudron
On 3 Oct 2023, at 17:03, Eelco Chaudron wrote: > On 19 Jun 2023, at 12:18, Chris Mi wrote: > >> On 6/19/2023 6:04 PM, Eelco Chaudron wrote: >>> >>> On 19 Jun 2023, at 7:05, Chris Mi wrote: >>> Add three sFlow offload test cases: 3: offloads - sflow with sampling=1 - offloads

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-10-03 Thread Eelco Chaudron
On 19 Jun 2023, at 12:18, Chris Mi wrote: > On 6/19/2023 6:04 PM, Eelco Chaudron wrote: >> >> On 19 Jun 2023, at 7:05, Chris Mi wrote: >> >>> Add three sFlow offload test cases: >>> >>>3: offloads - sflow with sampling=1 - offloads enabled ok >>>4: offloads - sflow with sampling=2 -

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-06-23 Thread Ilya Maximets
On 6/19/23 07:05, Chris Mi wrote: > Add three sFlow offload test cases: > > 3: offloads - sflow with sampling=1 - offloads enabled ok > 4: offloads - sflow with sampling=2 - offloads enabled ok > 5: offloads - ping over vxlan tunnel with sflow - offloads enabled ok > > Signed-off-by: Chris

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-06-19 Thread Chris Mi via dev
On 6/19/2023 6:04 PM, Eelco Chaudron wrote: On 19 Jun 2023, at 7:05, Chris Mi wrote: Add three sFlow offload test cases: 3: offloads - sflow with sampling=1 - offloads enabled ok 4: offloads - sflow with sampling=2 - offloads enabled ok 5: offloads - ping over vxlan tunnel with

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-06-19 Thread Eelco Chaudron
On 19 Jun 2023, at 7:05, Chris Mi wrote: > Add three sFlow offload test cases: > > 3: offloads - sflow with sampling=1 - offloads enabled ok > 4: offloads - sflow with sampling=2 - offloads enabled ok > 5: offloads - ping over vxlan tunnel with sflow - offloads enabled ok > >

Re: [ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-06-18 Thread 0-day Robot
Bleep bloop. Greetings Chris Mi, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 111 characters long (recommended limit is 79) #39 FILE:

[ovs-dev] [PATCH v28 8/8] system-offloads-traffic.at: Add sFlow offload test cases

2023-06-18 Thread Chris Mi via dev
Add three sFlow offload test cases: 3: offloads - sflow with sampling=1 - offloads enabled ok 4: offloads - sflow with sampling=2 - offloads enabled ok 5: offloads - ping over vxlan tunnel with sflow - offloads enabled ok Signed-off-by: Chris Mi Reviewed-by: Roi Dayan Acked-by: Eelco