commit 90a8efda33a17f30a2c579ac2bcc01142e1cba5b Author: Ondrej Lichtner olichtne@redhat.com Date: Mon Sep 29 16:08:27 2014 +0200
NetTestSlave: don't remove netns in the bye method
Removing network namespaces in the "bye" method removes part of the configuration which shouldn't happen on config_only runs. This patch fixes that. Instead we remove network namespaces in machine_cleanup which is called by the "hello" method.
Signed-off-by: Ondrej Lichtner olichtne@redhat.com Signed-off-by: Jiri Pirko jiri@resnulli.us
lnst/Slave/NetTestSlave.py | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) --- diff --git a/lnst/Slave/NetTestSlave.py b/lnst/Slave/NetTestSlave.py index eeb0337..14d1d48 100644 --- a/lnst/Slave/NetTestSlave.py +++ b/lnst/Slave/NetTestSlave.py @@ -99,10 +99,6 @@ class SlaveMethods: self._cache.del_old_entries() self.reset_file_transfers() self._remove_capture_files() - - for netns in self._net_namespaces.keys(): - self.del_namespace(netns) - self._net_namespaces = {} return "bye"
def kill_cmds(self):
lnst-developers@lists.fedorahosted.org