Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Adrian Moreno
On 9/22/22 16:11, Adrian Moreno wrote: On 9/22/22 13:15, Eelco Chaudron wrote: On 22 Sep 2022, at 12:03, Adrian Moreno wrote: I'm not a fan of this.  Any updates to these structs needs to be mirrored across the codebase.  I think it might be better to do something else. I agree with

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Adrian Moreno
On 9/22/22 13:15, Eelco Chaudron wrote: On 22 Sep 2022, at 12:03, Adrian Moreno wrote: I'm not a fan of this. Any updates to these structs needs to be mirrored across the codebase. I think it might be better to do something else. I agree with this, but for keeping things simple and

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Eelco Chaudron
On 22 Sep 2022, at 13:15, Adrian Moreno wrote: > On 9/21/22 11:17, Eelco Chaudron wrote: >> >> >> On 20 Sep 2022, at 20:52, Aaron Conole wrote: >> >>> Eelco Chaudron writes: >>> This patch adds a Python script that can be used to analyze the revalidator runs by providing statistics

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Eelco Chaudron
On 22 Sep 2022, at 12:03, Adrian Moreno wrote: >>> I'm not a fan of this. Any updates to these structs needs to be >>> mirrored across the codebase. I think it might be better to do >>> something else. >> >> I agree with this, but for keeping things simple and not replicating this in >>

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Adrian Moreno
On 9/21/22 11:17, Eelco Chaudron wrote: On 20 Sep 2022, at 20:52, Aaron Conole wrote: Eelco Chaudron writes: This patch adds a Python script that can be used to analyze the revalidator runs by providing statistics (including some real time graphs). The USDT events can also be captured

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-22 Thread Adrian Moreno
On 9/21/22 11:17, Eelco Chaudron wrote: On 20 Sep 2022, at 20:52, Aaron Conole wrote: Eelco Chaudron writes: This patch adds a Python script that can be used to analyze the revalidator runs by providing statistics (including some real time graphs). The USDT events can also be captured

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-21 Thread Eelco Chaudron
On 21 Sep 2022, at 11:17, Eelco Chaudron wrote: > On 20 Sep 2022, at 20:52, Aaron Conole wrote: > >> Eelco Chaudron writes: >> >>> This patch adds a Python script that can be used to analyze the >>> revalidator runs by providing statistics (including some real time >>> graphs). >>> >>> The

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-21 Thread Eelco Chaudron
On 20 Sep 2022, at 20:52, Aaron Conole wrote: > Eelco Chaudron writes: > >> This patch adds a Python script that can be used to analyze the >> revalidator runs by providing statistics (including some real time >> graphs). >> >> The USDT events can also be captured to a file and used for >>

Re: [ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-20 Thread Aaron Conole
Eelco Chaudron writes: > This patch adds a Python script that can be used to analyze the > revalidator runs by providing statistics (including some real time > graphs). > > The USDT events can also be captured to a file and used for > later offline analysis. > > Signed-off-by: Eelco Chaudron >

[ovs-dev] [PATCH v2] utilities: Add revalidator measurement script and needed USDT probes.

2022-09-20 Thread Eelco Chaudron
This patch adds a Python script that can be used to analyze the revalidator runs by providing statistics (including some real time graphs). The USDT events can also be captured to a file and used for later offline analysis. Signed-off-by: Eelco Chaudron --- v2: Added note that script only