[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Jakub Hrozek
On Wed, Aug 17, 2016 at 04:28:38PM +0200, Lukas Slebodnik wrote: > On (17/08/16 16:23), Jakub Hrozek wrote: > >On Wed, Aug 17, 2016 at 03:41:29PM +0200, Jakub Hrozek wrote: > >> On Wed, Aug 17, 2016 at 01:16:48PM +0200, Petr Cech wrote: > >> > @@ -379,7 +394,12 @@ static void sdap_get_tgt_done(stru

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Lukas Slebodnik
On (17/08/16 16:23), Jakub Hrozek wrote: >On Wed, Aug 17, 2016 at 03:41:29PM +0200, Jakub Hrozek wrote: >> On Wed, Aug 17, 2016 at 01:16:48PM +0200, Petr Cech wrote: >> > @@ -379,7 +394,12 @@ static void sdap_get_tgt_done(struct tevent_req >> > *subreq) >> > >> > sdap_close_fd(&state->child

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Jakub Hrozek
On Wed, Aug 17, 2016 at 03:41:29PM +0200, Jakub Hrozek wrote: > On Wed, Aug 17, 2016 at 01:16:48PM +0200, Petr Cech wrote: > > @@ -379,7 +394,12 @@ static void sdap_get_tgt_done(struct tevent_req > > *subreq) > > > > sdap_close_fd(&state->child->io->read_from_child_fd); > > > > -teven

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Jakub Hrozek
On Wed, Aug 17, 2016 at 01:16:48PM +0200, Petr Cech wrote: > @@ -379,7 +394,12 @@ static void sdap_get_tgt_done(struct tevent_req *subreq) > > sdap_close_fd(&state->child->io->read_from_child_fd); > > -tevent_req_done(req); > +if (state->kill_te == NULL) { > +tevent_req_don

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Petr Cech
On 08/17/2016 01:10 PM, Pavel Březina wrote: On 08/15/2016 01:27 PM, Petr Cech wrote: On 08/15/2016 09:59 AM, Jakub Hrozek wrote: On Mon, Aug 15, 2016 at 09:47:27AM +0200, Petr Cech wrote: On 08/12/2016 04:13 PM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote:

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-17 Thread Pavel Březina
On 08/15/2016 01:27 PM, Petr Cech wrote: On 08/15/2016 09:59 AM, Jakub Hrozek wrote: On Mon, Aug 15, 2016 at 09:47:27AM +0200, Petr Cech wrote: On 08/12/2016 04:13 PM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote: On 08/12/2016 03:07 PM, Jakub Hrozek wrote: L

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-15 Thread Petr Cech
On 08/15/2016 09:59 AM, Jakub Hrozek wrote: On Mon, Aug 15, 2016 at 09:47:27AM +0200, Petr Cech wrote: On 08/12/2016 04:13 PM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote: On 08/12/2016 03:07 PM, Jakub Hrozek wrote: Logs now look like: [root@albireo sssd]#

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-15 Thread Jakub Hrozek
On Mon, Aug 15, 2016 at 09:47:27AM +0200, Petr Cech wrote: > On 08/12/2016 04:13 PM, Jakub Hrozek wrote: > > On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote: > > > On 08/12/2016 03:07 PM, Jakub Hrozek wrote: > > > > > Logs now look like: > > > > > > > > > > > > [root@albireo sssd]# grep

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-15 Thread Petr Cech
On 08/12/2016 04:13 PM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote: On 08/12/2016 03:07 PM, Jakub Hrozek wrote: Logs now look like: [root@albireo sssd]# grep 'child' sssd_ipa.cygnus.dev.log [child_handler_setup] (0x2000): Setting up signal handler up for pid

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 03:41:26PM +0200, Petr Cech wrote: > On 08/12/2016 03:07 PM, Jakub Hrozek wrote: > > > Logs now look like: > > > > > > > > [root@albireo sssd]# grep 'child' sssd_ipa.cygnus.dev.log > > > > [child_handler_setup] (0x2000): Setting up signal handler up for pid > > > > [18835]

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-12 Thread Petr Cech
On 08/12/2016 03:07 PM, Jakub Hrozek wrote: Logs now look like: > > [root@albireo sssd]# grep 'child' sssd_ipa.cygnus.dev.log > [child_handler_setup] (0x2000): Setting up signal handler up for pid [18835] > [child_handler_setup] (0x2000): Signal handler set up for pid [18835] > [set_tgt_child_tim

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 12:53:25PM +0200, Petr Cech wrote: > On 08/12/2016 11:06 AM, Jakub Hrozek wrote: > > On Thu, Aug 11, 2016 at 02:23:39PM +0200, Petr Cech wrote: > > > On 08/09/2016 02:49 PM, Jakub Hrozek wrote: > > > > On Tue, Aug 09, 2016 at 12:57:58PM +0200, Petr Cech wrote: > > > > > On 0

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-12 Thread Petr Cech
On 08/12/2016 11:06 AM, Jakub Hrozek wrote: On Thu, Aug 11, 2016 at 02:23:39PM +0200, Petr Cech wrote: On 08/09/2016 02:49 PM, Jakub Hrozek wrote: On Tue, Aug 09, 2016 at 12:57:58PM +0200, Petr Cech wrote: On 08/09/2016 11:26 AM, Jakub Hrozek wrote: On Mon, Aug 08, 2016 at 09:46:55AM +0200, P

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-12 Thread Jakub Hrozek
On Thu, Aug 11, 2016 at 02:23:39PM +0200, Petr Cech wrote: > On 08/09/2016 02:49 PM, Jakub Hrozek wrote: > > On Tue, Aug 09, 2016 at 12:57:58PM +0200, Petr Cech wrote: > > > On 08/09/2016 11:26 AM, Jakub Hrozek wrote: > > > > On Mon, Aug 08, 2016 at 09:46:55AM +0200, Petr Cech wrote: > > > > > > On

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-11 Thread Petr Cech
On 08/09/2016 02:49 PM, Jakub Hrozek wrote: On Tue, Aug 09, 2016 at 12:57:58PM +0200, Petr Cech wrote: On 08/09/2016 11:26 AM, Jakub Hrozek wrote: On Mon, Aug 08, 2016 at 09:46:55AM +0200, Petr Cech wrote: On 08/04/2016 05:01 PM, Petr Cech wrote: On 08/04/2016 04:35 PM, Petr Cech wrote: Hi l

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-09 Thread Jakub Hrozek
On Tue, Aug 09, 2016 at 12:57:58PM +0200, Petr Cech wrote: > On 08/09/2016 11:26 AM, Jakub Hrozek wrote: > > On Mon, Aug 08, 2016 at 09:46:55AM +0200, Petr Cech wrote: > > > > On 08/04/2016 05:01 PM, Petr Cech wrote: > > > > > > On 08/04/2016 04:35 PM, Petr Cech wrote: > > > > > > > > Hi list, > >

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-09 Thread Petr Cech
On 08/09/2016 11:26 AM, Jakub Hrozek wrote: On Mon, Aug 08, 2016 at 09:46:55AM +0200, Petr Cech wrote: > On 08/04/2016 05:01 PM, Petr Cech wrote: > > On 08/04/2016 04:35 PM, Petr Cech wrote: > > > Hi list, > > > > > > there is the first version of patch for [1]. I need > > > to investigate if

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-09 Thread Jakub Hrozek
On Mon, Aug 08, 2016 at 09:46:55AM +0200, Petr Cech wrote: > On 08/04/2016 05:01 PM, Petr Cech wrote: > > On 08/04/2016 04:35 PM, Petr Cech wrote: > > > Hi list, > > > > > > there is the first version of patch for [1]. I need > > > to investigate if we have the same issue in other > > > *_childs.

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-08 Thread Petr Cech
On 08/04/2016 05:01 PM, Petr Cech wrote: On 08/04/2016 04:35 PM, Petr Cech wrote: Hi list, there is the first version of patch for [1]. I need to investigate if we have the same issue in other *_childs. I tested it with sleep() added into ldap_child code. [1] https://fedorahosted.org/sssd/tic

[SSSD] Re: [PATCH 1/2] LDAP: Adding support for SIGTERM signal

2016-08-04 Thread Petr Cech
On 08/04/2016 04:35 PM, Petr Cech wrote: Hi list, there is the first version of patch for [1]. I need to investigate if we have the same issue in other *_childs. I tested it with sleep() added into ldap_child code. [1] https://fedorahosted.org/sssd/ticket/3106 I know, it is not perfect. For