- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Update README.md
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Update README.md ...................................................................... Update README.md This patch updates the README.md, as it wasn't updated when the "_service" suffix was added to locator methods. Change-Id: I79d3cb04a9fd6c7893200594178624899eff6b25 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/README.md 1 file changed, 8 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52488 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79d3cb04a9fd6c7893200594178624899eff6b25 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add version header
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add version header ...................................................................... Add version header This patch changes the "Connection" class so that it will always send the header that indicates the requested version of the API, which for this version of the SDK will always be "4". The header looks like this: Version: 4 Change-Id: I65fd750d28bd279dd6c153c2f440ea5e929260f0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirt/sdk/v4/connection.rb 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52487 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65fd750d28bd279dd6c153c2f440ea5e929260f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Custom user agent
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Custom user agent ...................................................................... Custom user agent This patch changes the SDK "Connection" class so that it will always send the "User-Agent" header containing the version number of the SDK, something like this: User-Agent: RubySDK/4.0.0.alpha3 Change-Id: I983b1c131723f93aa6f53902ca79bc617df82dc1 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirt/sdk/v4/connection.rb 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52486 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I983b1c131723f93aa6f53902ca79bc617df82dc1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add "_service" to all locator methods
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add "_service" to all locator methods ...................................................................... Add "_service" to all locator methods Currently the locator methods are named like in the specification of the API. For example, the method used to find the virtual machines service is named "vms". This patch changes those methods so that they will have a "_service" suffix, making it clearer that they return references to services, not objects. Change-Id: If3dacbd895fda86b9672643bb045fd23cf2203a0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java M sdk/examples/add_cluster.rb M sdk/examples/add_data_center.rb M sdk/examples/add_host.rb M sdk/examples/add_nfs_data_storage_domain.rb M sdk/examples/add_nfs_iso_storage_domain.rb M sdk/examples/add_vm.rb M sdk/examples/add_vm_disk.rb M sdk/examples/add_vm_nic.rb M sdk/examples/attach_nfs_data_storage_domain.rb M sdk/examples/attach_nfs_iso_storage_domain.rb M sdk/examples/follow_vm_links.rb M sdk/examples/list_vms.rb M sdk/examples/remove_host.rb M sdk/examples/remove_vm.rb M sdk/examples/search_vms.rb M sdk/examples/start_vm.rb M sdk/examples/stop_vm.rb M sdk/examples/update_data_center.rb M sdk/lib/ovirt/sdk/v4/connection.rb M sdk/spec/clusters_service_spec.rb M sdk/spec/data_centers_service_spec.rb M sdk/spec/storage_domains_service_spec.rb M sdk/spec/vms_service_spec.rb 24 files changed, 68 insertions(+), 68 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52485 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3dacbd895fda86b9672643bb045fd23cf2203a0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Fix link following
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Fix link following ...................................................................... Fix link following This patch fixes the "Connection.url" method. It needs to return a URI object, not a string. Without this fix the link following capability doesn't work correctly. In addition the patch also adds an example that shows how to use the link following capability. Change-Id: I9bd82acb0ae6de5f6f2110eb3fdcf6a89dfada61 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/follow_vm_links.rb M sdk/lib/ovirt/sdk/v4/connection.rb 2 files changed, 69 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52479 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9bd82acb0ae6de5f6f2110eb3fdcf6a89dfada61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.2]: packaging: moving the ovirt extentions api dependencies to h...
by tnisan＠redhat.com 20 Jan '16

20 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: moving the ovirt extentions api dependencies to his own line ...................................................................... packaging: moving the ovirt extentions api dependencies to his own line - since the ovirt extentions api is the only depenency that has -SNAPSHOT, it makes sense to emphasis it in a seperate section to reduce risk on missing it for official releases and also reduce possibility of rebase conflicts with downstream code when one dependency changes Change-Id: I66d1ec3cd7ed71a507f56c86bd55d2d82231f9a0 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M pom.xml 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Eyal Edri: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52447 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66d1ec3cd7ed71a507f56c86bd55d2d82231f9a0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.2 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: moving the ovirt extentions api dependencies to h...
by tnisan＠redhat.com 20 Jan '16

20 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: moving the ovirt extentions api dependencies to his own line ...................................................................... packaging: moving the ovirt extentions api dependencies to his own line - since the ovirt extentions api is the only depenency that has -SNAPSHOT, it makes sense to emphasis it in a seperate section to reduce risk on missing it for official releases and also reduce possibility of rebase conflicts with downstream code when one dependency changes Change-Id: I66d1ec3cd7ed71a507f56c86bd55d2d82231f9a0 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M pom.xml 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Eyal Edri: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52446 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66d1ec3cd7ed71a507f56c86bd55d2d82231f9a0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine-sdk-ruby[master]: Improve error handling
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Improve error handling ...................................................................... Improve error handling This patch improves the error handling of the SDK, so that services will check the results received from the server and transform them into the corresponding exceptions. Change-Id: I55903e5278d3405c214e5c2380df9d389f780391 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ReadersGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/RubyNames.java M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/TypesGenerator.java M sdk/examples/add_cluster.rb M sdk/examples/add_data_center.rb M sdk/examples/add_host.rb M sdk/examples/add_nfs_data_storage_domain.rb M sdk/examples/add_nfs_iso_storage_domain.rb M sdk/examples/add_vm.rb M sdk/examples/add_vm_disk.rb M sdk/examples/add_vm_nic.rb M sdk/examples/attach_nfs_data_storage_domain.rb M sdk/examples/attach_nfs_iso_storage_domain.rb M sdk/examples/list_vms.rb A sdk/examples/remove_host.rb M sdk/examples/remove_vm.rb M sdk/examples/search_vms.rb M sdk/examples/start_vm.rb M sdk/examples/stop_vm.rb A sdk/examples/update_data_center.rb A sdk/lib/ovirt/sdk/v4/readers/action_reader.rb M sdk/lib/ovirt/sdk/v4/services/service.rb A sdk/lib/ovirt/sdk/v4/types/action.rb A sdk/spec/action_reader_spec.rb A sdk/spec/service_spec.rb 26 files changed, 683 insertions(+), 170 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52459 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55903e5278d3405c214e5c2380df9d389f780391 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add fault class and reader
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add fault class and reader ...................................................................... Add fault class and reader This patch adds a classes to represent and read "fault" objects returned by the server. Change-Id: Id2c96b9550aacad7709d118840362a0516c4745d Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ReadersGenerator.java M generator/src/main/java/org/ovirt/sdk/ruby/RubyNames.java M generator/src/main/java/org/ovirt/sdk/ruby/TypesGenerator.java A sdk/lib/ovirt/sdk/v4/readers/fault_reader.rb A sdk/lib/ovirt/sdk/v4/types/fault.rb A sdk/spec/fault_reader_spec.rb 6 files changed, 193 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52458 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2c96b9550aacad7709d118840362a0516c4745d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add request and response classes
by juan.hernandez＠redhat.com 20 Jan '16

20 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add request and response classes ...................................................................... Add request and response classes This patch adds two new "Request" and "Response" classes. These classes are intended to carry the details of HTTP requests and responses. With them the "Connection.send" method can return a response, containing the HTTP response code, message and body, so that other parts of the SDK will in the future be able to inspect them and act accordingly. Change-Id: I1801639bea2e419f956133e070b7f22dee24985d Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java M sdk/lib/ovirt/sdk/v4.rb M sdk/lib/ovirt/sdk/v4/connection.rb M sdk/spec/connection_spec.rb 4 files changed, 148 insertions(+), 109 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52409 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1801639bea2e419f956133e070b7f22dee24985d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0