[PATCH v2] regression_tests: simple_macsec add nperf_msg_size alias

[Kamil Jerabek]

This commit adds nperf_msg_size alias to our phase3 regression_tests
simple_macsec test. The default value is set to 16000.

Signed-off-by: Kamil Jerabek <kjerabek(a)redhat.com&gt;

---

Changes in
v2: * added set parameter nperf_msg_size to nperf results
---
 recipes/regression_tests/phase3/simple_macsec.py  | 9 +++++++++
 recipes/regression_tests/phase3/simple_macsec.xml | 1 +
 2 files changed, 10 insertions(+)

diff --git a/recipes/regression_tests/phase3/simple_macsec.py
b/recipes/regression_tests/phase3/simple_macsec.py
index b16214b..96f1ee3 100644
--- a/recipes/regression_tests/phase3/simple_macsec.py
+++ b/recipes/regression_tests/phase3/simple_macsec.py
@@ -36,6 +36,7 @@ nperf_cpu_util = ctl.get_alias("nperf_cpu_util")
 nperf_num_parallel = int(ctl.get_alias("nperf_num_parallel"))
 nperf_debug = ctl.get_alias("nperf_debug")
 nperf_max_dev = ctl.get_alias("nperf_max_dev")
+nperf_msg_size = int(ctl.get_alias("nperf_msg_size"))
 pr_user_comment = ctl.get_alias("perfrepo_comment")
 
 pr_comment = generate_perfrepo_comment([m1, m2], pr_user_comment)
@@ -144,6 +145,7 @@ for setting in encryption_settings:
             result_tcp.set_parameter('num_parallel', nperf_num_parallel)
 
         result_tcp.set_parameter('encryption', setting)
+        result_tcp.set_parameter("nperf_msg_size", nperf_msg_size)
 
         baseline = perf_api.get_baseline_of_result(result_tcp)
         baseline = perfrepo_baseline_to_dict(baseline)
@@ -158,6 +160,7 @@ for setting in encryption_settings:
                                             "runs": nperf_max_runs,
                                             "debug": nperf_debug,
                                             "max_deviation": nperf_max_dev,
+                                            "msg_size": nperf_msg_size,
                                             "netperf_opts": nperf_opts},
                                baseline = baseline,
                                timeout = (netperf_duration +
nperf_reserve)*nperf_max_runs)
@@ -178,6 +181,7 @@ for setting in encryption_settings:
             result_udp.set_parameter('num_parallel', nperf_num_parallel)
 
         result_udp.set_parameter('encryption', setting)
+        result_udp.set_parameter("nperf_msg_size", nperf_msg_size)
 
         baseline = perf_api.get_baseline_of_result(result_udp)
         baseline = perfrepo_baseline_to_dict(baseline)
@@ -192,6 +196,7 @@ for setting in encryption_settings:
                                             "runs": nperf_max_runs,
                                             "debug": nperf_debug,
                                             "max_deviation": nperf_max_dev,
+                                            "msg_size": nperf_msg_size,
                                             "netperf_opts": nperf_opts},
                                baseline = baseline,
                                timeout = (netperf_duration +
nperf_reserve)*nperf_max_runs)
@@ -220,6 +225,7 @@ for setting in encryption_settings:
             result_tcp.set_parameter('num_parallel', nperf_num_parallel)
 
         result_tcp.set_parameter('encryption', setting)
+        result_tcp.set_parameter("nperf_msg_size", nperf_msg_size)
 
         baseline = perf_api.get_baseline_of_result(result_tcp)
         baseline = perfrepo_baseline_to_dict(baseline)
@@ -234,6 +240,7 @@ for setting in encryption_settings:
                                             "runs": nperf_max_runs,
                                             "debug": nperf_debug,
                                             "max_deviation": nperf_max_dev,
+                                            "msg_size": nperf_msg_size,
                                             "netperf_opts" : nperf_opts +
" -6"},
                                baseline = baseline,
                                timeout = (netperf_duration +
nperf_reserve)*nperf_max_runs)
@@ -254,6 +261,7 @@ for setting in encryption_settings:
             result_udp.set_parameter('num_parallel', nperf_num_parallel)
 
         result_udp.set_parameter('encryption', setting)
+        result_udp.set_parameter("nperf_msg_size", nperf_msg_size)
 
         baseline = perf_api.get_baseline_of_result(result_udp)
         baseline = perfrepo_baseline_to_dict(baseline)
@@ -268,6 +276,7 @@ for setting in encryption_settings:
                                             "runs": nperf_max_runs,
                                             "debug": nperf_debug,
                                             "max_deviation": nperf_max_dev,
+                                            "msg_size": nperf_msg_size,
                                             "netperf_opts" : nperf_opts +
"-6"},
                                baseline = baseline,
                                timeout = (netperf_duration +
nperf_reserve)*nperf_max_runs)
diff --git a/recipes/regression_tests/phase3/simple_macsec.xml
b/recipes/regression_tests/phase3/simple_macsec.xml
index 2209af6..031314b 100644
--- a/recipes/regression_tests/phase3/simple_macsec.xml
+++ b/recipes/regression_tests/phase3/simple_macsec.xml
@@ -9,6 +9,7 @@
         <alias name="nperf_num_parallel" value="1" />
         <alias name="nperf_debug" value="0"/>
         <alias name="nperf_max_dev" value="20%"/>
+        <alias name="nperf_msg_size" value="16000" />
         <alias name="mapping_file" value="simple_macsec.mapping"
/>
         <alias name="net" value="192.168.0" />
         <alias name="driver" value=""/>
-- 
2.5.5

[Jan Tluka]

Thu, Feb 16, 2017 at 12:43:53PM CET, kjerabek(a)redhat.com wrote:
> This commit adds nperf_msg_size alias to our phase3 regression_tests
> simple_macsec test. The default value is set to 16000.
>
> Signed-off-by: Kamil Jerabek <kjerabek(a)redhat.com&gt;
>
> ---
>
> Changes in
> v2: * added set parameter nperf_msg_size to nperf results 
Kamil, I just realized that patch from Jiri already added this.

Please check the git log for commit b6294740cf7bf15dfc1918cce95a6b7064231691
regression-tests: add nperf_msg_size alias to all phase3 tests

-Jan