Change in ovirt-engine[master]: aaa: Add LdapServers into domain separated config values

yzaslavs at redhat.com yzaslavs at redhat.com
Fri Jun 20 06:54:28 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: aaa: Add LdapServers into domain separated config values
......................................................................


aaa: Add LdapServers into domain separated config values

1) Adds LdapServers into domain separated config values
   (EngineExtensionsManager.multipleValuesKeys)

2) LdapServers value may be empty, which is valid in case
   that --ldap-servers in engine-manage-domains was not specified,
   so EngineExtensionsManager.getValue was adapted to support empty
   values

Topic: AAA
Change-Id: I9ff932f1a8dec1ba2f9c834e4e62601202ee97ec
Bug-Url: https://bugzilla.redhat.com/1111071
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/extensionsmgr/EngineExtensionsManager.java
1 file changed, 15 insertions(+), 12 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9ff932f1a8dec1ba2f9c834e4e62601202ee97ec
Gerrit-PatchSet: 2
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: Oved Ourfali <oourfali 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