Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 12:30 +0300, Alexander Bokovoy wrote:
> On Tue, 20 Sep 2011, Martin Kosek wrote:
> > > Pushed to master, ipa-2-1.
> > > 
> > Alexander just noticed, that dirsrv stop in ipactl start fallback code
> > was not right either. One-liner patch attached.
> ACK as well.
> 
Pushed to master, ipa-2-1.

Martin

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Alexander Bokovoy
On Tue, 20 Sep 2011, Martin Kosek wrote:
> > Pushed to master, ipa-2-1.
> > 
> Alexander just noticed, that dirsrv stop in ipactl start fallback code
> was not right either. One-liner patch attached.
ACK as well.

-- 
/ Alexander Bokovoy

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 10:52 +0200, Martin Kosek wrote:
> On Tue, 2011-09-20 at 11:47 +0300, Alexander Bokovoy wrote:
> > On Tue, 20 Sep 2011, Martin Kosek wrote:
> > > Remove an invalid instance name passed to dirsrv service so that
> > > it is correctly stopped.
> > > 
> > > https://fedorahosted.org/freeipa/ticket/1800
> > ACK.
> > 
> 
> Pushed to master, ipa-2-1.
> 
> Martin

Alexander just noticed, that dirsrv stop in ipactl start fallback code
was not right either. One-liner patch attached.

Martin
>From 784abb812194883245e99d4c2d809072207503c3 Mon Sep 17 00:00:00 2001
From: Martin Kosek 
Date: Tue, 20 Sep 2011 11:15:52 +0200
Subject: [PATCH] dirsrv is not stopped correctly in the fallback

The previous patch fixed ipactl stop command. However, the dirsrv
stop in the ipactl start command fallback was not right either.

https://fedorahosted.org/freeipa/ticket/1800
---
 install/tools/ipactl |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/install/tools/ipactl b/install/tools/ipactl
index 63e69ec590e9f429a72123e874c9e72fb53e62b1..4055cf91bd6ec6b5a91c0e677ec4c7765c3bff6f 100755
--- a/install/tools/ipactl
+++ b/install/tools/ipactl
@@ -206,7 +206,7 @@ def ipa_start(options):
 except:
 pass
 try:
-svchandle.stop('dirsrv', capture_output=False)
+dirsrv.stop(capture_output=False)
 except:
 pass
 raise IpactlError("Aborting ipactl")
-- 
1.7.6

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 11:47 +0300, Alexander Bokovoy wrote:
> On Tue, 20 Sep 2011, Martin Kosek wrote:
> > Remove an invalid instance name passed to dirsrv service so that
> > it is correctly stopped.
> > 
> > https://fedorahosted.org/freeipa/ticket/1800
> ACK.
> 

Pushed to master, ipa-2-1.

Martin

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Alexander Bokovoy
On Tue, 20 Sep 2011, Martin Kosek wrote:
> Remove an invalid instance name passed to dirsrv service so that
> it is correctly stopped.
> 
> https://fedorahosted.org/freeipa/ticket/1800
ACK.

-- 
/ Alexander Bokovoy

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel