Change in ovirt-engine[ovirt-engine-3.5]: aaa: sync: copy isAdmin from existing user

[piotr.kliczewski at gmail.com]

Piotr Kliczewski has submitted this change and it was merged.

Change subject: aaa: sync: copy isAdmin from existing user
......................................................................


aaa: sync: copy isAdmin from existing user

in role based security there should be no duplication of roles, but in
ovirt-engine schema we have isAdmin marker, although the role can be
queried.

temporary solution is to copy existing user's bit into synced user.

long term solution is to remove this duplication.

Topic: AAA
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120720
Change-Id: Id62a2a774c102e3e31c022456809b4b2eb2319b2
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SyncUsers.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Alon Bar-Lev: Verified
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/30943
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id62a2a774c102e3e31c022456809b4b2eb2319b2
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server