August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Add example of import of external VM
by juan.hernandez＠redhat.com 31 Aug '16

31 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add example of import of external VM ...................................................................... Add example of import of external VM Added example code for importing a virtual machine from VmWare. Change-Id: I815d8c702a7b709027d4732fcf3592aa478bd259 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- A sdk/examples/import_external_vm.rb 1 file changed, 56 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Martin Betak: Verified -- To view, visit https://gerrit.ovirt.org/63060 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I815d8c702a7b709027d4732fcf3592aa478bd259 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Avoid of reading fault response in case response body is empty
by omachace＠redhat.com 31 Aug '16

31 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Avoid of reading fault response in case response body is empty ...................................................................... Avoid of reading fault response in case response body is empty In case there is 404 returned from API, we try to parse response body, but in this case it's useless and returned exception previously, this patch changes it, so we don't parse response in case it's empty. Change-Id: Ia3c6e7bee7a1926cc007db9477640d675487dbe6 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/lib/ovirtsdk4/service.py 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63058 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3c6e7bee7a1926cc007db9477640d675487dbe6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master 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[master]: core: HttpImageTaskVDSCommand - problematic type inference
by laravot＠redhat.com 31 Aug '16

31 Aug '16

Liron Aravot has submitted this change and it was merged. Change subject: core: HttpImageTaskVDSCommand - problematic type inference ...................................................................... core: HttpImageTaskVDSCommand - problematic type inference In change Iaed648bcf979d682389a4df92, the explicit Integer type was removed from the call to Config.getValue(), that causes the type to be infered as Long and fails on ClassCastException. When the type is explictly specified, the returned value is an Integer which is outboxed and can be passed to future.get(). Change-Id: I349a08a7d119bcc57f8f6a88331c9ea64790cc99 Bug-Url: https://bugzilla.redhat.com/1371833 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/HttpImageTaskVDSCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Liron Aravot: Verified -- To view, visit https://gerrit.ovirt.org/63053 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I349a08a7d119bcc57f8f6a88331c9ea64790cc99 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.4]: webadmin: Fix multi selection deletion in UI
by tnisan＠redhat.com 31 Aug '16

31 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix multi selection deletion in UI ...................................................................... webadmin: Fix multi selection deletion in UI Deletion of multiple selected items from a table view in UI resulted in an uncaught UI exception (out of bounds). This could have been observed in multiple disk profiles deletion multiple CPU profiles and network interfaces deletion. Multiple items deletion from a table view (using SHIFT + multiple selection) will not result in an exception. Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34 Bug-Url: https://bugzilla.redhat.com/1359483 Signed-off-by: yquinn <yquinn(a)redhat.com> (cherry picked from commit bc96835) Signed-off-by: yquinn <yquinn(a)redhat.com> (cherry picked from commit 0baad49354926623fc93e72b5e955bd4e3eb9cdd) --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/OrderedMultiSelectionModel.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Yanir Quinn: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/63050 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.4 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-4.0]: build: oVirt Live 4.0.4 RC1
by sbonazzo＠redhat.com 31 Aug '16

31 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: oVirt Live 4.0.4 RC1 ...................................................................... build: oVirt Live 4.0.4 RC1 Change-Id: I164d665a615a9e2bf4b482c5ff0f667b8f980bf9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/63049 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I164d665a615a9e2bf4b482c5ff0f667b8f980bf9 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[ovirt-engine-4.0]: webadmin: Fix multi selection deletion in UI
by tnisan＠redhat.com 31 Aug '16

31 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix multi selection deletion in UI ...................................................................... webadmin: Fix multi selection deletion in UI Deletion of multiple selected items from a table view in UI resulted in an uncaught UI exception (out of bounds). This could have been observed in multiple disk profiles deletion multiple CPU profiles and network interfaces deletion. Multiple items deletion from a table view (using SHIFT + multiple selection) will not result in an exception. Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34 Bug-Url: https://bugzilla.redhat.com/1359483 Signed-off-by: yquinn <yquinn(a)redhat.com> (cherry picked from commit bc96835) Signed-off-by: yquinn <yquinn(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/OrderedMultiSelectionModel.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Yanir Quinn: Verified -- To view, visit https://gerrit.ovirt.org/62724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib15be1a6474b9504053da0940825c23224d6fd34 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.0]: Check the type of debug data
by omachace＠redhat.com 31 Aug '16

31 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Check the type of debug data ...................................................................... Check the type of debug data Apparently some versions of PycURL provide the debug data as strings, and others as arrays of bytes. When they are arrays of bytes we need to convert them to strings before manipulating them with the 'replace', 'strip' and 'split' methods. Change-Id: I43715da41af2971198b5a0ffc6e42a656ed4694f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit f886e01d69f58c376167944eb0a82ee8a124456c) --- M sdk/lib/ovirtsdk4/__init__.py 1 file changed, 10 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63044 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43715da41af2971198b5a0ffc6e42a656ed4694f 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]: Check the type of debug data
by omachace＠redhat.com 31 Aug '16

31 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Check the type of debug data ...................................................................... Check the type of debug data Apparently some versions of PycURL provide the debug data as strings, and others as arrays of bytes. When they are arrays of bytes we need to convert them to strings before manipulating them with the 'replace', 'strip' and 'split' methods. Change-Id: I43715da41af2971198b5a0ffc6e42a656ed4694f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/__init__.py 1 file changed, 10 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63035 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43715da41af2971198b5a0ffc6e42a656ed4694f Gerrit-PatchSet: 1 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-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: build: post ovirt-hosted-engine-setup-2.0.2
by stirabos＠redhat.com 30 Aug '16

30 Aug '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-2.0.2 ...................................................................... build: post ovirt-hosted-engine-setup-2.0.2 Change-Id: Iddbffbcf1285844adb9f1cff8fd51ba728c0982a Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63017 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddbffbcf1285844adb9f1cff8fd51ba728c0982a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: build: ovirt-hosted-engine-setup-2.0.2
by stirabos＠redhat.com 30 Aug '16

30 Aug '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-2.0.2 ...................................................................... build: ovirt-hosted-engine-setup-2.0.2 Change-Id: Ie9051354d2ebeee27be3e9b5027dd0c192af594d Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 11 insertions(+), 11 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63016 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9051354d2ebeee27be3e9b5027dd0c192af594d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0