Change in ovirt-engine[ovirt-engine-3.5]: core: added permissions needed to change cluster only when c...

tnisan at redhat.com tnisan at redhat.com
Wed Aug 19 15:35:33 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: core: added permissions needed to change cluster only when changing the cluster
......................................................................


core: added permissions needed to change cluster only when changing the cluster

Since the backend can not trust that the TargetVdsGroupId is sent only when
actually needed, added a check which makes sure to expect the permissions only
when really needed.

Change-Id: I3d4b78b271ea58fceed6a15228d7b5f98c14508c
Bug-Url: https://bugzilla.redhat.com/1255061
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Tomas Jelinek: Verified
  Jenkins CI: Passed CI tests
  Omer Frenkel: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d4b78b271ea58fceed6a15228d7b5f98c14508c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list