Omer Frenkel has submitted this change and it was merged.
Change subject: core: remove check for permissions on pool on detach user from vm
(#850741)
......................................................................
core: remove check for permissions on pool on detach user from vm (#850741)
https://bugzilla.redhat.com/show_bug.cgi?id=850741
When detaching user from pool-vm, there is a check that the user has
permissions on the pool.
this check is not correct since user may have lost his permissions on
the pool before stopping the vm, this check was removed.
anyway user is detached from vms in this pool so it has no meaning.
Change-Id: I3bfb1a741c6be33427e94d944a8c41d3553cba80
Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DetachUserFromVmFromPoolCommand.java
1 file changed, 2 insertions(+), 12 deletions(-)
Approvals:
Omer Frenkel: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/7484
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3bfb1a741c6be33427e94d944a8c41d3553cba80
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Muli Salem <msalem(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server