Yair Zaslavsky has uploaded a new change for review.

Change subject: core: fix useDnsLookup behavior
......................................................................

core: fix useDnsLookup behavior

This patch fixes the usage of "useDnsLookup" flag
at engine-manage-domains configuration.
only When --ldap-servers is used, this flag should be ignored if set to true.

Change-Id: Ic96e2f9fafcac1e6ba952a198bfb9cecc160ba8b
Bug-Url: https://bugzilla.redhat.com/1062172
Signed-off-by: Yair Zaslavsky <[email protected]>
---
M 
backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
1 file changed, 6 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/89/24189/1

diff --git 
a/backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
 
b/backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
index efd25b8..2091958 100644
--- 
a/backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
+++ 
b/backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java
@@ -39,6 +39,7 @@
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.Date;
 import java.util.HashMap;
 import java.util.List;
@@ -720,7 +721,11 @@
             try {
                 log.info("Creating kerberos configuration for domain(s): " + 
gssapiDomainsString);
                 useDnsLookup = utilityConfiguration.getUseDnsLookup();
-                krbConfCreator = new KrbConfCreator(gssapiDomainsString, 
useDnsLookup, ldapServersPerGSSAPIDomains);
+                if (args.contains(ARG_LDAP_SERVERS)) {
+                    krbConfCreator = new KrbConfCreator(gssapiDomainsString, 
useDnsLookup, ldapServersPerGSSAPIDomains);
+                } else {
+                    krbConfCreator = new KrbConfCreator(gssapiDomainsString, 
useDnsLookup, Collections.EMPTY_MAP);
+                }
                 StringBuffer buffer = null;
                 buffer = krbConfCreator.parse("y");
                 
krbConfCreator.toFile(utilityConfiguration.getkrb5confFilePath() + 
TESTING_KRB5_CONF_SUFFIX, buffer);


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic96e2f9fafcac1e6ba952a198bfb9cecc160ba8b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to