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
On Tue, 2012-07-24 at 11:42 -0700, Matthew Harmsen wrote:
This patch documents continued implementation of the PKI Deployment
Framework based upon the revised filesystem layout documented here:
*
http://pki.fedoraproject.org/wiki/PKI_Instance_Deployment#CA_.2F_KRA_.2F_...
This patch addresses the following issues:
* PKI TRAC Ticket #244 - Non-user-friendly message when deleting
non-existent subsystem with pkidestroy
* PKI TRAC Ticket #246 - Incorrect parameter names in pkispawn
configuration
* PKI TRAC Ticket #248 - pki_ds_database should not be a DN
* PKI TRAC Ticket #249 - pki_ds_base_dn causing pkispawn failure
* PKI TRAC Ticket #250 - Creating/removing custom instances
should not require http/ajp ports
* PKI TRAC Ticket #251 - Instance name may conflict with other
files
* PKI TRAC Ticket #253 - Fix pki-destroy removal of
'/var/log/pki/{pki_instance_id}' directory . . .
* PKI TRAC Ticket #254 - Dogtag 10: Fix spec file to build
successfully via mock on Fedora 17 . . .
* PKI TRAC Ticket #255 - Missing resteasy-atom-provider.jar
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.
_______________________________________________
Pki-devel mailing list
Pki-devel(a)redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel