[PATCH] NeperFlowMeasurement.py: Bumped up server timeout

From: Perry Gagne

I found that sometimes when running things like tcp_crr the server needed a little more
time to complete.

Signed-off-by: Perry Gagne 
---
 lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
index 236c2c9b..1f667fb5 100644
--- a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
+++ b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
@@ -51,7 +51,7 @@ class NeperFlowMeasurement(BaseFlowMeasurement):
             for flow in test_flows:
                 client_neper = flow.client_job.what
                 flow.client_job.wait(timeout=client_neper.runtime_estimate())
-                flow.server_job.wait(timeout=5)
+                flow.server_job.wait(timeout=10)
         finally:
             for flow in test_flows:
                 flow.server_job.kill()

[Jan Tluka]

Fri, Mar 26, 2021 at 10:06:30PM CET, [e-mail] wrote:
> From: Perry Gagne
>
> I found that sometimes when running things like tcp_crr the server needed a little more
> time to complete.
> Signed-off-by: Perry Gagne
>
> lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> index 236c2c9b..1f667fb5 100644
> --- a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> +++ b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> @@ -51,7 +51,7 @@ class NeperFlowMeasurement(BaseFlowMeasurement):
>             for flow in test_flows:
>                 client_neper = flow.client_job.what
>                 flow.client_job.wait(timeout=client_neper.runtime_estimate())
>            flow.server_job.wait(timeout=5)
>            flow.server_job.wait(timeout=10)
>   finally:
>       for flow in test_flows:
>           flow.server_job.kill()
>
> -- 
> 2.30.2
> _______________________________________________
> 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]...
> Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure

Looks good.

Acked-by: Jan Tluka

[Ondrej Lichtner]

On Fri, Mar 26, 2021 at 05:06:30PM -0400, [e-mail] wrote:
> From: Perry Gagne
>
> I found that sometimes when running things like tcp_crr the server needed a little more
> time to complete.
> Signed-off-by: Perry Gagne
>
> lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> index 236c2c9b..1f667fb5 100644
> --- a/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> +++ b/lnst/RecipeCommon/Perf/Measurements/NeperFlowMeasurement.py
> @@ -51,7 +51,7 @@ class NeperFlowMeasurement(BaseFlowMeasurement):
>              for flow in test_flows:
>                  client_neper = flow.client_job.what
>                  flow.client_job.wait(timeout=client_neper.runtime_estimate())
>            flow.server_job.wait(timeout=5)
>            flow.server_job.wait(timeout=10)
>    finally:
>        for flow in test_flows:
>            flow.server_job.kill()
>
> -- 
> 2.30.2
> _______________________________________________
> 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]...
> Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure

thanks, pushed.

-Ondrej