June 2016 - Engine-commits - oVirt List Archives

Change in jenkins[master]: dashboard: Unify job configs & add stable branch for Engine 4.0
by dcaro＠redhat.com 15 Jun '16

15 Jun '16

David Caro has submitted this change and it was merged. Change subject: dashboard: Unify job configs & add stable branch for Engine 4.0 ...................................................................... dashboard: Unify job configs & add stable branch for Engine 4.0 1. ovirt-engine-dashboard_{build,checks}.yaml unified into ovirt-engine-dashboard_standard.yaml to avoid duplicating common metadata (such as versions/branches) 2. updated 4.0 nightly publisher config Change-Id: I3cc44097b70ac4485ab1dde89a6dd1e27a04d8da Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- D jobs/confs/projects/ovirt-engine-dashboard/ovirt-engine-dashboard_checks.yaml R jobs/confs/projects/ovirt-engine-dashboard/ovirt-engine-dashboard_standard.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml 3 files changed, 5 insertions(+), 18 deletions(-) Approvals: Alexander Wels: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/59183 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3cc44097b70ac4485ab1dde89a6dd1e27a04d8da Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(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-live[ovirt-live-3.6]: build: oVirt Live 3.6.7_rc4
by lveyde＠redhat.com 15 Jun '16

15 Jun '16

Lev Veyde has submitted this change and it was merged. Change subject: build: oVirt Live 3.6.7_rc4 ...................................................................... build: oVirt Live 3.6.7_rc4 Change-Id: I16b1c28441189114f81ae5865178dce85f14040a Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Lev Veyde: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59228 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I16b1c28441189114f81ae5865178dce85f14040a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: added two origin types: physical machine and hyper-v
by mbetak＠redhat.com 15 Jun '16

15 Jun '16

Martin Betak has submitted this change and it was merged. Change subject: core: added two origin types: physical machine and hyper-v ...................................................................... core: added two origin types: physical machine and hyper-v Needed by virt-v2v while importing to export domain, This may be needed by oVirt in future v2v support. Change-Id: I2b918701a3451ea31d81a7bc3d85f411e78b8db6 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/OriginType.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties 3 files changed, 9 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved Arik Hadas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59147 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b918701a3451ea31d81a7bc3d85f411e78b8db6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Preset imgbase-motd to enabled
by fabiand＠redhat.com 15 Jun '16

15 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Preset imgbase-motd to enabled ...................................................................... node: Preset imgbase-motd to enabled Change-Id: If9809ee0df9a5684d2d7490d311bbcc8895e1e0c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1326232 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M host-node/systemd/system-preset/98-ovirt-host-node.preset M ovirt-release40.spec.in 3 files changed, 5 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59223 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9809ee0df9a5684d2d7490d311bbcc8895e1e0c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Preset imgbase-motd to enabled
by fabiand＠redhat.com 15 Jun '16

15 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Preset imgbase-motd to enabled ...................................................................... node: Preset imgbase-motd to enabled Change-Id: If9809ee0df9a5684d2d7490d311bbcc8895e1e0c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1326232 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M host-node/systemd/system-preset/98-ovirt-host-node.preset M ovirt-release-master.spec.in 3 files changed, 5 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59222 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If9809ee0df9a5684d2d7490d311bbcc8895e1e0c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Fix affinity labels links
by tnisan＠redhat.com 15 Jun '16

15 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Fix affinity labels links ...................................................................... restapi: Fix affinity labels links Currently the links of the affinity labels resources aren't generated correctly, even if they are actually available. This patch adds the missing links to version 4 of the API, and modifies the version 3 compatibility layer so that it removes them. Change-Id: I28acbc2f26641984cd1e5bcf05ead1122a5f0e65 Bug-Url: https://bugzilla.redhat.com/1346248 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/ApiRootLinksCreator.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ApiOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3HostOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3VmOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java 7 files changed, 102 insertions(+), 12 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59216 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28acbc2f26641984cd1e5bcf05ead1122a5f0e65 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Fix affinity labels links
by juan.hernandez＠redhat.com 15 Jun '16

15 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix affinity labels links ...................................................................... restapi: Fix affinity labels links Currently the links of the affinity labels resources aren't generated correctly, even if they are actually available. This patch adds the missing links to version 4 of the API, and modifies the version 3 compatibility layer so that it removes them. Change-Id: I28acbc2f26641984cd1e5bcf05ead1122a5f0e65 Bug-Url: https://bugzilla.redhat.com/1346248 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/ApiRootLinksCreator.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ApiOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3HostOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3VmOutAdapter.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java 7 files changed, 102 insertions(+), 12 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Roy Golan: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59177 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28acbc2f26641984cd1e5bcf05ead1122a5f0e65 Gerrit-PatchSet: 3 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: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Prevent possible class cast exception during netwo...
by tnisan＠redhat.com 15 Jun '16

15 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Prevent possible class cast exception during network add. ...................................................................... webadmin: Prevent possible class cast exception during network add. A class cast exception can occur when AddNetworkCommand is used. Change-Id: I6b675d925b5360dcd7c1dcca166fdd517d5f01b9 Bug-Url: https://bugzilla.redhat.com/1216081 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NewNetworkModel.java 1 file changed, 45 insertions(+), 24 deletions(-) Approvals: Marcin Mirecki: Verified Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59124 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b675d925b5360dcd7c1dcca166fdd517d5f01b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(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: Correct flow of RunVmCommand.endAction()
by tnisan＠redhat.com 15 Jun '16

15 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Correct flow of RunVmCommand.endAction() ...................................................................... core: Correct flow of RunVmCommand.endAction() Recent change in RunVmCommand.getCallback() forces RunVmCommand to actively use the callback. This requires changes in other places to correctly execute RunVmCommand.endAction() in every situation. The flow selected by RunVmCommand.getFlow() method is now cached in RunVmParams. The cache in RunVmCommand itself is useless, because the command is created from scratch when CoCo calls its callback, and getFlow() cannot determine the flow correctly in this situation. AttachUserToVmFromPoolAndRunCommand is modified to correctly use the callback - without it endAction() is never called. RunVmCommand is started as COMMAND_MANAGED and performs by itself all actions needed to start a VM as stateless or stateful. AttachUserToVmFromPoolAndRunCommand.endAction() is executed after that and checks the final result of RunVmCommand. If it failed, user's ownership of the VM is reverted. Change-Id: I099d5a8b7ee3aef6b8abb124de39767de6a99c20 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1342795 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachUserToVmFromPoolAndRunCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RunVmCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RunVmParams.java 4 files changed, 94 insertions(+), 59 deletions(-) Approvals: Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59180 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I099d5a8b7ee3aef6b8abb124de39767de6a99c20 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(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: rename migration policies and change descriptions
by tnisan＠redhat.com 15 Jun '16

15 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: rename migration policies and change descriptions ...................................................................... core: rename migration policies and change descriptions Renamed the migration policies to be more user friendly. Renames: - change the "Safe but may not converge" to "Minimal downtime" - change the "Should converge but..." to "Suspend workload if needed" Added mention in all policy descriptions that "The guest agent hook mechanism is enabled/disabled." Change-Id: I10769ff5602e380107dc876d8c10ae0c7f94552e Bug-Url: https://bugzilla.redhat.com/1346194 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59204 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10769ff5602e380107dc876d8c10ae0c7f94552e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0