Michael Pasternak has uploaded a new change for review.

Change subject: cli: clean context regardless command success
......................................................................

cli: clean context regardless command success

Change-Id: Id50724c29fe100729b544cea8f2a26dbb553c74e
Signed-off-by: Michael pasternak <[email protected]>
---
M src/ovirtcli/command/disconnect.py
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine-cli refs/changes/17/21217/1

diff --git a/src/ovirtcli/command/disconnect.py 
b/src/ovirtcli/command/disconnect.py
index 20fcc23..1afae0c 100644
--- a/src/ovirtcli/command/disconnect.py
+++ b/src/ovirtcli/command/disconnect.py
@@ -46,12 +46,11 @@
             return
         try:
             StateMachine.disconnecting()  # @UndefinedVariable
-
-            self.context._clean_settings()
             connection.disconnect()
             self.context.status = ExecutionContext.OK
         except Exception:
             self.context.status = ExecutionContext.COMMAND_ERROR
         finally:
+            self.context._clean_settings()
             self.context.connection = None
             StateMachine.disconnected()  # @UndefinedVariable


-- 
To view, visit http://gerrit.ovirt.org/21217
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id50724c29fe100729b544cea8f2a26dbb553c74e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Michael Pasternak <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to