Change in ovirt-engine[master]: core: added logging of missing permissions

[tjelinek at redhat.com]

Tomas Jelinek has submitted this change and it was merged.

Change subject: core: added logging of missing permissions
......................................................................


core: added logging of missing permissions

In CommandBase we log if the permission check fails, but in AddVmCommand this
method has been overridden completely not logging failed attempts.

Fixed by logging it also in AddVmCommand.

Change-Id: I1ad67c22f12762674a6b817339f5f167422475ea
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
2 files changed, 14 insertions(+), 9 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Jenkins CI: Passed CI tests
  Arik Hadas: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1ad67c22f12762674a6b817339f5f167422475ea
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>