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

tnisan at redhat.com tnisan at redhat.com
Tue Nov 24 11:40:03 UTC 2015


Tal Nisan 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:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Liron Aravot: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia64ce39aeeb4a8874a6c92dc4b337e7e4b1664bc
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot 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