Michael Pasternak has uploaded a new change for review.

Change subject: cli: wrong response for suspend VM #886944
......................................................................

cli: wrong response for suspend VM #886944

https://bugzilla.redhat.com/show_bug.cgi?id=886944

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


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

diff --git a/src/ovirtcli/command/action.py b/src/ovirtcli/command/action.py
index 8996732..993de33 100644
--- a/src/ovirtcli/command/action.py
+++ b/src/ovirtcli/command/action.py
@@ -177,7 +177,7 @@
                 result = self.execute_method(resource, args[2], opts)
             except Exception, e:
                 self.error(str(e))
-            if result.status.state != 'complete':
+            if result.status.state == 'failed':
                 self.error(Messages.Info.ACTION_STATUS % result.status.state)
         else:
             self.error(Messages.Error.NO_SUCH_ACTION % args[2])
@@ -197,7 +197,6 @@
 
                 subst['type'] = args[0]
                 subst['id'] = args[1]
-                #subst['action'] = args[0]
 
                 base = self.resolve_base(self.options)
                 obj = self.get_object(args[0], args[1], base, 
context_variants=types[args[0]])


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I345b2bd1d62a06fef7671704eda70dc620755bcf
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