[PATCH] TRexFlowMeasurement: add missing timestamp initialization to PerfInterval

From: Ondrej Lichtner

Signed-off-by: Ondrej Lichtner 
---
 .../Perf/Measurements/TRexFlowMeasurement.py     | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
index d96ecc9..b780842 100644
--- a/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
+++ b/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
@@ -147,10 +147,10 @@ def _parse_results_by_port(self, job, port, flow):
         results.receiver_cpu_stats = SequentialPerfResult()

if not job.passed:
-            results.generator_results.append(PerfInterval(0, 0, "packets"))
-            results.generator_cpu_stats.append(PerfInterval(0, 0, "cpu_percent"))
-            results.receiver_results.append(PerfInterval(0, 0, "packets"))
-            results.receiver_cpu_stats.append(PerfInterval(0, 0, "cpu_percent"))
+            results.generator_results.append(PerfInterval(0, 0, "packets", None))
+            results.generator_cpu_stats.append(PerfInterval(0, 0, "cpu_percent", None))
+            results.receiver_results.append(PerfInterval(0, 0, "packets", None))
+            results.receiver_cpu_stats.append(PerfInterval(0, 0, "cpu_percent", None))
         else:
             prev_time = job.result["start_time"]
             prev_tx_val = 0
@@ -162,11 +162,11 @@ def _parse_results_by_port(self, job, port, flow):
                 results.generator_results.append(PerfInterval(
                             tx_delta,
                             time_delta,
-                            "pkts"))
+                            "pkts", i["timestamp"]))
                 results.receiver_results.append(PerfInterval(
                             rx_delta,
                             time_delta,
-                            "pkts"))
+                            "pkts", i["timestamp"]))

prev_time = i["timestamp"]
                 prev_tx_val = i["measurement"][port]["opackets"]
@@ -176,9 +176,9 @@ def _parse_results_by_port(self, job, port, flow):
                 results.generator_cpu_stats.append(PerfInterval(
                     cpu_delta,
                     time_delta,
-                    "cpu_percent"))
+                    "cpu_percent", i["timestamp"]))
                 results.receiver_cpu_stats.append(PerfInterval(
                     cpu_delta,
                     time_delta,
-                    "cpu_percent"))
+                    "cpu_percent", i["timestamp"]))
         return results

[Jan Tluka]

Fri, Jan 22, 2021 at 11:49:37AM CET, [e-mail] wrote:
> From: Ondrej Lichtner
> Signed-off-by: Ondrej Lichtner
>
> .../Perf/Measurements/TRexFlowMeasurement.py     | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
> index d96ecc9..b780842 100644
> --- a/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
> +++ b/lnst/RecipeCommon/Perf/Measurements/TRexFlowMeasurement.py
> @@ -147,10 +147,10 @@ def _parse_results_by_port(self, job, port, flow):
>         results.receiver_cpu_stats = SequentialPerfResult()
>     if not job.passed:
>        results.generator_results.append(PerfInterval(0, 0, "packets"))
>        results.generator_cpu_stats.append(PerfInterval(0, 0, "cpu_percent"))
>        results.receiver_results.append(PerfInterval(0, 0, "packets"))
>        results.receiver_cpu_stats.append(PerfInterval(0, 0, "cpu_percent"))
>        results.generator_results.append(PerfInterval(0, 0, "packets", None))
>        results.generator_cpu_stats.append(PerfInterval(0, 0, "cpu_percent", None))
>        results.receiver_results.append(PerfInterval(0, 0, "packets", None))
>        results.receiver_cpu_stats.append(PerfInterval(0, 0, "cpu_percent", None))
>   else:
>       prev_time = job.result["start_time"]
>       prev_tx_val = 0
>
> @@ -162,11 +162,11 @@ def _parse_results_by_port(self, job, port, flow):
>                 results.generator_results.append(PerfInterval(
>                             tx_delta,
>                             time_delta,
>                        "pkts"))
>                        "pkts", i["timestamp"]))
>           results.receiver_results.append(PerfInterval(
>                       rx_delta,
>                       time_delta,
>                        "pkts"))
>                        "pkts", i["timestamp"]))
>
>           prev_time = i["timestamp"]
>           prev_tx_val = i["measurement"][port]["opackets"]
>
> @@ -176,9 +176,9 @@ def _parse_results_by_port(self, job, port, flow):
>                 results.generator_cpu_stats.append(PerfInterval(
>                     cpu_delta,
>                     time_delta,
>                "cpu_percent"))
>                "cpu_percent", i["timestamp"]))
>           results.receiver_cpu_stats.append(PerfInterval(
>               cpu_delta,
>               time_delta,
>                "cpu_percent"))
>                "cpu_percent", i["timestamp"]))
>   return results
>
> -- 
> 2.30.0
> _______________________________________________
> LNST-developers mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
> Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: https://lists.fedorahosted.org/archives/list/[e-mail]...

Acked-by: Jan Tluka

[Ondrej Lichtner]

pushed

-Ondrej

On Fri, Jan 22, 2021 at 11:49:37AM +0100, [e-mail] wrote:
> From: Ondrej Lichtner
>
> Signed-off-by: Ondrej Lichtner