May 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: added support to override migration policies
by tjelinek＠redhat.com 06 May '16

06 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: added support to override migration policies ...................................................................... core: added support to override migration policies Added migration policy to VmBase. It is a field which will be taken from: - instance type, - from VM (per-VM custom value, template's value as default) - if not defined than the one from cluster Change-Id: I95e1774a25bca6bead2be22575fe6e9668177fa3 Bug-Url: https://bugzilla.redhat.com/1252426 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmManagementCommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/InstanceType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmBaseDao.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfProperties.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/CustomInstanceType.java M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/04_00_0610_add_migration_policy_to_vm.sql M packaging/dbscripts/vm_templates_sp.sql M packaging/dbscripts/vms_sp.sql 17 files changed, 112 insertions(+), 19 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56215 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95e1774a25bca6bead2be22575fe6e9668177fa3 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(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: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Removed el6/fc22 from fabric-ovirt
by dcaro＠redhat.com 06 May '16

06 May '16

David Caro has submitted this change and it was merged. Change subject: Removed el6/fc22 from fabric-ovirt ...................................................................... Removed el6/fc22 from fabric-ovirt Change-Id: I52156d2a860f8f4b0b08af0b4a28ff3dfb9deedd Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/fabic-ovirt/fabric-ovirt_standard.yaml 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/57150 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52156d2a860f8f4b0b08af0b4a28ff3dfb9deedd Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: node: add requirement for cockpit-ovirt plugin
by sbonazzo＠redhat.com 06 May '16

06 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: add requirement for cockpit-ovirt plugin ...................................................................... node: add requirement for cockpit-ovirt plugin Adding require to fetch the cockpit-ovirt plugin bits. Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release36.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57144 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: add requirement for cockpit-ovirt plugin
by sbonazzo＠redhat.com 06 May '16

06 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: add requirement for cockpit-ovirt plugin ...................................................................... node: add requirement for cockpit-ovirt plugin Adding require to fetch the cockpit-ovirt plugin bits. Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release40.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57145 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(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-release[ovirt-4.0]: automation: add virt sig oVirt 4.0 candidate
by sbonazzo＠redhat.com 06 May '16

06 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: add virt sig oVirt 4.0 candidate ...................................................................... automation: add virt sig oVirt 4.0 candidate Change-Id: I15d503032908281aec5386a8f40dbbaecbac55fe Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit f901a032e5d90e9650bff8099ca1199c5a6f7d70) --- M automation/check-patch.repos 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57143 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15d503032908281aec5386a8f40dbbaecbac55fe 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: add requirement for cockpit-ovirt plugin
by sbonazzo＠redhat.com 06 May '16

06 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: add requirement for cockpit-ovirt plugin ...................................................................... node: add requirement for cockpit-ovirt plugin Adding require to fetch the cockpit-ovirt plugin bits. Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release-master.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Ryan Barry: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57061 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf92afbc9488871fb1cba28dd2ce46befae5a0ef Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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-release[master]: automation: add virt sig oVirt 4.0 candidate
by sbonazzo＠redhat.com 06 May '16

06 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: add virt sig oVirt 4.0 candidate ...................................................................... automation: add virt sig oVirt 4.0 candidate Change-Id: I15d503032908281aec5386a8f40dbbaecbac55fe Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/check-patch.repos 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57142 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15d503032908281aec5386a8f40dbbaecbac55fe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master 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[master]: restapi: Fix mapping of boot devices
by juan.hernandez＠redhat.com 06 May '16

06 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix mapping of boot devices ...................................................................... restapi: Fix mapping of boot devices The following patch (commit ce9644c) introduced a regression in the mapping of boot devices: restapi: Use multiple "boot.devices.device" instead of multiple "boot" https://gerrit.ovirt.org/46744 The effect is that the boot devices returned by the API are only correct when the VM is running. This patch fixes that regression. Change-Id: I5e1f6ec4e8f64f94940d84a54a63275e5ef046fe Bug-Url: https://bugzilla.redhat.com/1333340 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57098 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e1f6ec4e8f64f94940d84a54a63275e5ef046fe Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Add V3 adapters for custom properties
by juan.hernandez＠redhat.com 06 May '16

06 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add V3 adapters for custom properties ...................................................................... restapi: Add V3 adapters for custom properties The V3 to V4 adapter classes for custom properties didn't exist. As a result creating or retrieving a virtual machine with custom properties generated an exception. This patch adds the required adapters. Change-Id: Ifda77f2caf269d36f0be9f2c70d167deff1618c8 Bug-Url: https://bugzilla.redhat.com/1333354 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3CustomPropertiesInAdapter.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3CustomPropertiesOutAdapter.java 2 files changed, 64 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57100 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifda77f2caf269d36f0be9f2c70d167deff1618c8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix V3 snapshot action links
by juan.hernandez＠redhat.com 06 May '16

06 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix V3 snapshot action links ...................................................................... restapi: Fix V3 snapshot action links In version 4 of the API all the URL segments are generated concatenating the words of the relevant name without separators. But in version 3 some URL segments didn't follow this rule, in particular the URL segments corresponding to snapshot actions used an underscore to separate words: preview_snapshot commit_snapshot undo_snapshot To preserve backwards compatibility this patch changes the V3 adapter so that it fixes the links to these actions, adding the required underscore. Change-Id: Idf1712f58dc29d6a91335f2c2f228e9fe9f746aa Bug-Url: https://bugzilla.redhat.com/1332986 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3VmOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmServer.java 2 files changed, 39 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57088 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf1712f58dc29d6a91335f2c2f228e9fe9f746aa Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0