diff options
| author | Andy Grover <agrover@redhat.com> | 2014-11-11 12:39:54 -0800 |
|---|---|---|
| committer | Andy Grover <agrover@redhat.com> | 2014-11-11 12:39:54 -0800 |
| commit | b7a85ef8bcbd2c4337f17d86a3f0d3c3f4e2b364 (patch) | |
| tree | e2ce69211ca889fd94e0668ff00e3a0c7f970408 | |
| parent | a3182301b10f1a2be0f26d9d6204021a4b8deac5 (diff) | |
| download | targetcli-b7a85ef8bcbd2c4337f17d86a3f0d3c3f4e2b364.tar.gz | |
Fix configuration tree inconsistencyfix-dir-del-issue
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 <emurphy@lessorsinc.com>
Reported-by: Leeman Strout <me@mooluv.com>
Signed-off-by: Andy Grover <agrover@redhat.com>
| -rw-r--r-- | targetcli/ui_backstore.py | 1 |
1 files changed, 0 insertions, 1 deletions
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): ''' |