November 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Cluster parameters override
by juan.hernandez＠redhat.com 25 Nov '14

25 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Cluster parameters override ...................................................................... restapi: Cluster parameters override This feature allows to configure the 'emulated machine' and 'cpu model' parameters for each VM separately instead of relying on the cluster default. Change-Id: Ibfe64e713edd43cad33b385699159db534112c02 Bug-Url: https://bugzilla.redhat.com/838487 Feature-page:http://www.ovirt.org/Features/Cluster_parameters_override Signed-off-by: Eldan Shachar <eshachar(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 4 files changed, 40 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31581 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibfe64e713edd43cad33b385699159db534112c02 Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eldan Shachar <eshachar(a)redhat.com> Gerrit-Reviewer: Eldan Shachar <eshachar(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(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]: Revert "setup: Changing task cleaner utility to also handle ...
by oourfali＠redhat.com 25 Nov '14

25 Nov '14

Oved Ourfali has submitted this change and it was merged. Change subject: Revert "setup: Changing task cleaner utility to also handle removal of commands" ...................................................................... Revert "setup: Changing task cleaner utility to also handle removal of commands" This reverts commit a5462c7356b90b1465e05f4bb84f63a0005892bc. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1167769 Change-Id: I5f08ff501606d06907704b60c57e950837d4e162 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M packaging/setup/dbutils/taskcleaner.sh M packaging/setup/dbutils/taskcleaner_sp.sql 2 files changed, 5 insertions(+), 88 deletions(-) Approvals: Yedidyah Bar David: Looks good to me, but someone else must approve Yair Zaslavsky: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f08ff501606d06907704b60c57e950837d4e162 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Image configuration adjustments for cloning a VM
by tnisan＠redhat.com 25 Nov '14

25 Nov '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Image configuration adjustments for cloning a VM ...................................................................... core: Image configuration adjustments for cloning a VM While cloning a VM, in some cases disk image configurations must be applied before executing the command. Specifically when cloning a VM's snapshot with thin NFS, raw disk allocation, to a VM with thin block disk allocation- the raw format needs to be changed to COW. Change-Id: Id924e9beab5d84277fad4f3a2a40b14571beaf22 Bug-Url: https://bugzilla.redhat.com/1121397 Signed-off-by: Amit Aviram <aaviram(a)redhat.com> --- 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/AddVmFromSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CloneVmCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmFromSnapshotCommandTest.java 4 files changed, 29 insertions(+), 9 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Amit Aviram: Verified -- To view, visit http://gerrit.ovirt.org/35225 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id924e9beab5d84277fad4f3a2a40b14571beaf22 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(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[master]: Revert "setup: Changing task cleaner utility to also handle ...
by yzaslavs＠redhat.com 25 Nov '14

25 Nov '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: Revert "setup: Changing task cleaner utility to also handle removal of commands" ...................................................................... Revert "setup: Changing task cleaner utility to also handle removal of commands" This reverts commit 036eb1652e4252c967d24e25518b140665eac9db. Change-Id: I5f08ff501606d06907704b60c57e950837d4e162 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M packaging/setup/dbutils/taskcleaner.sh M packaging/setup/dbutils/taskcleaner_sp.sql 2 files changed, 5 insertions(+), 88 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f08ff501606d06907704b60c57e950837d4e162 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix Live Merge recovery with JSON-RPC hosts
by oourfali＠redhat.com 25 Nov '14

25 Nov '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix Live Merge recovery with JSON-RPC hosts ...................................................................... core: Fix Live Merge recovery with JSON-RPC hosts JSON-RPC calls to the reconcileVolumeChain verb were not done correctly, causing recovery to fail if live merge was interrupted by e.g. host shutdown during the merge operation. Change-Id: If1b23580ae6023754d2e02000dc5913262b3ef8b Bug-Url: https://bugzilla.redhat.com/1161127 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcIIrsServer.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Greg Padgett: Verified Allon Mureinik: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35017 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1b23580ae6023754d2e02000dc5913262b3ef8b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: be more verbose when starting the HE VM fails
by msivak＠redhat.com 25 Nov '14

25 Nov '14

Martin Sivák has submitted this change and it was merged. Change subject: be more verbose when starting the HE VM fails ...................................................................... be more verbose when starting the HE VM fails Change-Id: Ieb4842048aa690925284d62685156f56485dd41e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1150087 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb4842048aa690925284d62685156f56485dd41e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: Latest template instace safe recognition
by tjelinek＠redhat.com 25 Nov '14

25 Nov '14

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal, webadmin: Latest template instace safe recognition ...................................................................... userportal, webadmin: Latest template instace safe recognition Regular template and latest template (the one pointing to the lates version) were recognized by matching of localized UI name. This patch introduces frontend only class LatestVmTemplate to mark latest templates. Change-Id: I161f1d9603ea17ef7fc3801eb3dfa0cb924af2b4 Bug-Url: https://bugzilla.redhat.com/1166152 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/LatestVmTemplate.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 6 files changed, 76 insertions(+), 56 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified -- To view, visit http://gerrit.ovirt.org/35472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I161f1d9603ea17ef7fc3801eb3dfa0cb924af2b4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Fixed issue introduced by #35238
by dcaroest＠redhat.com 25 Nov '14

25 Nov '14

David Caro has submitted this change and it was merged. Change subject: Fixed issue introduced by #35238 ...................................................................... Fixed issue introduced by #35238 Now all the strings have extra single quotes, and the script parses the config to get some values, adapted. Change-Id: Ie7a6ac35940ab2d89a748e0f5da9d68e2e39a3cd Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/build_mock_srcrpm.sh M jobs/confs/shell-scripts/mock_build_onlyrpm.sh 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35500 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7a6ac35940ab2d89a748e0f5da9d68e2e39a3cd Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(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 ovirt-dwh[master]: etl: etl version is now defined by package settings
by ydary＠redhat.com 25 Nov '14

25 Nov '14

Yaniv Dary has submitted this change and it was merged. Change subject: etl: etl version is now defined by package settings ...................................................................... etl: etl version is now defined by package settings This will pervent mismatch with spec defined version, since the default will be set by spec. Change-Id: Id058b661464ae8348fe03b4f549cf1d40c9fd5fe Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/ovirt_engine_dwh/historyetl_3_6/contexts/Default.properties M etl_export/ovirt_engine_dwh/parallelrun_3_6/contexts/Default.properties M etl_export/src/ovirt_engine_dwh/historyetl_3_6/HistoryETL.java M etl_export/src/ovirt_engine_dwh/parallelrun_3_6/ParallelRun.java M tos_project/process/HistoryETL_3.6.item M tos_project/process/HistoryETL_3.6.properties M tos_project/process/ParallelRun_3.6.item M tos_project/process/ParallelRun_3.6.properties 8 files changed, 53 insertions(+), 74 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35084 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id058b661464ae8348fe03b4f549cf1d40c9fd5fe Gerrit-PatchSet: 9 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: do not instantiate singleton class
by rgolan＠redhat.com 25 Nov '14

25 Nov '14

Roy Golan has submitted this change and it was merged. Change subject: core: do not instantiate singleton class ...................................................................... core: do not instantiate singleton class - removed instantiation of class which is singleton and should not be instantiated, and should be injected if needed, and which field was unsused - removed other unused stuff Change-Id: Id142e03724cf83e10aa631e2785632610d7cecc9 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandsFactory.java 1 file changed, 3 insertions(+), 4 deletions(-) Approvals: Martin Mucha: Verified Roy Golan: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30513 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id142e03724cf83e10aa631e2785632610d7cecc9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0