diff options
| author | Jerome Martin <jxm@risingtidesystems.com> | 2011-05-31 23:17:03 +0200 |
|---|---|---|
| committer | Jerome Martin <jxm@risingtidesystems.com> | 2011-05-31 23:19:29 +0200 |
| commit | 10e5bee057616dfbb01fc59724a46f53c7df36ba (patch) | |
| tree | bfe9e5758df29382bf58df89e04019fefc7aead6 | |
| parent | b91a6142ac9b146966049b49844a58ef07ecf5a6 (diff) | |
| download | targetcli-10e5bee057616dfbb01fc59724a46f53c7df36ba.tar.gz | |
Fixed regression in 1d55a9f (catch RTSLibNotInCFS)
* An abusive catch of RTSLibNotInCFS in execute_command was retrying the
command endlessly, which is bad on general principle but even worse
when the RTSLibNotInCFS is legit and should get through (i.e. trying
to delete a non-existent MappedLUN would result in and endless loop).
| -rw-r--r-- | rtsadmin/ui_node.py | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/rtsadmin/ui_node.py b/rtsadmin/ui_node.py index 749b38b..04b90cf 100644 --- a/rtsadmin/ui_node.py +++ b/rtsadmin/ui_node.py @@ -91,10 +91,6 @@ class UINode(ConfigNode): try: result = ConfigNode.execute_command(self, command, pparams, kparams) - except RTSLibNotInCFS: - self.log.debug("Detected configFS tree changes, refreshing.") - self.get_root().refresh() - return self.execute_command(command, pparams, kparams) except RTSLibError, msg: self.log.error(msg) else: |