Omer Frenkel has submitted this change and it was merged.
Change subject: gluster: Do not log removal of option group
......................................................................
gluster: Do not log removal of option group
The option "group" is a special case, where glusterfs internally
replaces it with the options defined in corresponding group file.
Hence, every time you set the option group, it eventually gets
replaced with a set of options.
For this reason, it is not appropriate to create an audit log when
GlusterManager detects that the option "group" has been removed.
Hence added a check to avoid the same.
Change-Id: Ic1fc981563239ca2209c7172805d802a9691cce4
Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterManager.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/constants/gluster/GlusterConstants.java
2 files changed, 10 insertions(+), 4 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/11265
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1fc981563239ca2209c7172805d802a9691cce4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>