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

2011-10-24 Thread Martin Kosek
On Tue, 2011-10-18 at 15:29 +0200, Martin Kosek wrote: On Tue, 2011-10-18 at 15:48 +0300, Alexander Bokovoy wrote: On Tue, 18 Oct 2011, Alexander Bokovoy wrote: ipa.init was removed from the git, but it was never moved to init/SystemV/. It should have been moved (rm+new file). I'll

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

2011-10-18 Thread Martin Kosek
On Mon, 2011-10-17 at 14:21 +0300, Alexander Bokovoy wrote: On Fri, 14 Oct 2011, Simo Sorce wrote: 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

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

2011-10-18 Thread Alexander Bokovoy
On Tue, 18 Oct 2011, Martin Kosek wrote: I tested our most recent master with simo's rebased patch and your patch 0004-Spin-for-connection-success-also-when-socket-is-not-.patch. It looks very good, I hit just few issues: 1) ipa service reports inactive (dead) status even though LDAP server

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

2011-10-18 Thread Alexander Bokovoy
On Tue, 18 Oct 2011, Alexander Bokovoy wrote: ipa.init was removed from the git, but it was never moved to init/SystemV/. It should have been moved (rm+new file). I'll check what's happening there, maybe Simo's patch omitted that one?

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

2011-10-18 Thread Simo Sorce
On Tue, 2011-10-18 at 14:27 +0300, Alexander Bokovoy wrote: On Tue, 18 Oct 2011, Martin Kosek wrote: ipa.init was removed from the git, but it was never moved to init/SystemV/. It should have been moved (rm+new file). I'll check what's happening there, maybe Simo's patch omitted that one?

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

2011-10-18 Thread Martin Kosek
On Tue, 2011-10-18 at 15:48 +0300, Alexander Bokovoy wrote: On Tue, 18 Oct 2011, Alexander Bokovoy wrote: ipa.init was removed from the git, but it was never moved to init/SystemV/. It should have been moved (rm+new file). I'll check what's happening there, maybe Simo's patch omitted

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

2011-10-18 Thread Alexander Bokovoy
On Tue, 18 Oct 2011, Martin Kosek wrote: 1) When ipa is not configured, it is ok that ipa.service status returns error. However, I still got ipa.service status error after the ipa was configured: # systemctl status ipa.service ipa.service - Identity, Policy, Audit Loaded: loaded

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

2011-10-17 Thread Alexander Bokovoy
On Fri, 14 Oct 2011, Simo Sorce wrote: 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

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

2011-10-17 Thread Simo Sorce
On Mon, 2011-10-17 at 14:21 +0300, Alexander Bokovoy wrote: On Fri, 14 Oct 2011, Simo Sorce wrote: 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

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

2011-10-14 Thread Simo Sorce
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

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

2011-10-14 Thread Simo Sorce
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

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

2011-10-10 Thread Alexander Bokovoy
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