April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: frontend: Add infrastructure for Console Foreign Menu
by vszocs＠redhat.com 08 Apr '15

08 Apr '15

Vojtech Szocs has submitted this change and it was merged. Change subject: frontend: Add infrastructure for Console Foreign Menu ...................................................................... frontend: Add infrastructure for Console Foreign Menu Added necessary hooks to implement outputting the [ovirt] section into VV file. Please note this just related to frontend infrastructure and contains no API commitments (does not alter VV file output in any way). Moved the RestApiSessionManager to gwt-common project so it would be accessible from both webadmin and userportal. Due to the fact that uicommon does not see the generated gwt-p events and handlers we need to attach the listener for RestApiSessionAcquiredEvent in UiCommonDefaultTypeResolver. To prevent excessive number of event listeners attached we cache the created SpiceNativeImpl and VncNativeImpl. Change-Id: I0e0cc2a2fd462bd8dc9085b300ec12fe46ff4518 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=975730 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/GwtDynamicHostPageServlet.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/WebAdminHostPageServlet.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/server/gwt/AbstractGwtDynamicHostPageServletTest.java M frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/server/gwt/WebAdminHostPageServletTest.java M frontend/webadmin/modules/gwt-common/exclude-filters.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseSystemModule.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseUiCommonModule.java R frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/restapi/EngineSessionTimeoutData.java R frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/restapi/RestApiSessionAcquired.java R frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/restapi/RestApiSessionManager.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/BaseApplicationInit.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/AbstractConsole.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/AbstractRdp.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/AbstractSpice.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/AbstractVnc.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/NoVncImpl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/RdpNativeImpl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/RdpPluginImpl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpiceHtml5Impl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpiceNativeImpl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpicePluginImpl.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/UiCommonDefaultTypeResolver.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/VncNativeImpl.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/model/AbstractConsoleWithForeignMenu.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ConsoleModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ISpiceNative.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/IVncNative.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/SpiceConsoleModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VncConsoleModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/restapi/HasForeignMenuData.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/system/ApplicationInit.java M frontend/webadmin/modules/webadmin/exclude-filters.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/PluginModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/PluginEventHandler.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/system/ApplicationInit.java 35 files changed, 361 insertions(+), 190 deletions(-) Approvals: Martin Betak: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38986 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e0cc2a2fd462bd8dc9085b300ec12fe46ff4518 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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-dwh[master]: history: change disks usage aggregation
by sradco＠redhat.com 08 Apr '15

08 Apr '15

Shirly Radco has submitted this change and it was merged. Change subject: history: change disks usage aggregation ...................................................................... history: change disks usage aggregation Made change in the disks usage aggregation process to make it more efficient. Change-Id: Iacff81b722d27d19f6ff304756255c5435045fe6 Bug-Url:https://bugzilla.redhat.com/1202661 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/src/ovirt_engine_dwh/aggregationtodaily_3_6/AggregationToDaily.java M etl_export/src/ovirt_engine_dwh/aggregationtohourly_3_6/AggregationToHourly.java M tos_project/process/AggregationToDaily_3.6.item M tos_project/process/AggregationToDaily_3.6.properties M tos_project/process/AggregationToHourly_3.6.item M tos_project/process/AggregationToHourly_3.6.properties 6 files changed, 2,166 insertions(+), 968 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacff81b722d27d19f6ff304756255c5435045fe6 Gerrit-PatchSet: 3 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-dwh[ovirt-engine-dwh-3.5]: history: change disks usage aggregation
by sradco＠redhat.com 08 Apr '15

08 Apr '15

Shirly Radco has submitted this change and it was merged. Change subject: history: change disks usage aggregation ...................................................................... history: change disks usage aggregation Made change in the disks usage aggregation process to make it more efficient. Change-Id: Iacff81b722d27d19f6ff304756255c5435045fe6 Bug-Url:https://bugzilla.redhat.com/1202661 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 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 6 files changed, 1,991 insertions(+), 206 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38812 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacff81b722d27d19f6ff304756255c5435045fe6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.5 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

1 0

Change in ovirt-engine[master]: database, packaging: fix duplicate upgrade script
by ofrenkel＠redhat.com 08 Apr '15

08 Apr '15

Omer Frenkel has submitted this change and it was merged. Change subject: database, packaging: fix duplicate upgrade script ...................................................................... database, packaging: fix duplicate upgrade script Only renaming to avoid the clash, no code change whatsoever Change-Id: I4060280a3b6f9c495ac4844a24dd2dae1aed4729 Signed-off-by: Francesco Romani <fromani(a)redhat.com> --- R packaging/dbscripts/upgrade/03_06_1180_add_user_profiles.sql 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Francesco Romani: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39684 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4060280a3b6f9c495ac4844a24dd2dae1aed4729 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(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]: database, core: Add the UserProfile entity
by vszocs＠redhat.com 08 Apr '15

08 Apr '15

Vojtech Szocs has submitted this change and it was merged. Change subject: database, core: Add the UserProfile entity ...................................................................... database, core: Add the UserProfile entity Create an entity to store additional user data (currently just the ssh public key for serial console access). Includes the DAO, unit tests and the schema update. Feature wiki page: http://www.ovirt.org/Features/Serial_Console Change-Id: Idca5769f84d379fd20a184608667a7774cc465c8 Signed-off-by: Vitor de Lima <vdelima(a)redhat.com> Signed-off-by: Francesco Romani <fromani(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/UserProfile.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/UserProfileDAO.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/UserProfileDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DbFacadeDAOTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DbUserDAOTest.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/UserProfileDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml A packaging/dbscripts/upgrade/03_06_1170_add_user_profiles.sql A packaging/dbscripts/user_profiles_sp.sql 10 files changed, 498 insertions(+), 2 deletions(-) Approvals: Francesco Romani: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38012 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idca5769f84d379fd20a184608667a7774cc465c8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(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-gluster]: engine: Fixed geo-rep auditlog issue
by sabose＠redhat.com 08 Apr '15

08 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine: Fixed geo-rep auditlog issue ...................................................................... engine: Fixed geo-rep auditlog issue Fixed a startup error logged due to missing AuditLogType and AuditLogMessages Change-Id: I7f6a110fd4d06d83bd16fcae9bff51201ca8140c Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/DeleteGeoRepSessionCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.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 5 files changed, 12 insertions(+), 5 deletions(-) Approvals: Sahina Bose: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39677 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f6a110fd4d06d83bd16fcae9bff51201ca8140c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: engine: Fix db scheduler test
by sabose＠redhat.com 08 Apr '15

08 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine: Fix db scheduler test ...................................................................... engine: Fix db scheduler test The tests fails periodically due to the timing between quartz scheduler thread and main thread Increased sleep duration in main thread. Change-Id: I2ff92a2331dfa3ac7d99e4ef8d858b21f1ffda05 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/scheduler/src/test/java/org/ovirt/engine/core/utils/timer/DBSchedulerUtilQuartzImplTest.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39353 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ff92a2331dfa3ac7d99e4ef8d858b21f1ffda05 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: dal: Get geo-rep session with slave host and volume
by sabose＠redhat.com 08 Apr '15

08 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: dal: Get geo-rep session with slave host and volume ...................................................................... dal: Get geo-rep session with slave host and volume Added a query to get a geo-rep session given a slave host and volume name Change-Id: I1776500724a56acee9798066241e8b2c88b16836 Bug-Url: https://bugzilla.redhat.com/1125845 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterGeoRepDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterGeoRepDaoDbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterGeoRepDaoTest.java M packaging/dbscripts/gluster_georep_sp.sql A packaging/dbscripts/upgrade/03_05_1350_add_idx_gluster_georep_slavehostname.sql 5 files changed, 35 insertions(+), 0 deletions(-) Approvals: Sahina Bose: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39676 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1776500724a56acee9798066241e8b2c88b16836 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: engine: Corrected geo-replication status sync
by sabose＠redhat.com 08 Apr '15

08 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine: Corrected geo-replication status sync ...................................................................... engine: Corrected geo-replication status sync Corrected issues while syncing geo-replication session w.r.t saving slave uuids and retrieving status Change-Id: If30bda56f117fedf3d42b51971201e4557f1c2cf Bug-Url: https://bugzilla.redhat.com/1195699 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterGeoRepSyncJob.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterGeoRepDaoDbFacadeImpl.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java 3 files changed, 15 insertions(+), 8 deletions(-) Approvals: Sahina Bose: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39675 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If30bda56f117fedf3d42b51971201e4557f1c2cf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: engine: Corrected VDS parsing for geo-rep config
by sabose＠redhat.com 08 Apr '15

08 Apr '15

Sahina Bose has submitted this change and it was merged. Change subject: engine: Corrected VDS parsing for geo-rep config ...................................................................... engine: Corrected VDS parsing for geo-rep config Corrected the output parsing for geo-rep config. Refactored the class name Change-Id: Ib293b98daf575f18182f453fab7c7d5aa78dcdc8 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeGeoRepConfigListVDSCommand.java D backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeConfigListXmlRpc.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeGeoRepConfigListXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 6 files changed, 54 insertions(+), 54 deletions(-) Approvals: Sahina Bose: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39674 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib293b98daf575f18182f453fab7c7d5aa78dcdc8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0