Change in ovirt-engine[master]: core: Fix group id on updating cache of users

yzaslavs at redhat.com yzaslavs at redhat.com
Thu Mar 6 06:20:31 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Fix group id on updating cache of users
......................................................................


core: Fix group id on updating cache of users

When db users cache is updated in refreshAllUsers, also user groups
membership is updated. If some group is not in local db, by mistake
externald id of group is set to user groups instead of internal one.
This patch sets empty GUID for groups that are not stored in local db.

Change-Id: I84d04420bc508f482c6ed4466e6937d8741bfe62
Bug-Url: https://bugzilla.redhat.com/1073235
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DbUserCacheManager.java
1 file changed, 12 insertions(+), 6 deletions(-)

Approvals:
  Martin Peřina: Verified
  Yair Zaslavsky: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I84d04420bc508f482c6ed4466e6937d8741bfe62
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list