Ravi Nori has uploaded a new change for review.

Change subject: cli: Update API fails in SDK & CLI for several entities
......................................................................

cli: Update API fails in SDK & CLI for several entities

All update operations fail from cli and sdk

Change-Id: I6d48a971ab50e9ff580d9624de3f317f180c48f6
Bug-Url: https://bugzilla.redhat.com/1122546
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M src/ovirtcli/command/update.py
1 file changed, 4 insertions(+), 3 deletions(-)


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

diff --git a/src/ovirtcli/command/update.py b/src/ovirtcli/command/update.py
index cfa8b55..99d8def 100644
--- a/src/ovirtcli/command/update.py
+++ b/src/ovirtcli/command/update.py
@@ -156,7 +156,7 @@
                Messages.Error.NO_SUCH_OBJECT % (args[0], args[1])
             )
         elif hasattr(resource, 'update'):
-            obj = 
self.update_object_data(self.__create_set_superclass(resource), opts)
+            obj = self.__create_set_superclass(resource, opts)
             result = self.execute_method(obj, 'update', opts)
         else:
             self.error(
@@ -164,12 +164,13 @@
             )
         self.context.formatter.format(self.context, result)
 
-    def __create_set_superclass(self, resource):
+    def __create_set_superclass(self, resource, opts):
         """Create an instance of param object."""
         param_obj = type(resource.superclass).factory()
         if hasattr(param_obj, 'id'):
             setattr(param_obj, 'id', getattr(resource, 'id'))
-        resource.superclass = param_obj
+        obj = self.update_object_data(param_obj, opts)
+        resource.superclass = obj
         return resource
 
     def show_help(self):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6d48a971ab50e9ff580d9624de3f317f180c48f6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to