Martin Kosek wrote:
On 08/10/2012 11:45 PM, Rob Crittenden wrote:
Robert Crittenden wrote:
This looks good, I just want to amend the commit with the tickets that this
addresses before pushing.

It seems that this covers 1670-1674. Is 1392 included as well?


----- Original Message -----
From: "John Dennis" <>
To: "freeipa-devel" <>
Sent: Friday, August 10, 2012 1:11:23 PM
Subject: [Freeipa-devel] [PATCH 76] Use DN objects instead of strings

Due to the size and disruptive nature of this patch it was reviewed and
tested outside the normal patch review process, see the thread "DN patch
and documentation" as well as other threads on this list. This patch is
the cumulative result of that review process.


Posting this since I haven't pushed the patch yet. I found this installing with

diff --git a/ipaserver/install/ b/ipaserver/install/
index 1bbee35..eebaa48 100644
--- a/ipaserver/install/
+++ b/ipaserver/install/
@@ -1048,7 +1048,7 @@ class CertDB(object):
          # Prepare a simple cert request
          req_dict = dict(PASSWORD=self.gen_password(),
-                        CERTNAME=DN('CN', nickname))
+                        CERTNAME=DN(('CN', nickname)))
          req_template = ipautil.SHARE_DIR + reqcfg + ".template"
          conf = ipautil.template_file(req_template, req_dict)
          fd = open(reqcfg, "w+")

Good job John, I think we are very close to push. I am attaching 2 patch to fix
the last major issues I found. One for "ipa-replica-manage list" which crashed
when winsync agreements were on and also few blocking issues in migration
plugin. Both patches are tested and issues resolved.

winsync replica agreements setup will need one more patch. But this issue is
not originated in DN - I will send a separate patch based on top of your work.


Squashed these in, along with my certs patch, and pushed the whole thing to master.


Freeipa-devel mailing list

Reply via email to