- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Add API for import of VMs from external providers
by tnisan＠redhat.com 04 Sep '16

04 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Add API for import of VMs from external providers ...................................................................... restapi: Add API for import of VMs from external providers Added /externalvmimports endpoint that can be used to import VMs from external providers, such as XEN, KVM or VMware. Change-Id: I1569faab4c52268f2405f49edea51dbe721adb7c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1346847 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmFromExternalUrlCommand.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ImportVmFromExternalUrlParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendExternalVmImportsResource.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendExternalVmImportsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 11 files changed, 710 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Martin Betak: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63145 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1569faab4c52268f2405f49edea51dbe721adb7c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: 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]: core: Always fix VMs status during fencing if host if off
by tnisan＠redhat.com 04 Sep '16

04 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Always fix VMs status during fencing if host if off ...................................................................... core: Always fix VMs status during fencing if host if off Always fix VMs status during fencing flow when host is off (detected by power management status command) and don't rely on successful execution of power management start of the host. Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Bug-Url: https://bugzilla.redhat.com/1367557 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/RestartVdsCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63227 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.4 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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]: core: Fix upgrade issue
by tnisan＠redhat.com 04 Sep '16

04 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix upgrade issue ...................................................................... core: Fix upgrade issue Fixes upgrade issue by copying original 03_06_1790_add_vm_auto_login_column_to_user_profiles_table.sql (which was by mistake not backported into ovirt-engine-3.6 branch during 3.6 version development) into 04_00_0880_add_vm_auto_login_column_to_user_profiles_table.sql. This will fix the issue for upgrade from 3.6 to 4.0. Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Bug-Url: https://bugzilla.redhat.com/1371515 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- A packaging/dbscripts/upgrade/04_00_0880_add_vm_auto_login_column_to_user_profiles_table.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63202 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.4 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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]: core: Fix upgrade issue
by tnisan＠redhat.com 04 Sep '16

04 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix upgrade issue ...................................................................... core: Fix upgrade issue Fixes upgrade issue by copying original 03_06_1790_add_vm_auto_login_column_to_user_profiles_table.sql (which was by mistake not backported into ovirt-engine-3.6 branch during 3.6 version development) into 04_00_0880_add_vm_auto_login_column_to_user_profiles_table.sql. This will fix the issue for upgrade from 3.6 to 4.0. Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Bug-Url: https://bugzilla.redhat.com/1371515 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- A packaging/dbscripts/upgrade/04_00_0880_add_vm_auto_login_column_to_user_profiles_table.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63201 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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]: core: Always fix VMs status during fencing if host if off
by tnisan＠redhat.com 04 Sep '16

04 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Always fix VMs status during fencing if host if off ...................................................................... core: Always fix VMs status during fencing if host if off Always fix VMs status during fencing flow when host is off (detected by power management status command) and don't rely on successful execution of power management start of the host. Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Bug-Url: https://bugzilla.redhat.com/1367557 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/RestartVdsCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63226 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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-3.6]: userportal, webadmin: default refresh
by oourfali＠redhat.com 04 Sep '16

04 Sep '16

Oved Ourfali has submitted this change and it was merged. Change subject: userportal,webadmin: default refresh ...................................................................... userportal,webadmin: default refresh - Defaulted refresh to false Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryParametersBase.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendPermissionResourceTest.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/communication/VdcOperationManagerTest.java 3 files changed, 14 insertions(+), 15 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63229 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Alexander Wels <awels(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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: userportal, webadmin: default refresh
by oourfali＠redhat.com 04 Sep '16

04 Sep '16

Oved Ourfali has submitted this change and it was merged. Change subject: userportal,webadmin: default refresh ...................................................................... userportal,webadmin: default refresh - Defaulted refresh to false Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryParametersBase.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendPermissionResourceTest.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/communication/VdcOperationManagerTest.java 3 files changed, 10 insertions(+), 6 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63225 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bf4eef17f82bf1552f89b6cf450533d3837921b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Alexander Wels <awels(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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix upgrade issue
by oourfali＠redhat.com 04 Sep '16

04 Sep '16

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix upgrade issue ...................................................................... core: Fix upgrade issue Fixes upgrade issue by copying original 03_06_1790_add_vm_auto_login_column_to_user_profiles_table.sql (which was by mistake not backported into ovirt-engine-3.6 branch during 3.6 version development) into 04_01_0270_add_vm_auto_login_column_to_user_profiles_table.sql. This does not make any harm on upgrade from 4.0 to 4.1 and it will fix direct upgrade from 3.6 to 4.1 if we will allow it in future. Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Bug-Url: https://bugzilla.redhat.com/1371515 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- A packaging/dbscripts/upgrade/04_01_0270_add_vm_auto_login_column_to_user_profiles_table.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63200 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e7f13a7a30aefde903c30ad5453201e53fe80e9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Always fix VMs status during fencing if host if off
by oourfali＠redhat.com 04 Sep '16

04 Sep '16

Oved Ourfali has submitted this change and it was merged. Change subject: core: Always fix VMs status during fencing if host if off ...................................................................... core: Always fix VMs status during fencing if host if off Always fix VMs status during fencing flow when host is off (detected by power management status command) and don't rely on successful execution of power management start of the host. Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Bug-Url: https://bugzilla.redhat.com/1367557 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/pm/RestartVdsCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63224 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I439d5a323092978a6cd58ceef93aa7f340e6dcf6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: restore check-patch for master on el7
by eedri＠redhat.com 04 Sep '16

04 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: restore check-patch for master on el7 ...................................................................... restore check-patch for master on el7 Change-Id: I1cf8bed6066f4dec29b596ca5ed26421815d356d Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/vdsm/vdsm_standard.yaml 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests Irit Goihman: Looks good to me -- To view, visit https://gerrit.ovirt.org/63235 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1cf8bed6066f4dec29b596ca5ed26421815d356d Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0