From b7a85ef8bcbd2c4337f17d86a3f0d3c3f4e2b364 Mon Sep 17 00:00:00 2001 From: Andy Grover Date: Tue, 11 Nov 2014 12:39:54 -0800 Subject: Fix configuration tree inconsistency Deleting a storage object refreshes the root. This causes the root to release its references to its children and regenerate them. However if we are cd'd to a node such as /backstores/ramdisk, then the objects we are using are the old ones, not the new ones connected to the root node, and creating a new storage object will update our tree but not root's since we've become disconnected. Fix this by no longer refreshing the root -- it should not require refreshing anyways. Reported-by: Eric Murphy Reported-by: Leeman Strout Signed-off-by: Andy Grover --- targetcli/ui_backstore.py | 1 - 1 file changed, 1 deletion(-) diff --git a/targetcli/ui_backstore.py b/targetcli/ui_backstore.py index 227a91e..f098fd9 100644 --- a/targetcli/ui_backstore.py +++ b/targetcli/ui_backstore.py @@ -151,7 +151,6 @@ class UIBackstore(UINode): child.rtsnode.delete() self.remove_child(child) self.shell.log.info("Deleted storage object %s." % name) - self.parent.parent.refresh() def ui_complete_delete(self, parameters, text, current_param): ''' -- cgit v1.2.1