Change in ovirt-engine[master]: core: Add audit entry with cold merge info

Code Review gerrit at ovirt.org
Mon Jul 3 11:48:43 UTC 2017


>From Maor Lipchuk <mlipchuk at redhat.com>:

Maor Lipchuk has submitted this change and it was merged. ( https://gerrit.ovirt.org/78628 )

Change subject: core: Add audit entry with cold merge info
......................................................................


core: Add audit entry with cold merge info

In commit 2420937 (cold-merge: Update step message to include the host
name), we added merge info, mainly the host running the merge job, to
the step message. In this patch, we introduce a mechanism to add an
audit log entry.
In order to add an audit log entry:
1. An audit log type with same step enum name must be added to
   AuditLogType.java
2. A message must be added to AuditLogMessages.properties. The arguments of
   the message must fit those in the message of the step enum. Usually,
   the same message is used

For example, for cold merge we previously added MERGE_SNAPSHOTS_ON_HOST
in StepEnum. Now, we add MERGE_SNAPSHOTS_ON_HOST to AuditLogType.java and
add a corresponding message (actually same message created for the
step enum) in AuditLogMessages.properties.

Change-Id: I685e9d833e67aa9ef0ea9680cdbacc1cc318eef8
Bug-Url: https://bugzilla.redhat.com/1464795
Signed-off-by: Ala Hino <ahino at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/utils/VdsCommandsHelper.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
3 files changed, 23 insertions(+), 0 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Maor Lipchuk: Looks good to me, approved
  Ala Hino: Verified



-- 
To view, visit https://gerrit.ovirt.org/78628
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I685e9d833e67aa9ef0ea9680cdbacc1cc318eef8
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list