<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <tt>NOTE:  This version of the patch applies the fixes previously
      requested:<br>
    </tt>
    <blockquote>
      <pre wrap="">Comments: 
1. As noted, please open trac ticket to remove custom args from the
command line from pkispawn, and add -i instance mandatory parameter to
pkidestroy.

2. In pkihelper, you add logic that separates out the case of the
subordinate CA because subordinate is not defined for any subsystem
other than a CA.  You could leave it in the same conditional, and
simplify the conditional to be: if (clone) or (!ca) or (subordinate):

3. Please shorten the name of
util.namespace.pki_instance_name_collision_detection_avoidance()

You can make those changes and check in without re-review.  As long as
you test of course to confirm you have not broken anything.

Ade</pre>
      <tt></tt></blockquote>
    <tt><br>
      This patch documents continued implementation of the PKI
      Deployment Framework based upon the revised filesystem layout
      documented here:<br>
    </tt>
    <ul>
      <li><tt><a class="moz-txt-link-freetext"
href="http://pki.fedoraproject.org/wiki/PKI_Instance_Deployment#CA_.2F_KRA_.2F_OCSP_.2F_RA_.2F_TKS_.2F_TPS">http://pki.fedoraproject.org/wiki/PKI_Instance_Deployment#CA_.2F_KRA_.2F_OCSP_.2F_RA_.2F_TKS_.2F_TPS</a></tt></li>
    </ul>
    <tt>This patch addresses the following issues:<br>
    </tt>
    <ul>
      <li>PKI TRAC Ticket #244 - Non-user-friendly message when deleting
        non-existent subsystem with pkidestroy</li>
      <li>PKI TRAC Ticket #246 - Incorrect parameter names in pkispawn
        configuration</li>
      <li>PKI TRAC Ticket #248 - pki_ds_database should not be a DN</li>
      <li>PKI TRAC Ticket #249 - pki_ds_base_dn causing pkispawn failure</li>
      <li>PKI TRAC Ticket #250 - Creating/removing custom instances
        should not require http/ajp ports</li>
      <li>PKI TRAC Ticket #251 - Instance name may conflict with other
        files</li>
      <li>PKI TRAC Ticket #253 - Fix pki-destroy removal of
        '/var/log/pki/{pki_instance_id}' directory . . .</li>
      <li>PKI TRAC Ticket #254 - Dogtag 10: Fix spec file to build
        successfully via mock on Fedora 17 . . .</li>
      <li>PKI TRAC Ticket #255 - Missing resteasy-atom-provider.jar</li>
      <li>PKI TRAC Ticket #260 - Dogtag 10: Change the layout of
        'pki_instance_id' . . .<br>
      </li>
    </ul>
    <b>WARNING:  As this patch does fundamentally change instance names
      from "[pki_admin_domain_name-]pki_instance_name" to
      "pki_instance_name[.pki_admin_domain_name]", users of this patch
      must FIRST remove all old PKI instances.</b><br>
    <br>
  </body>
</html>