- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[sdk_4.0]: 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> (cherry picked from commit 0e9ae79c0deb5700906adb3d7c5f3aad9d4c967d) --- A sdk/examples/import_external_vm.rb 1 file changed, 56 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63063 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I815d8c702a7b709027d4732fcf3592aa478bd259 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[master]: core: AutoRecoveryManager type inference
by amureini＠redhat.com 31 Aug '16

31 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: AutoRecoveryManager type inference ...................................................................... core: AutoRecoveryManager type inference Change-Id: I8a1e32a2a8cb9a492e68115136f882dd2fe0050a Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62980 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a1e32a2a8cb9a492e68115136f882dd2fe0050a 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: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(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[master]: core: Fix AutoRecoveryManager feature page link
by amureini＠redhat.com 31 Aug '16

31 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix AutoRecoveryManager feature page link ...................................................................... core: Fix AutoRecoveryManager feature page link Change-Id: I27d7a6c03fd4b5995e1ef489cacde16fc1319343 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62981 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27d7a6c03fd4b5995e1ef489cacde16fc1319343 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: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(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[master]: core: Simplify AutoRecoveryManager boolean expression
by amureini＠redhat.com 31 Aug '16

31 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Simplify AutoRecoveryManager boolean expression ...................................................................... core: Simplify AutoRecoveryManager boolean expression Change-Id: I7de30c380ac2f6e4e128b2e0f6468e9d49f6c056 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AutoRecoveryManager.java 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62982 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7de30c380ac2f6e4e128b2e0f6468e9d49f6c056 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: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(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-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