[PATCH 02/12] deconfiguration: restore_system_config before interface deconfiguration

[Ondrej Lichtner]

From: Ondrej Lichtner <olichtne(a)redhat.com&gt;

Since restore_system_config is often tied to interfaces it should
happen before the interfaces are deconfigured and removed. This bug was
revealed through the newly added IntefaceAPI method set_mtu.

Fixes #121

Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com&gt;
---
 lnst/Controller/Machine.py | 2 ++
 lnst/Slave/NetTestSlave.py | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/lnst/Controller/Machine.py b/lnst/Controller/Machine.py
index a7c5ea9..cae66f3 100644
--- a/lnst/Controller/Machine.py
+++ b/lnst/Controller/Machine.py
@@ -232,6 +232,8 @@ class Machine(object):
             for netns in self._namespaces:
                 self._rpc_call_to_netns(netns, "kill_cmds")
 
+            self.restore_system_config()
+
             if deconfigure:
                 ordered_ifaces.reverse()
                 for iface in ordered_ifaces:
diff --git a/lnst/Slave/NetTestSlave.py b/lnst/Slave/NetTestSlave.py
index efcdfdd..b3156cb 100644
--- a/lnst/Slave/NetTestSlave.py
+++ b/lnst/Slave/NetTestSlave.py
@@ -398,6 +398,8 @@ class SlaveMethods:
         logging.info("Performing machine cleanup.")
         self._command_context.cleanup()
 
+        self.restore_system_config()
+
         devs = self._if_manager.get_mapped_devices()
         for if_id, dev in devs.iteritems():
             peer = dev.get_peer()
@@ -416,7 +418,6 @@ class SlaveMethods:
 
         self._if_manager.clear_if_mapping()
         self._cache.del_old_entries()
-        self.restore_system_config()
         self._remove_capture_files()
         return True
 
-- 
2.1.0