To be able to get additional information about the nature of the Flow result a reference to the Flow is added to result data.
Signed-off-by: Jan Tluka jtluka@redhat.com --- lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py index 9cc91ab5..c5e65044 100644 --- a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py +++ b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py @@ -302,7 +302,8 @@ class BaseFlowMeasurement(BaseMeasurement): generator_flow_data=generator, generator_cpu_data=generator_cpu, receiver_flow_data=receiver, - receiver_cpu_data=receiver_cpu)) + receiver_cpu_data=receiver_cpu, + flow=flow_results.flow))
def aggregate_results(self, old, new): aggregated = []
would it be more universal to just link the full flow_results object instead? Could make it easier to access additional information if required for post processing or analysis.
-Ondrej
On Mon, Mar 22, 2021 at 05:25:34PM +0100, Jan Tluka wrote:
To be able to get additional information about the nature of the Flow result a reference to the Flow is added to result data.
Signed-off-by: Jan Tluka jtluka@redhat.com
lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py index 9cc91ab5..c5e65044 100644 --- a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py +++ b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py @@ -302,7 +302,8 @@ class BaseFlowMeasurement(BaseMeasurement): generator_flow_data=generator, generator_cpu_data=generator_cpu, receiver_flow_data=receiver,
receiver_cpu_data=receiver_cpu))
receiver_cpu_data=receiver_cpu,
flow=flow_results.flow))
def aggregate_results(self, old, new): aggregated = []
-- 2.26.2 _______________________________________________ LNST-developers mailing list -- lnst-developers@lists.fedorahosted.org To unsubscribe send an email to lnst-developers-leave@lists.fedorahosted.org 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/lnst-developers@lists.fedorahos... Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
Mon, Mar 22, 2021 at 06:00:58PM CET, olichtne@redhat.com wrote:
would it be more universal to just link the full flow_results object instead? Could make it easier to access additional information if required for post processing or analysis.
-Ondrej
Good idea. I will send v2. Thanks for checking!
On Mon, Mar 22, 2021 at 05:25:34PM +0100, Jan Tluka wrote:
To be able to get additional information about the nature of the Flow result a reference to the Flow is added to result data.
Signed-off-by: Jan Tluka jtluka@redhat.com
lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py index 9cc91ab5..c5e65044 100644 --- a/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py +++ b/lnst/RecipeCommon/Perf/Measurements/BaseFlowMeasurement.py @@ -302,7 +302,8 @@ class BaseFlowMeasurement(BaseMeasurement): generator_flow_data=generator, generator_cpu_data=generator_cpu, receiver_flow_data=receiver,
receiver_cpu_data=receiver_cpu))
receiver_cpu_data=receiver_cpu,
flow=flow_results.flow))
def aggregate_results(self, old, new): aggregated = []
-- 2.26.2 _______________________________________________ LNST-developers mailing list -- lnst-developers@lists.fedorahosted.org To unsubscribe send an email to lnst-developers-leave@lists.fedorahosted.org 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/lnst-developers@lists.fedorahos... Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
lnst-developers@lists.fedorahosted.org