- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: fix logical loop in deps
by sbonazzo＠redhat.com 13 Aug '14

13 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix logical loop in deps ...................................................................... packaging: setup: fix logical loop in deps An addition to I767a574aa19be57e66ad0541d38852c953591d2f for remove/rename. Change-Id: Id0f2f9d55ffbab69f0c8b74b003295fc65cfc90b Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/base/core/misc.py M packaging/setup/plugins/ovirt-engine-rename/ovirt-engine/misc.py 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/31449 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0f2f9d55ffbab69f0c8b74b003295fc65cfc90b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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: backward compatibility for template's monitor devices
by ahadas＠redhat.com 13 Aug '14

13 Aug '14

Arik Hadas has submitted this change and it was merged. Change subject: core: backward compatibility for template's monitor devices ...................................................................... core: backward compatibility for template's monitor devices In 8c53fa4 we implemented backward compatibility for VM's monitor devices. This patch do the same for templates. Change-Id: I00f365263827b4c154cd1f762fce9afbe73e5015 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- 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/OvfTemplateReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmReader.java 3 files changed, 10 insertions(+), 22 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/29843 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I00f365263827b4c154cd1f762fce9afbe73e5015 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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]: engine: confusing error message retrieving from http session
by tnisan＠redhat.com 13 Aug '14

13 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: engine: confusing error message retrieving from http session ...................................................................... engine: confusing error message retrieving from http session - Fixed confusing error message when retrieving non existing value from http session. Instead of throwing error message in log, just return null. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101551 Change-Id: I336430e35bbbad07ea9566c8f5293980056361de Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/GenericApiGWTServiceImpl.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexander Wels: Verified Einav Cohen: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30585 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I336430e35bbbad07ea9566c8f5293980056361de Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(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]: engine : Rename TaskManagerUtil to CommandCoordinatorUtil
by tnisan＠redhat.com 13 Aug '14

13 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: engine : Rename TaskManagerUtil to CommandCoordinatorUtil ...................................................................... engine : Rename TaskManagerUtil to CommandCoordinatorUtil Rename TaskManagerUtil to CommandCoordinatorUtil Change-Id: Idf796edd6c0812dac03bbf1e71b637e9d2b452e5 Bug-Url: https://bugzilla.redhat.com/1127774 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandEntityCleanupManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetTasksStatusesByTasksIDsQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HibernateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitBackendServicesOnStartupBean.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeCommandCallback.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandCallback.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotSingleDiskLiveCommandCallback.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmFromImportExportCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryImageRemover.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/AsyncTaskUtils.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCoordinatorUtil.java 20 files changed, 68 insertions(+), 68 deletions(-) Approvals: Ravi Nori: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31346 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf796edd6c0812dac03bbf1e71b637e9d2b452e5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(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]: packaging: spec: Confict with dwh/reports 3.4 setup
by tnisan＠redhat.com 13 Aug '14

13 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: packaging: spec: Confict with dwh/reports 3.4 setup ...................................................................... packaging: spec: Confict with dwh/reports 3.4 setup setup plugins of dwh/reports 3.4 do not work well with engine-setup 3.5 Change-Id: I41485912f97bef698897105ede46bd6e8b748986 Bug-Url: https://bugzilla.redhat.com/1129568 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/31435 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41485912f97bef698897105ede46bd6e8b748986 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(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]: packaging: spec: Confict with dwh/reports 3.4 setup
by sbonazzo＠redhat.com 13 Aug '14

13 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: Confict with dwh/reports 3.4 setup ...................................................................... packaging: spec: Confict with dwh/reports 3.4 setup setup plugins of dwh/reports 3.4 do not work well with engine-setup 3.5 Change-Id: I41485912f97bef698897105ede46bd6e8b748986 Bug-Url: https://bugzilla.redhat.com/1129568 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/31434 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41485912f97bef698897105ede46bd6e8b748986 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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]: engine:Trusted Compute Pools - Open Attestation integration ...
by gchaplik＠redhat.com 13 Aug '14

13 Aug '14

Gilad Chaplik has submitted this change and it was merged. Change subject: engine:Trusted Compute Pools - Open Attestation integration with oVirt engine proposal ...................................................................... engine:Trusted Compute Pools - Open Attestation integration with oVirt engine proposal Detailed description: http://wiki.ovirt.org/Trusted_compute_pools Fix parser function to parse complex response string returned from OAT server in AttestationService. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1107517 Change-Id: I2f541b004ba383a3098cf7fbfca7a000b11b750f Signed-off-by: Emily <lijuan.zhang(a)intel.com> Signed-off-by: Jimmy <gang.wei(a)intel.com> Signed-off-by: Dave <wei.d.chen(a)intel.com> Signed-off-by: Emily <lijuan.zhang(a)intel.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/attestation/AttestationService.java 1 file changed, 19 insertions(+), 44 deletions(-) Approvals: Emily Zhang: Verified; Looks good to me, but someone else must approve Gilad Chaplik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26381 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f541b004ba383a3098cf7fbfca7a000b11b750f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Emily Zhang <lijuan.zhang(a)intel.com> Gerrit-Reviewer: Dave Chen <wei.d.chen(a)intel.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Emily Zhang <lijuan.zhang(a)intel.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: spec: prevent direct upgrade of dwh/reports to 3.6
by sbonazzo＠redhat.com 13 Aug '14

13 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: prevent direct upgrade of dwh/reports to 3.6 ...................................................................... packaging: spec: prevent direct upgrade of dwh/reports to 3.6 Change-Id: I59da9d829fcdf4aca63f09c769b01ba916efea5e Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31145 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59da9d829fcdf4aca63f09c769b01ba916efea5e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: history: updated statistics history_id to bigint
by ydary＠redhat.com 13 Aug '14

13 Aug '14

Yaniv Dary has submitted this change and it was merged. Change subject: history: updated statistics history_id to bigint ...................................................................... history: updated statistics history_id to bigint Change-Id: Ibb60633b23a6de6d5d7a7b9c6f72f9ece44d764e Bug-Url: https://bugzilla.redhat.com/1121994 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/src/ovirt_engine_dwh/aggregationtodaily_3_5/AggregationToDaily.java M etl_export/src/ovirt_engine_dwh/aggregationtohourly_3_5/AggregationToHourly.java M etl_export/src/ovirt_engine_dwh/historydelete_3_5/HistoryDelete.java A packaging/dbscripts/upgrade/03_05_0070_updated_history_id_to_bigint.sql M tos_project/process/AggregationToDaily_3.5.item M tos_project/process/AggregationToDaily_3.5.properties M tos_project/process/AggregationToHourly_3.5.item M tos_project/process/AggregationToHourly_3.5.properties M tos_project/process/HistoryDelete_3.5.item M tos_project/process/HistoryDelete_3.5.properties 10 files changed, 707 insertions(+), 466 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31292 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb60633b23a6de6d5d7a7b9c6f72f9ece44d764e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(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: StoragePoolValidatorTest config mocking
by amureini＠redhat.com 12 Aug '14

12 Aug '14

Allon Mureinik has submitted this change and it was merged. Change subject: core: StoragePoolValidatorTest config mocking ...................................................................... core: StoragePoolValidatorTest config mocking Removed redundant mocking that's not used in the test and streamlined the existing mocking to use two version - 3.0 as an "old" version which does not support new features and 3.1 and a "new" version which does. Related-To: https://bugzilla.redhat.com/1127590 Change-Id: If916e1080686ec037dba6f79264307e73b1502d2 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StoragePoolValidatorTest.java 1 file changed, 12 insertions(+), 13 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/31266 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If916e1080686ec037dba6f79264307e73b1502d2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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: 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0