[Freeipa-devel] [PATCH, 2.1] 0021 Fedora 16 and systemd support

[Simo Sorce]

On Fri, 2011-10-14 at 16:14 -0400, Simo Sorce wrote:
> On Fri, 2011-10-14 at 13:56 -0400, Simo Sorce wrote:
> > On Mon, 2011-10-10 at 17:07 +0300, Alexander Bokovoy wrote:
> > > On Mon, 10 Oct 2011, Alexander Bokovoy wrote:
> > > > rebased, updated package dependencies, and verified against 
> > > > Fedora 16+updates-testing.
> > > > 
> > > > This patch is for ipa-2-1 branch. I need to do few cosmetic changes in 
> > > > freeipa.spec.in to accomodate it to 3.0 (master branch) as ipa_kpasswd 
> > > > is gone there.
> > > Forgot to add that altogether this patch fixes:
> > > 
> > > https://fedorahosted.org/freeipa/ticket/1192 -- support systemd
> > > https://fedorahosted.org/freeipa/ticket/1651 -- update F16 dependencies
> > > https://fedorahosted.org/freeipa/ticket/1871 -- not setting HOSTNAME if it was missing from the configuration file
> > > 
> > > The latter one is integrated within the systemd patch because the same 
> > > function is re-used for editing systemd service files and 
> > > /etc/sysconfig/krb5kdc and it simply makes little sense to separate 
> > > them as without editing systemd services for dirsrv, one cannot start 
> > > dirsrv with number of file descriptors required by IPA defaults, and 
> > > krb5kdc configuration should be written properly before krb5kdc is 
> > > started as its systemd service unit uses parameters from the 
> > > configuration file.
> > 
> > Attached a rebased patch with the modifications needed to apply it on
> > master.
> > 
> > Everything seem to work on master but I haven't tested ipa-2-1 so this
> > is a partial ACK of the original patch as well.
> 
> A bit of bad news, I restarted the machine and I am having issue
> properly restarting services.
> This patch is still better than nothing as otherwise nothing works at
> all on f16, but we need to work out why starting services is unreliable.

Ok found the issue and it is a bug in the conversion to systemd.
I opened ticket #1990 for this.

Attached find a rebased patch that fixes enough of the bug to let the
server work (they keytab part), but it doesn't address the ulimit part.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-2-Add-support-for-systemd-environments-and-use-it-to-s.patch
Type: text/x-patch
Size: 32827 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111014/ec9dd5c1/attachment.bin>