Change in ovirt-engine[master]: core: LiveMerge: Remove audit message when delete succeeds

amureini at redhat.com amureini at redhat.com
Thu Mar 17 08:46:39 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: core: LiveMerge: Remove audit message when delete succeeds
......................................................................


core: LiveMerge: Remove audit message when delete succeeds

Currently, if live merge succeeds we get an audit message
indicating that volume doesn't exist. This hapens during
DESTROY_IMAGE_CHECK step where we verify that image removed. In a
successful use case, the image is indeed removed hence, we get an
audit message alerting that volume doesn't exist.  To fix this, we
added an option to indicate whether we want to log audit message.

Change-Id: I18d190aa05c7f1e404c10b11fd537e92ba77e434
Bug-Url: https://bugzilla.redhat.com/1310615
Signed-off-by: Ala Hino <ahino at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DestroyImageCheckCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/DestroyImageCheckCommandTest.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/SPMGetVolumeInfoVDSCommand.java
3 files changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved; Passed CI tests
  Ala Hino: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I18d190aa05c7f1e404c10b11fd537e92ba77e434
Gerrit-PatchSet: 21
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: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list