April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: [RFE]Backup Awareness - DAO
by emesika＠redhat.com 16 Apr '15

16 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE]Backup Awareness - DAO ...................................................................... core: [RFE]Backup Awareness - DAO This patch adds DAO for the new BE, changes in AuditLog DAO and tests. Change-Id: I9b016e050f732eecb7f2872991d9fa9d31d9c024 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188127 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/AuditLogDAODbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EngineBackupLogDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/EngineBackupLogDaoDbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AuditLogDAOTest.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/EngineBackupLogDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/AuditLogDaoMocker.java 9 files changed, 214 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39525 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b016e050f732eecb7f2872991d9fa9d31d9c024 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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]: core: [RFE]Backup Awareness - BE
by emesika＠redhat.com 16 Apr '15

16 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE]Backup Awareness - BE ...................................................................... core: [RFE]Backup Awareness - BE This patch adds the EngineBackupHistory business entity. Change-Id: Ia3ca5ba16d1eaaa10c48e1391a2d606a152926b6 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188127 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/EngineBackupLog.java 1 file changed, 87 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39524 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3ca5ba16d1eaaa10c48e1391a2d606a152926b6 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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]: core: [RFE]Backup Awareness - DB and Alerts
by emesika＠redhat.com 16 Apr '15

16 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE]Backup Awareness - DB and Alerts ...................................................................... core: [RFE]Backup Awareness - DB and Alerts This patch adds the engine_backup_history and the SPs needed for backup events recording. This patch also adds the alerts and messages for this feature. Change-Id: I7824eb8a720bd654e94dada987aee19b9624a516 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188127 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- 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 packaging/dbscripts/audit_log_sp.sql A packaging/dbscripts/engine_backup_log_sp.sql A packaging/dbscripts/upgrade/03_06_1230_add_backup_awareness.sql 5 files changed, 90 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39523 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7824eb8a720bd654e94dada987aee19b9624a516 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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]: core: [RFE]Backup Awareness - Config
by emesika＠redhat.com 16 Apr '15

16 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE]Backup Awareness - Config ...................................................................... core: [RFE]Backup Awareness - Config Adding backup awareness to engine Currently backups are offered only as best practice, there is no alert or event that informs the user that no backup was done or that a long time passed from the last successful backup and he should backup the application database again. In case of an accident, this may lead to long engine downtime and lots of efforts restoring the engine DB if no backup was done or only an old backup exists The feature will enable to track backups done using engine-backup utility and to alert the administrator if a pre-configurable time had passed from the last successful backup. Feature page : http://www.ovirt.org/Features/BackupAwareness This patch adds the configuration values needed for the feature: BackupCheckPeriodInHours the period on which we check for a fresh backup default : 6, set to -1 to disable this feature BackupAlertPeriodInDays the max number of days allowed without a fresh backup. default : 1 Change-Id: I1ddd8c817742406d8206624599a82d8ff36b26ee Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1188127 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 3 files changed, 17 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39522 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ddd8c817742406d8206624599a82d8ff36b26ee Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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]: core: fix default network rename
by emesika＠redhat.com 16 Apr '15

16 Apr '15

Eli Mesika has submitted this change and it was merged. Change subject: core: fix default network rename ...................................................................... core: fix default network rename Upgrade script 03_06_0850 tried to rename the config option key ManagementNetwork to DefaultManagementNetwork There were 2 problems in that 1) The version given was misspelled : 'genenral', causing the rename to fail 2) The right place for configuration changes is dbscripts/upgrade/pre_upgrade/0000_config.sql This patch fixes both issues Change-Id: I21f964a12ecd4b99ea2aed512f89dc84882d70c7 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1211649 Signed-off-by: emesika <emesika(a)redhat.com> --- M packaging/dbscripts/upgrade/03_06_0850_update_management_network_vdc_option.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified Yedidyah Bar David: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39885 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I21f964a12ecd4b99ea2aed512f89dc84882d70c7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(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]: restapi: Show full Gluster storage domain details
by tnisan＠redhat.com 15 Apr '15

15 Apr '15

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Show full Gluster storage domain details ...................................................................... restapi: Show full Gluster storage domain details Before this patch the only storage detail of a Gluster displayed via REST was the type thus emitting important data such as address, path and mount options Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Bug-Url: https://bugzilla.redhat.com/1211153 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java 2 files changed, 18 insertions(+), 14 deletions(-) Approvals: Tal Nisan: Verified Juan Hernandez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39880 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Add 'description' as optional parameter for add disk
by juan.hernandez＠redhat.com 15 Apr '15

15 Apr '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add 'description' as optional parameter for add disk ...................................................................... restapi: Add 'description' as optional parameter for add disk Change-Id: Ib196bea4eae2dacd3c5d1c256aa5a78a2862a083 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1211314 --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 9 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified Juan Hernandez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39872 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib196bea4eae2dacd3c5d1c256aa5a78a2862a083 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(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-gluster]: webadmin : CheckBoxGroup and DaysOfMonthSelector Widgets
by kmayilsa＠redhat.com 15 Apr '15

15 Apr '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : CheckBoxGroup and DaysOfMonthSelector Widgets ...................................................................... webadmin : CheckBoxGroup and DaysOfMonthSelector Widgets CheckBoxGroup Widget and DaysOfMonthSelector Widgets Change-Id: I38daa0d2c151eb0e34603488496a8a1ea4719c87 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/CheckBoxGroup.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/CheckBoxGroupCss.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/DaysOfMonthSelector.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/DaysOfMonthSelectorCss.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelCheckBoxGroup.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelCheckBoxGroupEditor.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelDaysOfMonthSelector.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelDaysOfMonthSelectorEditor.java A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/CheckBoxGroup.css A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/DaysOfMonthSelector.css M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 11 files changed, 628 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39356 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38daa0d2c151eb0e34603488496a8a1ea4719c87 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: webadmin: UI for gluster volume snapshot actions
by kmayilsa＠redhat.com 15 Apr '15

15 Apr '15

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: UI for gluster volume snapshot actions ...................................................................... webadmin: UI for gluster volume snapshot actions Added UI changes for gluster volume snapshot actions - Restore - Delete - Delete All - Activate - Deactivate Change-Id: I3d5b6162c6c6931cd0366db5ba600566ab8ac7c9 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/help/HelpTag.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterVolumeSnapshotListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/gluster/SubTabGlusterVolumeSnapshotView.java 4 files changed, 386 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39355 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d5b6162c6c6931cd0366db5ba600566ab8ac7c9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(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[master]: restapi: Show full Gluster storage domain details
by tnisan＠redhat.com 15 Apr '15

15 Apr '15

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Show full Gluster storage domain details ...................................................................... restapi: Show full Gluster storage domain details Before this patch the only storage detail of a Gluster displayed via REST was the type thus emitting important data such as address, path and mount options Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Bug-Url: https://bugzilla.redhat.com/1211153 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageDomainMapper.java 2 files changed, 18 insertions(+), 14 deletions(-) Approvals: Tal Nisan: Verified Juan Hernandez: Looks good to me, but someone else must approve Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39810 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2294ab6a0256f5c8bc5f48e04dada506500f79d5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(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