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 <ofren...@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 <ofren...@redhat.com> Gerrit-Reviewer: Muli Salem <msa...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
