Yair Zaslavsky has uploaded a new change for review.

Change subject: core: Fix multiple domains to support useDnsLookup flag
......................................................................

core: Fix multiple domains to support useDnsLookup flag

Even if useDnsLookup flag is set to true at
/etc/ovirt-engine/engine-manage-domains/engine-manage-domains.conf
The [domain_realm] with domain realm mapping must exist in case there is more 
than one managed domain.

Change-Id: Iebad744a40370c970d9462fb628092d9ebed13b1
bug-url: https://bugzilla.redhat.com/905291
Signed-off-by: Yair Zaslavsky <[email protected]>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/55/11655/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java
index 472667f..5f421ce 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java
@@ -121,7 +121,8 @@
             }
 
             // populate domain realms by domains
-            if (!useDnsLookup && line.matches(DOMAIN_REALM_SECTION)) {
+            //In case useDnsLookup is true
+            if (!(useDnsLookup  && realms.size() == 1) && 
line.matches(DOMAIN_REALM_SECTION)) {
                 line = appendDomainRealms(realms);
                 log.debug("setting domain realm");
             }


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iebad744a40370c970d9462fb628092d9ebed13b1
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