Change in ovirt-engine-sdk-ruby[master]: Use "Action" type

juan.hernandez at redhat.com juan.hernandez at redhat.com
Thu Apr 28 11:16:33 UTC 2016


Juan Hernandez has submitted this change and it was merged.

Change subject: Use "Action" type
......................................................................


Use "Action" type

Currently when the SDK needs to perform an action, like starting a
virtual machine, the corresponding service class generates directly the
"<action>...</action>" XML text, and it doesn't extract the output
parameters of some actions. This patch changes it so that instead of
generating directly the XML text it creates an instance of the "Action"
struct, populates it, and then generates the XML text using the
"ActionWriter" class. In also modifies the SDK so that it takes the
returned action and extracts the output parameters.

Change-Id: If2e254cf2b89baa91aaa13a058274a0c0c6b02a0
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java
M sdk/lib/ovirtsdk4/service.rb
M sdk/spec/spec_helper.rb
A sdk/spec/storage_domain_service_spec.rb
A sdk/spec/vm_service_spec.rb
5 files changed, 121 insertions(+), 47 deletions(-)

Approvals:
  Ondra Machacek: Looks good to me, but someone else must approve
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/56737
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If2e254cf2b89baa91aaa13a058274a0c0c6b02a0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-sdk-ruby
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list