- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: setup: handle ssh selinux context
by sbonazzo＠redhat.com 02 Dec '13

02 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: handle ssh selinux context ...................................................................... packaging: setup: handle ssh selinux context Restore ~/.ssh selinux context if selinux is enabled on the system. Change-Id: I4ffe0a5f5509a74318b73ee2c4d72cb2bd9a6ddf Bug-Url: https://bugzilla.redhat.com/1034786 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 1 file changed, 36 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/21843 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ffe0a5f5509a74318b73ee2c4d72cb2bd9a6ddf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: cleanup storagePool/domainMonitor on error
by sbonazzo＠redhat.com 02 Dec '13

02 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: cleanup storagePool/domainMonitor on error ...................................................................... packaging: setup: cleanup storagePool/domainMonitor on error disconnect storage pool connection and domain monitoring if an error occurs while deploying the hosted engine VM Change-Id: If8a408a7291b7a4478380fb6a98a0120699563e3 Bug-Url: https://bugzilla.redhat.com/1026344 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 31 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/21866 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8a408a7291b7a4478380fb6a98a0120699563e3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: api: redirect /api to /ovirt-engine/api using apa...
by sbonazzo＠redhat.com 02 Dec '13

02 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: api: redirect /api to /ovirt-engine/api using apache rewrite rules ...................................................................... packaging: api: redirect /api to /ovirt-engine/api using apache rewrite rules Using forward servlet is not a complete solution as there is some limitation of sharing context. So java based solution is not easily possible. Jboss rewrite rules cannot be used as it ignores the matrix ';' suffix. Using apache. One side effect is a redirect between /api and /api/ as apache must normalize the directory due to internal limitation. Tested using: httpd-2.2.22-25.ep6.el6.x86_64 httpd-2.4.6-2.fc19.x86_64 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1031597 Change-Id: I2ad9bb362719c7805be9f69d7e14bd3ad7b1f5c7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml M backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml M ovirt-engine.spec.in M packaging/conf/ovirt-engine-proxy.conf.v2.in M packaging/setup/plugins/ovirt-engine-setup/apache/ovirt_engine.py 6 files changed, 24 insertions(+), 19 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21566 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ad9bb362719c7805be9f69d7e14bd3ad7b1f5c7 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Ignore temporary tables exists in engine database
by iheim＠redhat.com 02 Dec '13

02 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: core: Ignore temporary tables exists in engine database ...................................................................... core: Ignore temporary tables exists in engine database Ignore temporary tables exists in engine database upon upgrade. Change-Id: I3664a60e1e97e52875882864995e41f3776f7fcd Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1034929 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/db/schema.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21896 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3664a60e1e97e52875882864995e41f3776f7fcd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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.3]: core: set action and object type on ChangeDiskCommand
by iheim＠redhat.com 02 Dec '13

02 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: core: set action and object type on ChangeDiskCommand ...................................................................... core: set action and object type on ChangeDiskCommand Change-Id: Ib3903eff5123b74b613cfdcb83733cbb34d8543f Bug-Url: https://bugzilla.redhat.com/1014030 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeDiskCommand.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21895 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib3903eff5123b74b613cfdcb83733cbb34d8543f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster : In the task tab, size of rebalanced files is shown...
by ofrenkel＠redhat.com 02 Dec '13

02 Dec '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster : In the task tab, size of rebalanced files is shown with units ...................................................................... gluster : In the task tab, size of rebalanced files is shown with units The size of files rebalanced was shown in bytes. Now,the size is shown in larger units. Change-Id: Iea6c90a098bfddfb616bc2b8ce58c9d0bb567f66 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1020883 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterAsyncTaskStatusQueryBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/tasks/GlusterTaskUtils.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeTaskStatusEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/SizeConverter.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterTasksListReturnForXmlRpc.java 5 files changed, 80 insertions(+), 16 deletions(-) Approvals: anmolbabu: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20511 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea6c90a098bfddfb616bc2b8ce58c9d0bb567f66 Gerrit-PatchSet: 28 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: remove pending VMs counting
by ofrenkel＠redhat.com 02 Dec '13

02 Dec '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: remove pending VMs counting ...................................................................... core: remove pending VMs counting The number of pending VMs is not in use anywhere, but it is still computed. In order to compute the pending VMs we increment and decrement the counter while running VMs in synchronized blocks and it is not good in terms of performance. So since this code is redundant, it is better to remove it. Change-Id: I3bd7a757be2979b86227ce56064e6165080035eb Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java 2 files changed, 0 insertions(+), 24 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21884 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bd7a757be2979b86227ce56064e6165080035eb Gerrit-PatchSet: 2 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: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Reinitialize VDSManager on change cluster
by iheim＠redhat.com 02 Dec '13

02 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Reinitialize VDSManager on change cluster ...................................................................... engine: Reinitialize VDSManager on change cluster When changing cluster, the VDSManager dictionary has to be updated. This is needed because the target cluster may support different services from the source cluster, and the monitoring strategies need to be updated. Change-Id: Icb25c12d35113af09c67d5fe6e275c6901725e54 Bug-Url: https://bugzilla.redhat.com/968178 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sahina Bose: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21905 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb25c12d35113af09c67d5fe6e275c6901725e54 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Reinitialize VDSManager on change cluster
by kmayilsa＠redhat.com 02 Dec '13

02 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: Reinitialize VDSManager on change cluster ...................................................................... engine: Reinitialize VDSManager on change cluster When changing cluster, the VDSManager dictionary has to be updated. This is needed because the target cluster may support different services from the source cluster, and the monitoring strategies need to be updated. Change-Id: Icb25c12d35113af09c67d5fe6e275c6901725e54 Bug-Url: https://bugzilla.redhat.com/968178 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/21855 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb25c12d35113af09c67d5fe6e275c6901725e54 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: event message for rebalance and remove brick complete
by kmayilsa＠redhat.com 02 Dec '13

02 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: event message for rebalance and remove brick complete ...................................................................... gluster: event message for rebalance and remove brick complete A suitable event message will be displayed when rebalance is completed in a volume and when data migration for brick removal is complete. Change-Id: If4da9730cd8c251bc5707df981204b092948fb6f bug url: https://bugzilla.redhat.com/show_bug.cgi?id=1015016 Signed-off-by: ndarshan <dnarayan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/tasks/GlusterTaskUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterTasksSyncJobTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties A packaging/dbscripts/upgrade/03_04_0150_gluster_async_tasks_finished_event_map.sql 9 files changed, 49 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Darshan N: Verified -- To view, visit http://gerrit.ovirt.org/21080 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If4da9730cd8c251bc5707df981204b092948fb6f Gerrit-PatchSet: 14 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0