At the end of the test we turn on all of the tested offloads. There was a typo which reset only the last offload in the set.
Signed-off-by: Jan Tluka jtluka@redhat.com --- recipes/regression_tests/phase1/virtual_bridge_2_vlans_over_bond.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
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 6b79dff..e2a9449 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 @@ -378,7 +378,7 @@ for setting in offload_settings: #reset offload states dev_features = "" for offload in offloads: - dev_features = " %s %s" % (offload, "on") + dev_features += " %s %s" % (offload, "on") h1.run("ethtool -K %s %s" % (h1_nic1.get_devname(), dev_features)) h1.run("ethtool -K %s %s" % (h1_nic2.get_devname(), dev_features)) h2.run("ethtool -K %s %s" % (h2_nic1.get_devname(), dev_features))
lnst-developers@lists.fedorahosted.org