December 2014 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Adding ovirt node and ovirt node iso projects
by dcaroest＠redhat.com 12 Dec '14

12 Dec '14

David Caro has submitted this change and it was merged. Change subject: Adding ovirt node and ovirt node iso projects ...................................................................... Adding ovirt node and ovirt node iso projects fc20 based should work. el7 build will most like ly failed. The build uses master branch, other branches will follow Change-Id: If543fe28f200e8254d4d96fa6789db3ced44f2b1 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- A jobs/confs/shell-scripts/build-local-ovirt-node.sh A jobs/confs/shell-scripts/ovirt_node_cleanup.sh A jobs/confs/yaml/builders/ovirt-node-builders.yaml A jobs/confs/yaml/jobs/ovirt-node/ovirt-node-job.yaml A jobs/confs/yaml/publishers/ovirt-node-cleanup.yaml A jobs/confs/yaml/scms/ovirt-node-scms.yaml A jobs/confs/yaml/templates/ovirt-node-job-templates.yaml 7 files changed, 223 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36058 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If543fe28f200e8254d4d96fa6789db3ced44f2b1 Gerrit-PatchSet: 12 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: GraphicsInfo db persistence part
by ahadas＠redhat.com 12 Dec '14

12 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: core: GraphicsInfo db persistence part ...................................................................... core: GraphicsInfo db persistence part (cleanup of deprecated columns is in separate cleanup patch) Change-Id: I5a129454fc64aa34493b0495fb76f22f3d667e14 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1033547 --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDynamicDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/create_views.sql A packaging/dbscripts/upgrade/03_06_0590_add_graphics_info_columns.sql M packaging/dbscripts/vms_sp.sql 5 files changed, 214 insertions(+), 10 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28570 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a129454fc64aa34493b0495fb76f22f3d667e14 Gerrit-PatchSet: 42 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: add graphics device to management classes
by ahadas＠redhat.com 12 Dec '14

12 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: core: add graphics device to management classes ...................................................................... core: add graphics device to management classes This patch adds graphics device to vm/template management (CRUD) commands. Change-Id: I39c84ce94cd3cb52286c52d765a389e69c9b0e62 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1033547 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java 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/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmCommandTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmPoolWithVmsParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmManagementParametersBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmTemplateParametersBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DisplayType.java 14 files changed, 269 insertions(+), 35 deletions(-) Approvals: Arik Hadas: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28567 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I39c84ce94cd3cb52286c52d765a389e69c9b0e62 Gerrit-PatchSet: 35 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: GraphicsInfo in entities
by ahadas＠redhat.com 11 Dec '14

11 Dec '14

Arik Hadas has submitted this change and it was merged. Change subject: engine: GraphicsInfo in entities ...................................................................... engine: GraphicsInfo in entities This patch adds GraphicsInfo structure to VM and VmDynamic. GraphicsInfo is a class that holds dynamic information about GraphicsDevice (which contains static, non-runtime information only). Change-Id: Ib11a8b63e76031f3137f2dfd0cd6551e9c19aca0 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1033547 --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/GraphicsInfo.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/VmDynamic.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 4 files changed, 84 insertions(+), 1 deletion(-) Approvals: Frank Kobzik: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28569 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib11a8b63e76031f3137f2dfd0cd6551e9c19aca0 Gerrit-PatchSet: 26 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: fixed NPE when vdsm returns emtpy balloon info
by tnisan＠redhat.com 11 Dec '14

11 Dec '14

Tal Nisan has submitted this change and it was merged. Change subject: core: fixed NPE when vdsm returns emtpy balloon info ...................................................................... core: fixed NPE when vdsm returns emtpy balloon info Bug-Url: https://bugzilla.redhat.com/1171491 Change-Id: Ieb2b479e216610a0372f1c2542d268cb1f898f4c Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Jiří Moskovčák: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36066 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb2b479e216610a0372f1c2542d268cb1f898f4c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: more accurate cores(threads) ui representation
by tnisan＠redhat.com 11 Dec '14

11 Dec '14

Tal Nisan has submitted this change and it was merged. Change subject: core: more accurate cores(threads) ui representation ...................................................................... core: more accurate cores(threads) ui representation in case of 2 cores 2 sockets and cluster-count threads as cores, we want to show at the host general tab 2(4) which reads 2 cores which is actual 4 for scheduling. without that fixed it showed the overall count which was 2(8) Change-Id: I4a2c3563bfd667331a58bf558c1161d4e379c624 Bug-Url: https://bugzilla.redhat.com/1082681 Signed-off-by: Roy Golan <rgolan(a)redhat.com> (cherry picked from commit f9ca88209249dd711722a8758922d625d3ac755b) --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostHardwareGeneralModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Doron Fediuck: Looks good to me, approved Roy Golan: Verified -- To view, visit http://gerrit.ovirt.org/36015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a2c3563bfd667331a58bf558c1161d4e379c624 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: Upload only the DC diskless VMs/Templates to OVF stores
by tnisan＠redhat.com 11 Dec '14

11 Dec '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Upload only the DC diskless VMs/Templates to OVF stores ...................................................................... core: Upload only the DC diskless VMs/Templates to OVF stores Since change Iaeecc6c4 OVF data of diskless VMs/Templates is uploaded to all the DC domain. The problem with that change is that the loaded VMs/Templates aren't only for that data center - which is fixed in this change. Change-Id: Ic51689cdf0479548c8c4be1267d478fcb8f89de0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1138134 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDynamicDAOTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStaticDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/vms_sp.sql 8 files changed, 118 insertions(+), 23 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/36098 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic51689cdf0479548c8c4be1267d478fcb8f89de0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: OVFs that aren't stored on any domain should be stored...
by tnisan＠redhat.com 11 Dec '14

11 Dec '14

Tal Nisan has submitted this change and it was merged. Change subject: core: OVFs that aren't stored on any domain should be stored on all ...................................................................... core: OVFs that aren't stored on any domain should be stored on all When the VM/Template has no non shareable image disks, its OVF file won't be stored on any domain. This patches changes this behavior by storing those OVFs on all the domains. Change-Id: Iaeecc6c4526800656315842384d83830fdf0b72b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1138134 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStorageDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessOvfUpdateForStoragePoolCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStaticDAOTest.java M packaging/dbscripts/vms_sp.sql 7 files changed, 87 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Liron Aravot: Verified -- To view, visit http://gerrit.ovirt.org/35818 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaeecc6c4526800656315842384d83830fdf0b72b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: vdsm: enable hooks building also on master
by dcaroest＠redhat.com 11 Dec '14

11 Dec '14

David Caro has submitted this change and it was merged. Change subject: vdsm: enable hooks building also on master ...................................................................... vdsm: enable hooks building also on master Change-Id: Iac21f40d4754d463c891e959316b3f643dff8490 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm/vdsm_create-rpms.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified David Caro: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36094 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac21f40d4754d463c891e959316b3f643dff8490 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: Remove unused import
by alitke＠redhat.com 11 Dec '14

11 Dec '14

Adam Litke has submitted this change and it was merged. Change subject: Remove unused import ...................................................................... Remove unused import Leftover, because Plotter is not used at all in this file. Change-Id: Ib0682ae7225c1ddb532ff815f4cb6055b9ce4454 Signed-off-by: Martin Pavlasek <mpavlase(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/35411 Reviewed-by: Adam Litke <alitke(a)redhat.com> Tested-by: Adam Litke <alitke(a)redhat.com> --- M mom/HostMonitor.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Adam Litke: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0682ae7225c1ddb532ff815f4cb6055b9ce4454 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Pavlásek <mpavlase(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0