Change in ovirt-engine[master]: core: remove check for permissions on pool on detach user fr...

ofrenkel at redhat.com ofrenkel at redhat.com
Tue Aug 28 09:11:03 UTC 2012


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 at 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 at redhat.com>
Gerrit-Reviewer: Muli Salem <msalem at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list