Re: [Freeipa-devel] [PATCH] 1043 fix ipa-replica-manage connect

2012-08-16 Thread Martin Kosek
On 08/15/2012 07:59 PM, Rob Crittenden wrote:
 A dn needed to be converted to a DN object.
 
 rob
 

Good catch, I re-tested all ipa-replica-manage actions and they worked fine.

ACK. Pushed to master.

Martin

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


[Freeipa-devel] [PATCH] 1043 fix ipa-replica-manage connect

2012-08-15 Thread Rob Crittenden

A dn needed to be converted to a DN object.

rob
From f242875811f887d80328eb84c96efdfd0ad5fa72 Mon Sep 17 00:00:00 2001
From: Rob Crittenden rcrit...@redhat.com
Date: Wed, 15 Aug 2012 13:56:04 -0400
Subject: [PATCH] Use DN object for Directory Manager in ipa-replica-manage
 connect command

---
 install/tools/ipa-replica-manage |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install/tools/ipa-replica-manage b/install/tools/ipa-replica-manage
index a4b8e8274c33e210297c8cfa069bc85e7c0fefb7..a135183334341d5cdecb5383c306e6deb2d1954f 100755
--- a/install/tools/ipa-replica-manage
+++ b/install/tools/ipa-replica-manage
@@ -502,7 +502,7 @@ def add_link(realm, replica1, replica2, dirman_passwd, options):
 
 except errors.NotFound:
 sys.exit(You cannot connect to a previously deleted master)
-repl1.setup_gssapi_replication(replica2, cn=Directory Manager, dirman_passwd)
+repl1.setup_gssapi_replication(replica2, DN(('cn', 'Directory Manager')), dirman_passwd)
 print Connected '%s' to '%s' % (replica1, replica2)
 
 def re_initialize(realm, thishost, fromhost, dirman_passwd):
-- 
1.7.10.4

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel