Michael Pasternak has posted comments on this change.

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


Patch Set 1:

indeed, two questions:

1. how did you created the datacenter?
(manually/fetched by datacenters.get())

2. did you changed the content of datacenter
or is it comes from server this way?

- if it comes this way from the server, the bug in de-serialization.

- if it assigns attributes to self rather than to self.superclass when you 
update datacenter content,
the bug is in __setattr__ and IIRC only patch you reverted
changed it (please look deeper in change log, - i may be wrong)

in any case this is regression, to find the problem, you should examine latest 
changes in infrastructure.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf51e314e2e8381c882031e749780811a8e14b16
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine-sdk
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mishka8...@yahoo.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to