Change in ovirt-engine[master]: core: backup awareness, full backup check
Eli Mesika has submitted this change and it was merged.
Change subject: core: backup awareness, full backup check
......................................................................
core: backup awareness, full backup check
This patch changes the backup awareness logic as follows:
We will check only for FULL backup , a full backup contains 'db' and
'files' as scope
If one of 'db' or 'files' is missing or both we will generate NO BACKUP
alert
If two exists but at least one is OLD we will generate an alert with the
oldest date
Change-Id: I71a8b2b4278db039518f7bfc9b684f390ca449fe
Signed-off-by: emesika <emesika(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EngineBackupAwarenessManager.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 backend/manager/modules/dal/src/test/resources/fixtures.xml
M packaging/dbscripts/audit_log_sp.sql
M packaging/dbscripts/engine_backup_log_sp.sql
6 files changed, 40 insertions(+), 14 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/40543
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I71a8b2b4278db039518f7bfc9b684f390ca449fe
Gerrit-PatchSet: 4
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: Jenkins CI
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: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org