[SSSD] Re: [PATCH] Better error message if sssctl is ran w/o activating the IFP responder

2016-08-15 Thread Petr Cech
On 08/16/2016 08:05 AM, Lukas Slebodnik wrote: On (12/08/16 12:24), Justin Stephenson wrote: Simple error message patch, resolves https://fedorahosted.org/sssd/ticket/3130 Kind regards, Justin Stephenson From 080f9639e120329d069d4f0ba5edcc776e0179c2 Mon Sep 17 00:00:00 2001 From: Justin St

[SSSD] Re: [PATCH] Better error message if sssctl is ran w/o activating the IFP responder

2016-08-15 Thread Lukas Slebodnik
On (12/08/16 12:24), Justin Stephenson wrote: >Simple error message patch, resolves >https://fedorahosted.org/sssd/ticket/3130 > >Kind regards, > >Justin Stephenson > >From 080f9639e120329d069d4f0ba5edcc776e0179c2 Mon Sep 17 00:00:00 2001 >From: Justin Stephenson >Date: Fri, 12 Aug 2016 12:12:57

[SSSD] Re: [PATCH] Better error message if sssctl is ran w/o activating the IFP responder

2016-08-15 Thread Justin Stephenson
Thanks Petr! Kind regards, Justin Stephenson On 08/15/2016 02:13 AM, Petr Cech wrote: On 08/15/2016 07:58 AM, Petr Cech wrote: On 08/12/2016 06:24 PM, Justin Stephenson wrote: Simple error message patch, resolves https://fedorahosted.org/sssd/ticket/3130 Kind regards, Justin Stephenson

[SSSD] Re: [PATCH] sssd-secrets systemd service and small fixes

2016-08-15 Thread Simo Sorce
On Mon, 2016-08-15 at 16:05 +0200, Jakub Hrozek wrote: > On Mon, Aug 15, 2016 at 04:01:13PM +0200, Jakub Hrozek wrote: > > Hi, > > > > attached are three small but important patches related to sssd-secrets. > > The context is that I started to write tests and manpage for > > sssd-secrets and notic

[SSSD] Re: [PATCH] sssd-secrets systemd service and small fixes

2016-08-15 Thread Fabiano Fidêncio
On Mon, Aug 15, 2016 at 4:05 PM, Jakub Hrozek wrote: > On Mon, Aug 15, 2016 at 04:01:13PM +0200, Jakub Hrozek wrote: >> Hi, >> >> attached are three small but important patches related to sssd-secrets. >> The context is that I started to write tests and manpage for >> sssd-secrets and noticed some

[SSSD] Re: [PATCH] sssd-secrets systemd service and small fixes

2016-08-15 Thread Jakub Hrozek
On Mon, Aug 15, 2016 at 04:01:13PM +0200, Jakub Hrozek wrote: > Hi, > > attached are three small but important patches related to sssd-secrets. > The context is that I started to write tests and manpage for > sssd-secrets and noticed some issues. I hope the patches themselves > offer a nice commit

[SSSD] Re: [PATCH SET] AD_PROVIDER: ad_enabled_domains

2016-08-15 Thread Petr Cech
On 08/12/2016 04:05 PM, Petr Cech wrote: On 08/12/2016 03:36 PM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 02:51:21PM +0200, Petr Cech wrote: On 08/12/2016 11:27 AM, Jakub Hrozek wrote: On Wed, Aug 10, 2016 at 08:54:25AM +0200, Petr Cech wrote: Sorry, I experienced some issue with mailing l

[SSSD] [PATCH] sssd-secrets systemd service and small fixes

2016-08-15 Thread Jakub Hrozek
Hi, attached are three small but important patches related to sssd-secrets. The context is that I started to write tests and manpage for sssd-secrets and noticed some issues. I hope the patches themselves offer a nice commit message. To test the socket activation, you can just install the RPMs bu

[SSSD] [SSSD} [PATCH] Remove no longer used code

2016-08-15 Thread Fabiano Fidêncio
Those 3 patches are from Jakub and I've just done some minor adjustments and add myself as co-author of the first 2 patches. CI has passed: http://sssd-ci.duckdns.org/logs/job/51/55/summary.html Best Regards, -- Fabiano Fidêncio From aa6204816cde0a7d75b9303916d038ed06e467ba Mon Sep 17 00:00:00 20

[SSSD] Re: [PATCH] Fix Japanese translation bug

2016-08-15 Thread Muneyuki Noguchi
Understood. I'm waiting for these language coordinators to add me to the language group: https://fedora.zanata.org/language/view/ja?dswid=-5318 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-

[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: [PATCHES] sssctl: print active server and server list

2016-08-15 Thread Lukas Slebodnik
On (15/08/16 11:49), Pavel Březina wrote: >On 08/15/2016 11:21 AM, Lukas Slebodnik wrote: >> > >> > +void sbus_request_reply_error(struct sbus_request *sbus_req, >> > + const char *error_name, >> > + const char *fmt, >> > +

[SSSD] Re: [PATCHES] sssctl: print active server and server list

2016-08-15 Thread Pavel Březina
On 08/15/2016 11:21 AM, Lukas Slebodnik wrote: +void sbus_request_reply_error(struct sbus_request *sbus_req, + const char *error_name, + const char *fmt, + ...); + It would be good to check format string at c

[SSSD] Re: [PATCHES] sssctl: print active server and server list

2016-08-15 Thread Lukas Slebodnik
On (12/08/16 11:40), Pavel Březina wrote: >On 08/12/2016 11:37 AM, Jakub Hrozek wrote: >> On Thu, Aug 11, 2016 at 11:49:07AM +0200, Pavel Březina wrote: >> > On 08/09/2016 11:21 AM, Jakub Hrozek wrote: >> > > On Fri, Aug 05, 2016 at 12:02:19PM +0200, Pavel Březina wrote: >> > > > On 07/25/2016 12:5

[SSSD] Re: [PATCH] Remove old DP interface

2016-08-15 Thread Lukas Slebodnik
On (12/08/16 10:35), Pavel Březina wrote: >On 08/11/2016 04:03 PM, Lukas Slebodnik wrote: >> On (20/07/16 11:10), Pavel Březina wrote: >> > CI: http://sssd-ci.duckdns.org/logs/job/50/01/summary.html >> > >> > The failure is about missing dependencies, unrelated to these patches. >> > >> > It depe

[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] Create pidfile after responders have started

2016-08-15 Thread Lukas Slebodnik
On (12/08/16 16:08), Victor Tapia wrote: > >>> >>> I haven't tried using "Type=notify", but I don't think it would make a >>> difference if we can't hold the notification signal until the responders >>> are running. >>> >> You would need to call sd_*notify function after initialisation of responder