Change in ovirt-engine[master]: core: more accurate audit log when domain is deactivated

laravot at redhat.com laravot at redhat.com
Sun Nov 15 09:14:57 UTC 2015


Liron Aravot has submitted this change and it was merged.

Change subject: core: more accurate audit log when domain is deactivated
......................................................................


core: more accurate audit log when domain is deactivated

when a Storage Domain is being deactivated it first moves to "Preparing for
maintenance" status and only when it's confirmed that its not accessed
(by monitoring) from any host it moves to Maintenance status.
However, currently when it moves to "Preparing for maintenance" the
audit log indicates that the domain was deactivated and when it actually
moves to Maintenance there's no audit log at all.
This patch fixes the current audit log and adds a new one for the time
the domain status is changed to Maintenance.

Change-Id: Ia64ce39aeeb4a8874a6c92dc4b337e7e4b1664bc
Bug-Url: https://bugzilla.redhat.com/1247957
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainCommand.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/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxyData.java
4 files changed, 26 insertions(+), 11 deletions(-)

Approvals:
  Liron Aravot: Verified; Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia64ce39aeeb4a8874a6c92dc4b337e7e4b1664bc
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list