July 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[sdk_4.0]: [maven-release-plugin] prepare release 4.0.0.alpha18
by juan.hernandez＠redhat.com 29 Jul '16

29 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare release 4.0.0.alpha18 ...................................................................... [maven-release-plugin] prepare release 4.0.0.alpha18 Change-Id: Iaf53f2a499b9117fb3633810065c89686d803895 --- M generator/pom.xml M pom.xml M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61679 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf53f2a499b9117fb3633810065c89686d803895 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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[sdk_4.0]: Update CHANGES.adoc for release 4.0.0.alpha18
by juan.hernandez＠redhat.com 29 Jul '16

29 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Update CHANGES.adoc for release 4.0.0.alpha18 ...................................................................... Update CHANGES.adoc for release 4.0.0.alpha18 This patch updates the CHANGES.adoc file, documenting the relevant changes in release 4.0.0.alpha18. Change-Id: I5ae446bb2d7ee098ac0497ee64525833c5b9d7ba Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/CHANGES.adoc 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61678 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ae446bb2d7ee098ac0497ee64525833c5b9d7ba Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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-live[ovirt-live-4.0]: centos7: kickstart: use 4.0 release repo instead of candidate
by sbonazzo＠redhat.com 29 Jul '16

29 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: centos7: kickstart: use 4.0 release repo instead of candidate ...................................................................... centos7: kickstart: use 4.0 release repo instead of candidate Change-Id: Ia1e0951421502954d0e957a517e30307fed327d3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61676 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia1e0951421502954d0e957a517e30307fed327d3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Handle correctly error responses without body
by juan.hernandez＠redhat.com 29 Jul '16

29 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Handle correctly error responses without body ...................................................................... Handle correctly error responses without body Currently when the SDK receives a response with an error code but without a body it generates an incorrect exception because it tries to create an XML reader that wasn't actually created. To avoid that this patch changes the SDK so that it checks if the response body is empty, and in that case it doesn't try to create or close the XML reader. Change-Id: If898a44b964bce3944e008dc8e10a2866f180452 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/service.rb M sdk/spec/service_spec.rb 2 files changed, 50 insertions(+), 5 deletions(-) Approvals: Boris Odnopozov: 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/61657 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If898a44b964bce3944e008dc8e10a2866f180452 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(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[sdk_4.0]: Handle correctly error responses without body
by juan.hernandez＠redhat.com 29 Jul '16

29 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: Handle correctly error responses without body ...................................................................... Handle correctly error responses without body Currently when the SDK receives a response with an error code but without a body it generates an incorrect exception because it tries to create an XML reader that wasn't actually created. To avoid that this patch changes the SDK so that it checks if the response body is empty, and in that case it doesn't try to create or close the XML reader. Change-Id: If898a44b964bce3944e008dc8e10a2866f180452 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/service.rb M sdk/spec/service_spec.rb 2 files changed, 50 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61675 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If898a44b964bce3944e008dc8e10a2866f180452 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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-release[ovirt-4.0]: el7: move SIG repo to release
by sbonazzo＠redhat.com 29 Jul '16

29 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: el7: move SIG repo to release ...................................................................... el7: move SIG repo to release Change-Id: Id9a9cfbaf76b0d90ae4bc33a11147a0352efdf4f --- M configure.ac M ovirt-el7-deps.repo.in M ovirt-release40.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61674 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9a9cfbaf76b0d90ae4bc33a11147a0352efdf4f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: cockpit-ovirt: move master to fc24
by sbonazzo＠redhat.com 29 Jul '16

29 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: cockpit-ovirt: move master to fc24 ...................................................................... cockpit-ovirt: move master to fc24 Change-Id: I5d450c99be24272bdc135f045ada90b78bbe057b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/cockpit-ovirt/cockpit-ovirt.yaml 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61672 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d450c99be24272bdc135f045ada90b78bbe057b Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.0]: Use Python logging
by omachace＠redhat.com 29 Jul '16

29 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use Python logging ...................................................................... Use Python logging Currently the SDK writes its debug output directly to a file. This isn't convenient when he SDK is integrated into other applications that use the standard Python "logging" module. To simplify that integration this patch changes the SDK so that it uses the "logging" module. Change-Id: Ifeefb5b4b75d5afb78149df0579493b8b2618538 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 3c9e04be88c069aad28f59f857500b2b55a5f9ce) --- M sdk/examples/add_cluster.py M sdk/examples/add_data_center.py M sdk/examples/add_host.py M sdk/examples/add_nfs_data_storage_domain.py M sdk/examples/add_nfs_iso_storage_domain.py M sdk/examples/add_vm.py M sdk/examples/add_vm_disk.py M sdk/examples/add_vm_nic.py M sdk/examples/add_vm_snapshot.py M sdk/examples/attach_nfs_data_storage_domain.py M sdk/examples/attach_nfs_iso_storage_domain.py M sdk/examples/connection_builder.py M sdk/examples/enable_compression.py M sdk/examples/follow_vm_links.py M sdk/examples/list_vm_disks.py M sdk/examples/list_vms.py M sdk/examples/page_vms.py M sdk/examples/remove_host.py M sdk/examples/remove_vm.py M sdk/examples/search_vms.py M sdk/examples/set_vm_serial_number.py M sdk/examples/show_summary.py M sdk/examples/start_vm.py M sdk/examples/stop_vm.py M sdk/examples/test_connection.py M sdk/examples/update_data_center.py M sdk/lib/ovirtsdk4/__init__.py M sdk/tests/test_connection_create.py 28 files changed, 132 insertions(+), 50 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifeefb5b4b75d5afb78149df0579493b8b2618538 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Use Python logging
by omachace＠redhat.com 29 Jul '16

29 Jul '16

Ondra Machacek has submitted this change and it was merged. Change subject: Use Python logging ...................................................................... Use Python logging Currently the SDK writes its debug output directly to a file. This isn't convenient when he SDK is integrated into other applications that use the standard Python "logging" module. To simplify that integration this patch changes the SDK so that it uses the "logging" module. Change-Id: Ifeefb5b4b75d5afb78149df0579493b8b2618538 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/examples/add_cluster.py M sdk/examples/add_data_center.py M sdk/examples/add_host.py M sdk/examples/add_nfs_data_storage_domain.py M sdk/examples/add_nfs_iso_storage_domain.py M sdk/examples/add_vm.py M sdk/examples/add_vm_disk.py M sdk/examples/add_vm_nic.py M sdk/examples/add_vm_snapshot.py M sdk/examples/attach_nfs_data_storage_domain.py M sdk/examples/attach_nfs_iso_storage_domain.py M sdk/examples/connection_builder.py M sdk/examples/enable_compression.py M sdk/examples/follow_vm_links.py M sdk/examples/list_vm_disks.py M sdk/examples/list_vms.py M sdk/examples/page_vms.py M sdk/examples/remove_host.py M sdk/examples/remove_vm.py M sdk/examples/search_vms.py M sdk/examples/set_vm_serial_number.py M sdk/examples/show_summary.py M sdk/examples/start_vm.py M sdk/examples/stop_vm.py M sdk/examples/test_connection.py M sdk/examples/update_data_center.py M sdk/lib/ovirtsdk4/__init__.py M sdk/tests/test_connection_create.py 28 files changed, 132 insertions(+), 50 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61662 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifeefb5b4b75d5afb78149df0579493b8b2618538 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine-sdk 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: ImportVmCommandTest mocking type strictness
by amureini＠redhat.com 28 Jul '16

28 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: ImportVmCommandTest mocking type strictness ...................................................................... core: ImportVmCommandTest mocking type strictness Made the mocking a tad stricter by using type-safe matchers such as anyListOf(Class) instead of raw matches such as anyList(). This change makes the test code a bit easier to read, and helps avoid stupid mistakes in the test. Change-Id: Ia5d1340196e04da0a38de15f97166cdefd82a697 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandTest.java 1 file changed, 14 insertions(+), 14 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/61590 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5d1340196e04da0a38de15f97166cdefd82a697 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0