[PATCH] recipes: fix missing msg size parameter in perfrepo result for SCTP

[Jan Tluka]

The netperf packet size was not set in the PerfRepo result for
SCTP_STREAM test.

Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
---
 recipes/regression_tests/phase1/3_vlans.py           |  6 ++++++
 recipes/regression_tests/phase1/3_vlans_over_bond.py |  6 ++++++
 recipes/regression_tests/phase1/bonding_test.py      |  6 ++++++
 recipes/regression_tests/phase1/simple_netperf.py    |  6 ++++++
 recipes/regression_tests/phase2/3_vlans_over_team.py |  6 ++++++
 recipes/regression_tests/phase2/team_test.py         | 12 ++++++++++++
 6 files changed, 42 insertions(+)

diff --git a/recipes/regression_tests/phase1/3_vlans.py
b/recipes/regression_tests/phase1/3_vlans.py
index 2b72f4e..d7783df 100644
--- a/recipes/regression_tests/phase1/3_vlans.py
+++ b/recipes/regression_tests/phase1/3_vlans.py
@@ -364,6 +364,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
@@ -458,6 +461,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
diff --git a/recipes/regression_tests/phase1/3_vlans_over_bond.py
b/recipes/regression_tests/phase1/3_vlans_over_bond.py
index 8cf0e7e..3d453df 100644
--- a/recipes/regression_tests/phase1/3_vlans_over_bond.py
+++ b/recipes/regression_tests/phase1/3_vlans_over_bond.py
@@ -364,6 +364,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
@@ -458,6 +461,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
diff --git a/recipes/regression_tests/phase1/bonding_test.py
b/recipes/regression_tests/phase1/bonding_test.py
index 324a455..cb3b85f 100644
--- a/recipes/regression_tests/phase1/bonding_test.py
+++ b/recipes/regression_tests/phase1/bonding_test.py
@@ -348,6 +348,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.add_tag(product_name)
             if nperf_mode == "multi":
                 result_sctp.add_tag("multithreaded")
@@ -438,6 +441,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.add_tag(product_name)
             if nperf_mode == "multi":
                 result_sctp.add_tag("multithreaded")
diff --git a/recipes/regression_tests/phase1/simple_netperf.py
b/recipes/regression_tests/phase1/simple_netperf.py
index e159953..1aad6f9 100644
--- a/recipes/regression_tests/phase1/simple_netperf.py
+++ b/recipes/regression_tests/phase1/simple_netperf.py
@@ -310,6 +310,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.add_tag(product_name)
             if nperf_mode == "multi":
                 result_sctp.add_tag("multithreaded")
@@ -396,6 +399,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.add_tag(product_name)
             if nperf_mode == "multi":
                 result_sctp.add_tag("multithreaded")
diff --git a/recipes/regression_tests/phase2/3_vlans_over_team.py
b/recipes/regression_tests/phase2/3_vlans_over_team.py
index c4ea8e8..bad0563 100644
--- a/recipes/regression_tests/phase2/3_vlans_over_team.py
+++ b/recipes/regression_tests/phase2/3_vlans_over_team.py
@@ -364,6 +364,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
@@ -458,6 +461,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
             result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
             result_sctp.add_tag(product_name)
diff --git a/recipes/regression_tests/phase2/team_test.py
b/recipes/regression_tests/phase2/team_test.py
index 2fa03dd..51da678 100644
--- a/recipes/regression_tests/phase2/team_test.py
+++ b/recipes/regression_tests/phase2/team_test.py
@@ -353,6 +353,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server',
"testmachine1")
             result_sctp.set_parameter('netperf_client',
"testmachine2")
             result_sctp.add_tag(product_name)
@@ -449,6 +452,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server',
"testmachine1")
             result_sctp.set_parameter('netperf_client',
"testmachine2")
             result_sctp.add_tag(product_name)
@@ -596,6 +602,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server',
"testmachine2")
             result_sctp.set_parameter('netperf_client',
"testmachine1")
             result_sctp.add_tag(product_name)
@@ -692,6 +701,9 @@ for setting in offload_settings:
             for offload in setting:
                 result_sctp.set_parameter(offload[0], offload[1])
 
+            if nperf_msg_size is not None:
+                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
+
             result_sctp.set_parameter('netperf_server',
"testmachine2")
             result_sctp.set_parameter('netperf_client',
"testmachine1")
             result_sctp.add_tag(product_name)
-- 
2.9.5

[Ondrej Lichtner]

On Mon, Nov 20, 2017 at 07:50:51PM +0100, Jan Tluka wrote:
>  The netperf packet size was not set in the PerfRepo result for
>  SCTP_STREAM test.
>  
>  Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
>  ---
>   recipes/regression_tests/phase1/3_vlans.py           |  6 ++++++
>   recipes/regression_tests/phase1/3_vlans_over_bond.py |  6 ++++++
>   recipes/regression_tests/phase1/bonding_test.py      |  6 ++++++
>   recipes/regression_tests/phase1/simple_netperf.py    |  6 ++++++
>   recipes/regression_tests/phase2/3_vlans_over_team.py |  6 ++++++
>   recipes/regression_tests/phase2/team_test.py         | 12 ++++++++++++
>   6 files changed, 42 insertions(+)
>  
>  diff --git a/recipes/regression_tests/phase1/3_vlans.py
> b/recipes/regression_tests/phase1/3_vlans.py
>  index 2b72f4e..d7783df 100644
>  --- a/recipes/regression_tests/phase1/3_vlans.py
>  +++ b/recipes/regression_tests/phase1/3_vlans.py
>  @@ -364,6 +364,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  @@ -458,6 +461,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  diff --git a/recipes/regression_tests/phase1/3_vlans_over_bond.py
> b/recipes/regression_tests/phase1/3_vlans_over_bond.py
>  index 8cf0e7e..3d453df 100644
>  --- a/recipes/regression_tests/phase1/3_vlans_over_bond.py
>  +++ b/recipes/regression_tests/phase1/3_vlans_over_bond.py
>  @@ -364,6 +364,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  @@ -458,6 +461,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  diff --git a/recipes/regression_tests/phase1/bonding_test.py
> b/recipes/regression_tests/phase1/bonding_test.py
>  index 324a455..cb3b85f 100644
>  --- a/recipes/regression_tests/phase1/bonding_test.py
>  +++ b/recipes/regression_tests/phase1/bonding_test.py
>  @@ -348,6 +348,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.add_tag(product_name)
>               if nperf_mode == "multi":
>                   result_sctp.add_tag("multithreaded")
>  @@ -438,6 +441,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.add_tag(product_name)
>               if nperf_mode == "multi":
>                   result_sctp.add_tag("multithreaded")
>  diff --git a/recipes/regression_tests/phase1/simple_netperf.py
> b/recipes/regression_tests/phase1/simple_netperf.py
>  index e159953..1aad6f9 100644
>  --- a/recipes/regression_tests/phase1/simple_netperf.py
>  +++ b/recipes/regression_tests/phase1/simple_netperf.py
>  @@ -310,6 +310,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.add_tag(product_name)
>               if nperf_mode == "multi":
>                   result_sctp.add_tag("multithreaded")
>  @@ -396,6 +399,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.add_tag(product_name)
>               if nperf_mode == "multi":
>                   result_sctp.add_tag("multithreaded")
>  diff --git a/recipes/regression_tests/phase2/3_vlans_over_team.py
> b/recipes/regression_tests/phase2/3_vlans_over_team.py
>  index c4ea8e8..bad0563 100644
>  --- a/recipes/regression_tests/phase2/3_vlans_over_team.py
>  +++ b/recipes/regression_tests/phase2/3_vlans_over_team.py
>  @@ -364,6 +364,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  @@ -458,6 +461,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server_on_vlan', vlans[0])
>               result_sctp.set_parameter('netperf_client_on_vlan', vlans[0])
>               result_sctp.add_tag(product_name)
>  diff --git a/recipes/regression_tests/phase2/team_test.py
> b/recipes/regression_tests/phase2/team_test.py
>  index 2fa03dd..51da678 100644
>  --- a/recipes/regression_tests/phase2/team_test.py
>  +++ b/recipes/regression_tests/phase2/team_test.py
>  @@ -353,6 +353,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server',
> "testmachine1")
>               result_sctp.set_parameter('netperf_client',
> "testmachine2")
>               result_sctp.add_tag(product_name)
>  @@ -449,6 +452,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server',
> "testmachine1")
>               result_sctp.set_parameter('netperf_client',
> "testmachine2")
>               result_sctp.add_tag(product_name)
>  @@ -596,6 +602,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server',
> "testmachine2")
>               result_sctp.set_parameter('netperf_client',
> "testmachine1")
>               result_sctp.add_tag(product_name)
>  @@ -692,6 +701,9 @@ for setting in offload_settings:
>               for offload in setting:
>                   result_sctp.set_parameter(offload[0], offload[1])
>   
>  +            if nperf_msg_size is not None:
>  +                result_sctp.set_parameter("nperf_msg_size", nperf_msg_size)
>  +
>               result_sctp.set_parameter('netperf_server',
> "testmachine2")
>               result_sctp.set_parameter('netperf_client',
> "testmachine1")
>               result_sctp.add_tag(product_name)
>  -- 
>  2.9.5
>  _______________________________________________
>  LNST-developers mailing list -- lnst-developers(a)lists.fedorahosted.org
>  To unsubscribe send an email to lnst-developers-leave(a)lists.fedorahosted.org 
pushed, thanks

-Ondrej