[PATCH v2] regression_tests: add set_mtu to virtual ovs bridge tests

[Kamil Jerabek]

This commit adds missing set_mtu to virtual ovs bridge tests in our
regression tests in phase2.

Signed-off-by: Kamil Jerabek <kjerabek(a)redhat.com&gt;
---
 ...ual_ovs_bridge_2_vlans_over_active_backup_bond.py | 20 ++++++++++++++++++++
 ...al_ovs_bridge_2_vlans_over_active_backup_bond.xml |  1 +
 .../phase2/virtual_ovs_bridge_vlan_in_guest.py       | 12 ++++++++++++
 .../phase2/virtual_ovs_bridge_vlan_in_guest.xml      |  1 +
 .../phase2/virtual_ovs_bridge_vlan_in_host.py        | 11 +++++++++++
 .../phase2/virtual_ovs_bridge_vlan_in_host.xml       |  1 +
 6 files changed, 46 insertions(+)

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 409cb88..c2f3b78 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
@@ -33,6 +33,7 @@ g4.sync_resources(modules=["IcmpPing", "Icmp6Ping",
"Netperf"])
 # TESTS
 # ------
 
+mtu = ctl.get_alias("mtu")
 ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
@@ -230,6 +231,25 @@ ping_mod6_bad2 = ctl.get_module("Icmp6Ping",
                                "interval" : 0.1
                            })
 
+
+# configure mtu
+h1_nic1.set_mtu(mtu)
+h1_nic2.set_mtu(mtu)
+h1.get_interface("tap1").set_mtu(mtu)
+h1.get_interface("tap2").set_mtu(mtu)
+h1.get_interface("bridge").set_mtu(mtu)
+
+h2_nic1.set_mtu(mtu)
+h2_nic2.set_mtu(mtu)
+h2.get_interface("tap1").set_mtu(mtu)
+h2.get_interface("tap2").set_mtu(mtu)
+h2.get_interface("bridge").set_mtu(mtu)
+
+g1.get_interface("guestnic").set_mtu(mtu)
+g2.get_interface("guestnic").set_mtu(mtu)
+g3.get_interface("guestnic").set_mtu(mtu)
+g4.get_interface("guestnic").set_mtu(mtu)
+
 ctl.wait(15)
 
 for setting in offload_settings:
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 32b4c06..01c4291 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
@@ -7,6 +7,7 @@
         <alias name="nperf_max_runs" value="5" />
         <alias name="nperf_mode" value="default"/>
         <alias name="nperf_num_parallel" value="2"/>
+        <alias name="mtu" value="1500" />
         <alias name="nperf_debug" value="0"/>
         <alias name="nperf_max_dev" value="20%"/>
         <alias name="mapping_file"
value="virtual_ovs_bridge_2_vlans_over_active_backup_bond.mapping" />
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 b0dc574..812bc27 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
@@ -27,6 +27,7 @@ h2.sync_resources(modules=["IcmpPing", "Icmp6Ping",
"Netperf"])
 # TESTS
 # ------
 
+mtu = ctl.get_alias("mtu")
 ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
@@ -178,6 +179,17 @@ if nperf_udp_size is not None:
     netperf_cli_udp.update_options({"udp_size" : nperf_udp_size})
     netperf_cli_udp6.update_options({"udp_size" : nperf_udp_size})
 
+#set mtu
+h1.get_interface("nic").set_mtu(mtu)
+h1.get_interface("tap").set_mtu(mtu)
+h1.get_interface("br").set_mtu(mtu)
+
+h2.get_interface("nic").set_mtu(mtu)
+h2.get_interface("vlan10").set_mtu(mtu)
+
+g1.get_interface("guestnic").set_mtu(mtu)
+g1.get_interface("vlan10").set_mtu(mtu)
+
 ctl.wait(15)
 
 for setting in offload_settings:
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 ae85997..786b9d3 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
@@ -9,6 +9,7 @@
         <alias name="nperf_num_parallel" value="2"/>
         <alias name="nperf_debug" value="0"/>
         <alias name="nperf_max_dev" value="20%"/>
+        <alias name="mtu" value="1500" />
         <alias name="mapping_file"
value="virtual_ovs_bridge_vlan_in_guest.mapping" />
         <alias name="vlan10_net" value="192.168.10"/>
         <alias name="vlan10_tag" value="10"/>
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 25f19f1..69560b9 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
@@ -27,6 +27,7 @@ h2.sync_resources(modules=["IcmpPing", "Icmp6Ping",
"Netperf"])
 # TESTS
 # ------
 
+mtu = ctl.get_alias("mtu")
 ipv = ctl.get_alias("ipv")
 netperf_duration = int(ctl.get_alias("netperf_duration"))
 nperf_reserve = int(ctl.get_alias("nperf_reserve"))
@@ -177,6 +178,16 @@ if nperf_udp_size is not None:
     netperf_cli_udp.update_options({"udp_size" : nperf_udp_size})
     netperf_cli_udp6.update_options({"udp_size" : nperf_udp_size})
 
+#set mtu
+h1.get_interface("nic").set_mtu(mtu)
+h1.get_interface("tap").set_mtu(mtu)
+h1.get_interface("ovs_br").set_mtu(mtu)
+
+h2.get_interface("nic").set_mtu(mtu)
+h2.get_interface("vlan10").set_mtu(mtu)
+
+g1.get_interface("guestnic").set_mtu(mtu)
+
 ctl.wait(15)
 
 for setting in offload_settings:
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 6b2a271..5f8b9e5 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
@@ -9,6 +9,7 @@
         <alias name="nperf_num_parallel" value="2"/>
         <alias name="nperf_debug" value="0"/>
         <alias name="nperf_max_dev" value="20%"/>
+        <alias name="mtu" value="1500" />
         <alias name="mapping_file"
value="virtual_ovs_bridge_vlan_in_host.mapping" />
         <alias name="vlan10_net" value="192.168.10"/>
         <alias name="vlan10_tag" value="10"/>
-- 
2.5.5

[Jan Tluka]

Tue, Oct 25, 2016 at 04:24:46PM CEST, kjerabek(a)redhat.com wrote:
> This commit adds missing set_mtu to virtual ovs bridge tests in our
> regression tests in phase2.
>
> Signed-off-by: Kamil Jerabek <kjerabek(a)redhat.com&gt; 
I ran this in our lab and I confirm the patch is safe and does not
change the original hashes. With mtu alias set to 8000 the hashes
changed as expected.

Acked-by: Jan Tluka <jtluka(a)redhat.com&gt;

[Ondrej Lichtner]

On Tue, Oct 25, 2016 at 04:24:46PM +0200, Kamil Jerabek wrote:
>  This commit adds missing set_mtu to virtual ovs bridge tests in our
>  regression tests in phase2.
>  
>  Signed-off-by: Kamil Jerabek <kjerabek(a)redhat.com&gt;
>  ---
>   ...ual_ovs_bridge_2_vlans_over_active_backup_bond.py | 20 ++++++++++++++++++++
>   ...al_ovs_bridge_2_vlans_over_active_backup_bond.xml |  1 +
>   .../phase2/virtual_ovs_bridge_vlan_in_guest.py       | 12 ++++++++++++
>   .../phase2/virtual_ovs_bridge_vlan_in_guest.xml      |  1 +
>   .../phase2/virtual_ovs_bridge_vlan_in_host.py        | 11 +++++++++++
>   .../phase2/virtual_ovs_bridge_vlan_in_host.xml       |  1 +
>   6 files changed, 46 insertions(+)
>  
>  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 409cb88..c2f3b78 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
>  @@ -33,6 +33,7 @@ g4.sync_resources(modules=["IcmpPing", "Icmp6Ping",
> "Netperf"])
>   # TESTS
>   # ------
>   
>  +mtu = ctl.get_alias("mtu")
>   ipv = ctl.get_alias("ipv")
>   netperf_duration = int(ctl.get_alias("netperf_duration"))
>   nperf_reserve = int(ctl.get_alias("nperf_reserve"))
>  @@ -230,6 +231,25 @@ ping_mod6_bad2 = ctl.get_module("Icmp6Ping",
>                                  "interval" : 0.1
>                              })
>   
>  +
>  +# configure mtu
>  +h1_nic1.set_mtu(mtu)
>  +h1_nic2.set_mtu(mtu)
>  +h1.get_interface("tap1").set_mtu(mtu)
>  +h1.get_interface("tap2").set_mtu(mtu)
>  +h1.get_interface("bridge").set_mtu(mtu)
>  +
>  +h2_nic1.set_mtu(mtu)
>  +h2_nic2.set_mtu(mtu)
>  +h2.get_interface("tap1").set_mtu(mtu)
>  +h2.get_interface("tap2").set_mtu(mtu)
>  +h2.get_interface("bridge").set_mtu(mtu)
>  +
>  +g1.get_interface("guestnic").set_mtu(mtu)
>  +g2.get_interface("guestnic").set_mtu(mtu)
>  +g3.get_interface("guestnic").set_mtu(mtu)
>  +g4.get_interface("guestnic").set_mtu(mtu)
>  +
>   ctl.wait(15)
>   
>   for setting in offload_settings:
>  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 32b4c06..01c4291 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
>  @@ -7,6 +7,7 @@
>           <alias name="nperf_max_runs" value="5" />
>           <alias name="nperf_mode" value="default"/>
>           <alias name="nperf_num_parallel" value="2"/>
>  +        <alias name="mtu" value="1500" />
>           <alias name="nperf_debug" value="0"/>
>           <alias name="nperf_max_dev" value="20%"/>
>           <alias name="mapping_file"
> value="virtual_ovs_bridge_2_vlans_over_active_backup_bond.mapping" />
>  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 b0dc574..812bc27 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
>  @@ -27,6 +27,7 @@ h2.sync_resources(modules=["IcmpPing", "Icmp6Ping",
> "Netperf"])
>   # TESTS
>   # ------
>   
>  +mtu = ctl.get_alias("mtu")
>   ipv = ctl.get_alias("ipv")
>   netperf_duration = int(ctl.get_alias("netperf_duration"))
>   nperf_reserve = int(ctl.get_alias("nperf_reserve"))
>  @@ -178,6 +179,17 @@ if nperf_udp_size is not None:
>       netperf_cli_udp.update_options({"udp_size" : nperf_udp_size})
>       netperf_cli_udp6.update_options({"udp_size" : nperf_udp_size})
>   
>  +#set mtu
>  +h1.get_interface("nic").set_mtu(mtu)
>  +h1.get_interface("tap").set_mtu(mtu)
>  +h1.get_interface("br").set_mtu(mtu)
>  +
>  +h2.get_interface("nic").set_mtu(mtu)
>  +h2.get_interface("vlan10").set_mtu(mtu)
>  +
>  +g1.get_interface("guestnic").set_mtu(mtu)
>  +g1.get_interface("vlan10").set_mtu(mtu)
>  +
>   ctl.wait(15)
>   
>   for setting in offload_settings:
>  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 ae85997..786b9d3 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
>  @@ -9,6 +9,7 @@
>           <alias name="nperf_num_parallel" value="2"/>
>           <alias name="nperf_debug" value="0"/>
>           <alias name="nperf_max_dev" value="20%"/>
>  +        <alias name="mtu" value="1500" />
>           <alias name="mapping_file"
> value="virtual_ovs_bridge_vlan_in_guest.mapping" />
>           <alias name="vlan10_net" value="192.168.10"/>
>           <alias name="vlan10_tag" value="10"/>
>  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 25f19f1..69560b9 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
>  @@ -27,6 +27,7 @@ h2.sync_resources(modules=["IcmpPing", "Icmp6Ping",
> "Netperf"])
>   # TESTS
>   # ------
>   
>  +mtu = ctl.get_alias("mtu")
>   ipv = ctl.get_alias("ipv")
>   netperf_duration = int(ctl.get_alias("netperf_duration"))
>   nperf_reserve = int(ctl.get_alias("nperf_reserve"))
>  @@ -177,6 +178,16 @@ if nperf_udp_size is not None:
>       netperf_cli_udp.update_options({"udp_size" : nperf_udp_size})
>       netperf_cli_udp6.update_options({"udp_size" : nperf_udp_size})
>   
>  +#set mtu
>  +h1.get_interface("nic").set_mtu(mtu)
>  +h1.get_interface("tap").set_mtu(mtu)
>  +h1.get_interface("ovs_br").set_mtu(mtu)
>  +
>  +h2.get_interface("nic").set_mtu(mtu)
>  +h2.get_interface("vlan10").set_mtu(mtu)
>  +
>  +g1.get_interface("guestnic").set_mtu(mtu)
>  +
>   ctl.wait(15)
>   
>   for setting in offload_settings:
>  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 6b2a271..5f8b9e5 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
>  @@ -9,6 +9,7 @@
>           <alias name="nperf_num_parallel" value="2"/>
>           <alias name="nperf_debug" value="0"/>
>           <alias name="nperf_max_dev" value="20%"/>
>  +        <alias name="mtu" value="1500" />
>           <alias name="mapping_file"
> value="virtual_ovs_bridge_vlan_in_host.mapping" />
>           <alias name="vlan10_net" value="192.168.10"/>
>           <alias name="vlan10_tag" value="10"/>
>  -- 
>  2.5.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