[PATCH] recipes: add nperf_max_runs alias instead of improper retrieval from confidence parameter

[Jan Tluka]

This patch fixes the improper retrieval of maximal number of iterations
when running netperf with confidence level and interval set. The second number
of this parameter specifies the confidence interval in percents rather than
runs. If a user wants to specify maximal number of iterations he can do so by
specifying nperf_max_runs alias.

Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt;
---
 recipes/regression_tests/phase1/3_vlans.py                              | 2 +-
 recipes/regression_tests/phase1/3_vlans.xml                             | 1 +
 recipes/regression_tests/phase1/3_vlans_over_active_backup_bond.xml     | 1 +
 recipes/regression_tests/phase1/3_vlans_over_bond.py                    | 2 +-
 recipes/regression_tests/phase1/3_vlans_over_round_robin_bond.xml       | 1 +
 recipes/regression_tests/phase1/active_backup_bond.xml                  | 1 +
 recipes/regression_tests/phase1/active_backup_double_bond.xml           | 1 +
 recipes/regression_tests/phase1/bonding_test.py                         | 2 +-
 recipes/regression_tests/phase1/round_robin_bond.xml                    | 1 +
 recipes/regression_tests/phase1/round_robin_double_bond.xml             | 1 +
 .../phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml           | 1 +
 recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py     | 2 +-
 recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py         | 2 +-
 recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml        | 1 +
 recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py          | 2 +-
 recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml         | 1 +
 recipes/regression_tests/phase2/3_vlans_over_active_backup_team.xml     | 1 +
 recipes/regression_tests/phase2/3_vlans_over_round_robin_team.xml       | 1 +
 recipes/regression_tests/phase2/3_vlans_over_team.py                    | 2 +-
 recipes/regression_tests/phase2/active_backup_double_team.xml           | 1 +
 recipes/regression_tests/phase2/active_backup_team.xml                  | 1 +
 .../phase2/active_backup_team_vs_active_backup_bond.xml                 | 1 +
 .../regression_tests/phase2/active_backup_team_vs_round_robin_bond.xml  | 1 +
 recipes/regression_tests/phase2/round_robin_double_team.xml             | 1 +
 recipes/regression_tests/phase2/round_robin_team.xml                    | 1 +
 .../regression_tests/phase2/round_robin_team_vs_active_backup_bond.xml  | 1 +
 .../regression_tests/phase2/round_robin_team_vs_round_robin_bond.xml    | 1 +
 recipes/regression_tests/phase2/team_test.py                            | 2 +-
 .../phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.py        | 2 +-
 .../phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.xml       | 1 +
 recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py     | 2 +-
 recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.xml    | 1 +
 recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py      | 2 +-
 recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.xml     | 1 +
 34 files changed, 34 insertions(+), 11 deletions(-)

diff --git a/recipes/regression_tests/phase1/3_vlans.py
b/recipes/regression_tests/phase1/3_vlans.py
index 6e04f67..b3c3966 100644
--- a/recipes/regression_tests/phase1/3_vlans.py
+++ b/recipes/regression_tests/phase1/3_vlans.py
@@ -44,7 +44,7 @@ mtu = ctl.get_alias("mtu")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 m1_phy1 = m1.get_interface("eth1")
 m1_phy1.set_mtu(mtu)
diff --git a/recipes/regression_tests/phase1/3_vlans.xml
b/recipes/regression_tests/phase1/3_vlans.xml
index 86b82b6..18fc539 100644
--- a/recipes/regression_tests/phase1/3_vlans.xml
+++ b/recipes/regression_tests/phase1/3_vlans.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file" value="3_vlans.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/3_vlans_over_active_backup_bond.xml
b/recipes/regression_tests/phase1/3_vlans_over_active_backup_bond.xml
index aabf2c6..a412adc 100644
--- a/recipes/regression_tests/phase1/3_vlans_over_active_backup_bond.xml
+++ b/recipes/regression_tests/phase1/3_vlans_over_active_backup_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="3_vlans_over_active_backup_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/3_vlans_over_bond.py
b/recipes/regression_tests/phase1/3_vlans_over_bond.py
index 95936d1..0fb6356 100644
--- a/recipes/regression_tests/phase1/3_vlans_over_bond.py
+++ b/recipes/regression_tests/phase1/3_vlans_over_bond.py
@@ -44,7 +44,7 @@ mtu = ctl.get_alias("mtu")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 m1_bond = m1.get_interface("test_bond")
 m1_bond.set_mtu(mtu)
diff --git a/recipes/regression_tests/phase1/3_vlans_over_round_robin_bond.xml
b/recipes/regression_tests/phase1/3_vlans_over_round_robin_bond.xml
index fda2f1b..35786fb 100644
--- a/recipes/regression_tests/phase1/3_vlans_over_round_robin_bond.xml
+++ b/recipes/regression_tests/phase1/3_vlans_over_round_robin_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="3_vlans_over_round_robin_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/active_backup_bond.xml
b/recipes/regression_tests/phase1/active_backup_bond.xml
index 08f8718..c9b3883 100644
--- a/recipes/regression_tests/phase1/active_backup_bond.xml
+++ b/recipes/regression_tests/phase1/active_backup_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="active_backup_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/active_backup_double_bond.xml
b/recipes/regression_tests/phase1/active_backup_double_bond.xml
index 256ebfc..45258e9 100644
--- a/recipes/regression_tests/phase1/active_backup_double_bond.xml
+++ b/recipes/regression_tests/phase1/active_backup_double_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="active_backup_double_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/bonding_test.py
b/recipes/regression_tests/phase1/bonding_test.py
index a343b9e..2e2e041 100644
--- a/recipes/regression_tests/phase1/bonding_test.py
+++ b/recipes/regression_tests/phase1/bonding_test.py
@@ -43,7 +43,7 @@ mtu = ctl.get_alias("mtu")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 test_if1 = m1.get_interface("test_if")
 test_if1.set_mtu(mtu)
diff --git a/recipes/regression_tests/phase1/round_robin_bond.xml
b/recipes/regression_tests/phase1/round_robin_bond.xml
index cc055ec..9c2b716 100644
--- a/recipes/regression_tests/phase1/round_robin_bond.xml
+++ b/recipes/regression_tests/phase1/round_robin_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="round_robin_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase1/round_robin_double_bond.xml
b/recipes/regression_tests/phase1/round_robin_double_bond.xml
index 0f5a98b..896404c 100644
--- a/recipes/regression_tests/phase1/round_robin_double_bond.xml
+++ b/recipes/regression_tests/phase1/round_robin_double_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mapping_file"
value="round_robin_double_bond.mapping" />
     </define>
     <network>
diff --git
a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
index a9646d8..04de618 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
+++ b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_active_backup_bond.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mtu" value="1500" />
         <alias name="mapping_file"
value="virtual_bridge_2_vlans_over_active_backup_bond.mapping" />
     </define>
diff --git a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
index 8226fc4..d96eaec 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
+++ b/recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py
@@ -49,7 +49,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 mtu = ctl.get_alias("mtu")
 enable_udp_perf = ctl.get_alias("enable_udp_perf")
diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
index 0c7ffa4..daba9f0 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.py
@@ -43,7 +43,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 mtu = ctl.get_alias("mtu")
 enable_udp_perf = ctl.get_alias("enable_udp_perf")
diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
index 63e23fe..7d36ba5 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_guest.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mtu" value="1500" />
         <alias name="mapping_file"
value="virtual_bridge_vlan_in_guest.mapping" />
     </define>
diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
index 509ea57..10d0696 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.py
@@ -43,7 +43,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 mtu = ctl.get_alias("mtu")
 enable_udp_perf = ctl.get_alias("enable_udp_perf")
diff --git a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
index 4f8f1e2..ae351b2 100644
--- a/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
+++ b/recipes/regression_tests/phase1/virtual_bridge_vlan_in_host.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5"/>
         <alias name="mtu" value="1500" />
         <alias name="mapping_file"
value="virtual_bridge_vlan_in_host.mapping" />
     </define>
diff --git a/recipes/regression_tests/phase2/3_vlans_over_active_backup_team.xml
b/recipes/regression_tests/phase2/3_vlans_over_active_backup_team.xml
index 8937bef..2d2d6bf 100644
--- a/recipes/regression_tests/phase2/3_vlans_over_active_backup_team.xml
+++ b/recipes/regression_tests/phase2/3_vlans_over_active_backup_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="3_vlans_over_active_backup_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/3_vlans_over_round_robin_team.xml
b/recipes/regression_tests/phase2/3_vlans_over_round_robin_team.xml
index cab3d7a..7bbf9a2 100644
--- a/recipes/regression_tests/phase2/3_vlans_over_round_robin_team.xml
+++ b/recipes/regression_tests/phase2/3_vlans_over_round_robin_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="3_vlans_over_round_robin_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/3_vlans_over_team.py
b/recipes/regression_tests/phase2/3_vlans_over_team.py
index 157da14..361f497 100644
--- a/recipes/regression_tests/phase2/3_vlans_over_team.py
+++ b/recipes/regression_tests/phase2/3_vlans_over_team.py
@@ -43,7 +43,7 @@ mtu = ctl.get_alias("mtu")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 m1_team = m1.get_interface("test_if")
 m1_team.set_mtu(mtu)
diff --git a/recipes/regression_tests/phase2/active_backup_double_team.xml
b/recipes/regression_tests/phase2/active_backup_double_team.xml
index a3f0549..34ed65e 100644
--- a/recipes/regression_tests/phase2/active_backup_double_team.xml
+++ b/recipes/regression_tests/phase2/active_backup_double_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="active_backup_double_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/active_backup_team.xml
b/recipes/regression_tests/phase2/active_backup_team.xml
index 30fe553..ac3eb6f 100644
--- a/recipes/regression_tests/phase2/active_backup_team.xml
+++ b/recipes/regression_tests/phase2/active_backup_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="active_backup_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/active_backup_team_vs_active_backup_bond.xml
b/recipes/regression_tests/phase2/active_backup_team_vs_active_backup_bond.xml
index 1badb1f..d20348e 100644
--- a/recipes/regression_tests/phase2/active_backup_team_vs_active_backup_bond.xml
+++ b/recipes/regression_tests/phase2/active_backup_team_vs_active_backup_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="active_backup_team_vs_active_backup_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/active_backup_team_vs_round_robin_bond.xml
b/recipes/regression_tests/phase2/active_backup_team_vs_round_robin_bond.xml
index c921037..08a2d1e 100644
--- a/recipes/regression_tests/phase2/active_backup_team_vs_round_robin_bond.xml
+++ b/recipes/regression_tests/phase2/active_backup_team_vs_round_robin_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="active_backup_team_vs_round_robin_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/round_robin_double_team.xml
b/recipes/regression_tests/phase2/round_robin_double_team.xml
index 585e446..0d6cb6f 100644
--- a/recipes/regression_tests/phase2/round_robin_double_team.xml
+++ b/recipes/regression_tests/phase2/round_robin_double_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="round_robin_double_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/round_robin_team.xml
b/recipes/regression_tests/phase2/round_robin_team.xml
index c257851..4d6a3c7 100644
--- a/recipes/regression_tests/phase2/round_robin_team.xml
+++ b/recipes/regression_tests/phase2/round_robin_team.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="round_robin_team.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/round_robin_team_vs_active_backup_bond.xml
b/recipes/regression_tests/phase2/round_robin_team_vs_active_backup_bond.xml
index b59188a..6b5f065 100644
--- a/recipes/regression_tests/phase2/round_robin_team_vs_active_backup_bond.xml
+++ b/recipes/regression_tests/phase2/round_robin_team_vs_active_backup_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="round_robin_team_vs_active_backup_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/round_robin_team_vs_round_robin_bond.xml
b/recipes/regression_tests/phase2/round_robin_team_vs_round_robin_bond.xml
index 88c2617..aeb6e7e 100644
--- a/recipes/regression_tests/phase2/round_robin_team_vs_round_robin_bond.xml
+++ b/recipes/regression_tests/phase2/round_robin_team_vs_round_robin_bond.xml
@@ -5,6 +5,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="round_robin_team_vs_round_robin_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/team_test.py
b/recipes/regression_tests/phase2/team_test.py
index 4e7aed6..e132be9 100644
--- a/recipes/regression_tests/phase2/team_test.py
+++ b/recipes/regression_tests/phase2/team_test.py
@@ -43,7 +43,7 @@ mtu = ctl.get_alias("mtu")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 test_if1 = m1.get_interface("test_if")
 test_if1.set_mtu(mtu)
diff --git
a/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.py
b/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.py
index 6e6aa10..fa7961c 100644
---
a/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.py
+++
b/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.py
@@ -49,7 +49,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 ping_mod = ctl.get_module("IcmpPing",
                            options={
diff --git
a/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.xml
b/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.xml
index 71f4778..97f9b6e 100644
---
a/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.xml
+++
b/recipes/regression_tests/phase2/virtual_ovs_bridge_2_vlans_over_active_backup_bond.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="virtual_ovs_bridge_2_vlans_over_active_backup_bond.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py
b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py
index 1ce8b36..d953ab7 100644
--- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py
+++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.py
@@ -43,7 +43,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 ping_mod = ctl.get_module("IcmpPing",
                            options={
diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.xml
b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.xml
index bbd0902..80ad328 100644
--- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.xml
+++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_guest.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="virtual_ovs_bridge_vlan_in_guest.mapping" />
     </define>
     <network>
diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py
b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py
index 43381da..058f9ea 100644
--- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py
+++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.py
@@ -43,7 +43,7 @@ ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
 nperf_confidence = ctl.get_alias("nperf_confidence")
-nperf_max_runs = int(nperf_confidence.split(",")[1])
+nperf_max_runs = int(ctl.get_alias("nperf_max_runs"))
 
 ping_mod = ctl.get_module("IcmpPing",
                            options={
diff --git a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.xml
b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.xml
index e41aa26..9995891 100644
--- a/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.xml
+++ b/recipes/regression_tests/phase2/virtual_ovs_bridge_vlan_in_host.xml
@@ -4,6 +4,7 @@
         <alias name="netperf_duration" value="60" />
         <alias name="nperf_reserve" value="20" />
         <alias name="nperf_confidence" value="99,5" />
+        <alias name="nperf_max_runs" value="5" />
         <alias name="mapping_file"
value="virtual_ovs_bridge_vlan_in_host.mapping" />
     </define>
     <network>
-- 
2.1.0

[Ondrej Lichtner]

On Fri, Oct 02, 2015 at 02:34:48PM +0200, Jan Tluka wrote:
>  This patch fixes the improper retrieval of maximal number of
> iterations
>  when running netperf with confidence level and interval set. The second number
>  of this parameter specifies the confidence interval in percents rather than
>  runs. If a user wants to specify maximal number of iterations he can do so by
>  specifying nperf_max_runs alias.
>  
>  Signed-off-by: Jan Tluka <jtluka(a)redhat.com&gt; 
Acked-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;