On 08/23/2013 02:23 PM, Tomas Babej wrote:
On 08/05/2013 05:43 PM, Martin Kosek wrote:
On 08/02/2013 03:32 PM, Tomas Babej wrote:
This patch makes sure that all edits to CS.cfg configuration file
are performed while pki-tomcatd service is stopped.
Introduces a new contextmanager stopped_service for handling
a general problem of performing a task that needs certain service
1) I think it would make sense to ideally run the steps updating
together, stop PKI before this group and start it after it finishes.
installer runs many service stops and starts which may be error prone,
especially given the fragile (and sometimes slow) java server handling.
2) I am thinking that stopped_service context manager could be as well
in ipaserver/install/service.py, as a context manager of the class.
every installer class could use that like:
# do something
I considered this approach, but this might introduce unnecessary errors if
we ever reorder the install steps in cainstance.py. I rather added two
explicit steps to stop and start the CA instance.
That way, context manager could just use self.name to avoid numerous
service names like:
Yes, but there are functions outside CAInstance class that leverage this
3) After I installed pki-ca, I saw no published CRL files:
# ls -la /var/lib/ipa/pki-ca/publish/
I am not sure what is the root cause, maybe some of the numerous
broke the publisher process.
I'm not seeing this with the updated version of the patch anymore.
Updated patch attached.
ACK for master & ipa-3-3. For ipa-3-2, the patch needs a rebase.
Freeipa-devel mailing list