Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fix DB storing for principal names as provided by Active-Directory ...................................................................... core: Fix DB storing for principal names as provided by Active-Directory IPA provides principal correctly (format of user@domain). AD provides only the username. We would like to present at UI in format of principal name. Since this issue may occur with other directory services, I prefered to handle this at the LdapBrokerCommandBase , regardless of provider type. The upgrade script is required to handle already existing users. Bug-Url: https://bugzilla.redhat.com/888736 Change-Id: Ia9f8b60280d76b72a26f0d1feaf2650c4ef1ce3f Signed-off-by: Yair Zaslavsky <yzaslavs@redhat.com> --- A backend/manager/dbscripts/upgrade/03_02_0310_fix_principal_name_in_users_for_ad_users M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapBrokerCommandBase.java 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10844 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia9f8b60280d76b72a26f0d1feaf2650c4ef1ce3f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com>