LNST-developers February 2014

lnst-developers@lists.fedorahosted.org

4 participants
43 discussions

[lnst] NetTestController: fix typo in log message
by Jiří Pírko 10 Feb '14

10 Feb '14

commit 49b73e130fc8bed16b185a321b9ca13f6223c348 Author: Ondrej Lichtner <olichtne(a)redhat.com> Date: Thu Feb 6 13:30:48 2014 +0100 NetTestController: fix typo in log message Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com> Signed-off-by: Jiri Pirko <jiri(a)resnulli.us> lnst/Controller/NetTestController.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) --- diff --git a/lnst/Controller/NetTestController.py b/lnst/Controller/NetTestController.py index d1ef7ea..c646b51 100644 --- a/lnst/Controller/NetTestController.py +++ b/lnst/Controller/NetTestController.py @@ -448,7 +448,7 @@ class NetTestController: return if cfg: - logging.info("Cleaning ip leftover configuration from previous "\ + logging.info("Cleaning up leftover configuration from previous "\ "virtualized config_only run.") for hostname, machine in cfg["machines"].iteritems(): port = lnst_config.get_option("environment", "rpcport")

1 0

[PATCH] NetTestSlave: fix ctl_clean_exit typos
by Ondrej Lichtner 06 Feb '14

06 Feb '14

From: Ondrej Lichtner <olichtne(a)redhat.com> The internal variable ctl_clean_exit in the SlaveMethods class was incorrectly used a few types, python didn't pick this up because of it's dynamic nature... This however caused that the cleanup didn't work properly. This commit fixes that. Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com> --- lnst/Slave/NetTestSlave.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lnst/Slave/NetTestSlave.py b/lnst/Slave/NetTestSlave.py index f8dad2b..0cd6185 100644 --- a/lnst/Slave/NetTestSlave.py +++ b/lnst/Slave/NetTestSlave.py @@ -61,12 +61,12 @@ class SlaveMethods: self._resource_table = {} - self.ctl_clean_exit = True + self._ctl_clean_exit = True def hello(self, recipe_path): - if not self.ctl_clean_exit: - self._methods.machine_cleanup() - self._methods.ctl_clean_exit = True + if not self._ctl_clean_exit: + self.machine_cleanup() + self._ctl_clean_exit = True logging.info("Recieved a controller connection.") self.clear_resource_table() @@ -94,7 +94,7 @@ class SlaveMethods: self._cache.del_old_entries() self.reset_file_transfers() self._remove_capture_files() - self.ctl_clean_exit = True + self._ctl_clean_exit = True return "bye" def kill_cmds(self): -- 1.8.5.3

1 0

[PATCH] NetTestController: fix typo in log message
by Ondrej Lichtner 06 Feb '14

06 Feb '14

From: Ondrej Lichtner <olichtne(a)redhat.com> Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com> --- lnst/Controller/NetTestController.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lnst/Controller/NetTestController.py b/lnst/Controller/NetTestController.py index d1ef7ea..c646b51 100644 --- a/lnst/Controller/NetTestController.py +++ b/lnst/Controller/NetTestController.py @@ -448,7 +448,7 @@ class NetTestController: return if cfg: - logging.info("Cleaning ip leftover configuration from previous "\ + logging.info("Cleaning up leftover configuration from previous "\ "virtualized config_only run.") for hostname, machine in cfg["machines"].iteritems(): port = lnst_config.get_option("environment", "rpcport") -- 1.8.5.3

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

LNST-developers February 2014