[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 23:00), Lukas Slebodnik wrote: >On (12/08/16 16:41), Jakub Hrozek wrote: >>On Fri, Aug 12, 2016 at 02:38:44PM +0200, Lukas Slebodnik wrote: >>> On (12/08/16 14:13), Jakub Hrozek wrote: >>> >On Fri, Aug 12, 2016 at 01:47:04PM +0200, Lukas Slebodnik wrote: >>> >> On (12/08/16 07:39), Ste

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 16:41), Jakub Hrozek wrote: >On Fri, Aug 12, 2016 at 02:38:44PM +0200, Lukas Slebodnik wrote: >> On (12/08/16 14:13), Jakub Hrozek wrote: >> >On Fri, Aug 12, 2016 at 01:47:04PM +0200, Lukas Slebodnik wrote: >> >> On (12/08/16 07:39), Stephen Gallagher wrote: >> >> >On 08/12/2016 07:30

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

2016-08-12 Thread Justin Stephenson
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 -0400 Subject: [PATCH] SSSCTL: More helpful error

[SSSD] Re: [PATCH] Add support for disabling netlink use

2016-08-12 Thread Justin Stephenson
code patch and man page attached, also added the PATCH: prefix to the commit message for the code patch. Kind regards, Justin Stephenson On 08/12/2016 06:00 AM, Jakub Hrozek wrote: On Tue, Aug 09, 2016 at 12:04:56PM -0400, Justin Stephenson wrote: Hello, The attached patch fixes: https://f

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 02:38:44PM +0200, Lukas Slebodnik wrote: > On (12/08/16 14:13), Jakub Hrozek wrote: > >On Fri, Aug 12, 2016 at 01:47:04PM +0200, Lukas Slebodnik wrote: > >> On (12/08/16 07:39), Stephen Gallagher wrote: > >> >On 08/12/2016 07:30 AM, Lukas Slebodnik wrote: > >> >> On (12/08/1

[SSSD] Re: [PATCH] intg: test nested membership

2016-08-12 Thread Lukas Slebodnik
On (09/08/16 19:12), Lukas Slebodnik wrote: >On (09/08/16 14:54), Jakub Hrozek wrote: >>On Tue, Aug 09, 2016 at 12:36:11PM +0200, Jakub Hrozek wrote: >>> On Wed, Aug 03, 2016 at 09:56:40AM +0200, Lukas Slebodnik wrote: >>> > On (13/07/16 17:48), Lukas Slebodnik wrote: >>> > >ehlo, >>> > > >>> > >at

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

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 14:43), Muneyuki Noguchi wrote: >I could login with Fedora FAS Login. I'll try to use fedora.zanata.org. Let us now if the fix is merged there. We can still apply patch in git but I would prefer review another native speaker :-) LS ___ sssd-

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

2016-08-12 Thread Muneyuki Noguchi
I could login with Fedora FAS Login. I'll try to use fedora.zanata.org. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

[SSSD] Re: [PATCH 2/2] sdap: Skip exact duplicates when extending maps

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 16:14), Jakub Hrozek wrote: >On Fri, Aug 12, 2016 at 04:05:22PM +0200, Lukas Slebodnik wrote: >> On (10/08/16 20:59), Michal Židek wrote: >> >On 08/10/2016 08:36 PM, Lukas Slebodnik wrote: >> >> On (10/08/16 17:41), Michal Židek wrote: >> >> > Hi, >> >> > >> >> > see the attached patc

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

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 14:15), Muneyuki Noguchi wrote: >> On (12/08/16 11:29), Jakub Hrozek wrote: >> Should it be rather fixed in zanata[1] >> There woudl be proper review from native speakers >> >> LS >> >> [1] https://translate.zanata.org/ > >I couldn't find the sssd project in Zanata. >https://translat

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

2016-08-12 Thread Muneyuki Noguchi
> On (12/08/16 11:29), Jakub Hrozek wrote: > Should it be rather fixed in zanata[1] > There woudl be proper review from native speakers > > LS > > [1] https://translate.zanata.org/ I couldn't find the sssd project in Zanata. https://translate.zanata.org/search/sssd?dswid=5634 Can I create a pro

[SSSD] Re: [PATCH 2/2] sdap: Skip exact duplicates when extending maps

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 04:05:22PM +0200, Lukas Slebodnik wrote: > On (10/08/16 20:59), Michal Židek wrote: > >On 08/10/2016 08:36 PM, Lukas Slebodnik wrote: > >> On (10/08/16 17:41), Michal Židek wrote: > >> > Hi, > >> > > >> > see the attached patch. > >> > > >> > I modified the detection of du

[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] Warn if ad_server contains IP address

2016-08-12 Thread Justin Stephenson
Thanks Jakub for your help, updated patch set attached. Kind regards, Justin Stephenson On 08/12/2016 06:05 AM, Jakub Hrozek wrote: On Wed, Aug 10, 2016 at 12:09:10PM -0400, Justin Stephenson wrote: On 08/09/2016 05:41 AM, Jakub Hrozek wrote: On Fri, Aug 05, 2016 at 12:09:27PM -0400, Justin

[SSSD] Re: [PATCH] Create pidfile after responders have started

2016-08-12 Thread Victor Tapia
>> >> 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 responders > man 3 sd_notify > https://www.freedesktop.or

[SSSD] Re: [PATCH 2/2] sdap: Skip exact duplicates when extending maps

2016-08-12 Thread Lukas Slebodnik
On (10/08/16 20:59), Michal Židek wrote: >On 08/10/2016 08:36 PM, Lukas Slebodnik wrote: >> On (10/08/16 17:41), Michal Židek wrote: >> > Hi, >> > >> > see the attached patch. >> > >> > I modified the detection of duplicates when >> > extending the maps (sysdb_attr:ldap_attr). >> > >> > When we

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

2016-08-12 Thread Petr Cech
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 list. So I send it again. Forward

[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 SET] AD_PROVIDER: ad_enabled_domains

2016-08-12 Thread Jakub Hrozek
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 list. > > > So I send it again. > > > > > > Forwarded Message

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

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 14:34), Victor Tapia wrote: >Hi Lukas, > >> I tried to look into systemd documentation and I could not see >> anything in documentation which would say that it is a reliable >> way to use a pid file. (It might change in future if it is not documented) >> >> man systemd.service says >>

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

2016-08-12 Thread Petr Cech
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 list. So I send it again. Forwarded Message Subject: Re: [SSSD] Re: [PATCH SET] AD_PROVIDER: ad_enabled_domains Date: Tue, 9 Aug

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 14:13), Jakub Hrozek wrote: >On Fri, Aug 12, 2016 at 01:47:04PM +0200, Lukas Slebodnik wrote: >> On (12/08/16 07:39), Stephen Gallagher wrote: >> >On 08/12/2016 07:30 AM, Lukas Slebodnik wrote: >> >> On (12/08/16 13:24), Jakub Hrozek wrote: >> >>> Hi, >> >>> >> >>> a simple one-liner i

[SSSD] Re: [PATCH] Create pidfile after responders have started

2016-08-12 Thread Victor Tapia
Hi Lukas, > I tried to look into systemd documentation and I could not see > anything in documentation which would say that it is a reliable > way to use a pid file. (It might change in future if it is not documented) > > man systemd.service says >If set to forking, it is expected tha

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 01:47:04PM +0200, Lukas Slebodnik wrote: > On (12/08/16 07:39), Stephen Gallagher wrote: > >On 08/12/2016 07:30 AM, Lukas Slebodnik wrote: > >> On (12/08/16 13:24), Jakub Hrozek wrote: > >>> Hi, > >>> > >>> a simple one-liner is attached. > >> > >>>From c7bd0b7e695d031258ab

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

2016-08-12 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 11:40:24AM +0200, Pavel Březina wrote: > > I'm sorry, I sent wrong patches. How about now? > Now I have last two comments before I ack :) both are in the largest patch, the other can be considered acked. > From de872fd874b8395f526743f1836dea335726b3ff Mon Sep 17 00:00:00

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 07:39), Stephen Gallagher wrote: >On 08/12/2016 07:30 AM, Lukas Slebodnik wrote: >> On (12/08/16 13:24), Jakub Hrozek wrote: >>> Hi, >>> >>> a simple one-liner is attached. >> >>>From c7bd0b7e695d031258ab47d8c425c9d5843d4069 Mon Sep 17 00:00:00 2001 >>> From: Jakub Hrozek >>> Date: F

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Stephen Gallagher
On 08/12/2016 07:30 AM, Lukas Slebodnik wrote: > On (12/08/16 13:24), Jakub Hrozek wrote: >> Hi, >> >> a simple one-liner is attached. > >>From c7bd0b7e695d031258ab47d8c425c9d5843d4069 Mon Sep 17 00:00:00 2001 >> From: Jakub Hrozek >> Date: Fri, 12 Aug 2016 13:23:16 +0200 >> Subject: [PATCH] CONF

[SSSD] Re: [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 13:24), Jakub Hrozek wrote: >Hi, > >a simple one-liner is attached. >From c7bd0b7e695d031258ab47d8c425c9d5843d4069 Mon Sep 17 00:00:00 2001 >From: Jakub Hrozek >Date: Fri, 12 Aug 2016 13:23:16 +0200 >Subject: [PATCH] CONFIG: full_name_format is an allowed option for all domains > >--

[SSSD] [PATCH] CONFIG: full_name_format is an allowed option for all domains

2016-08-12 Thread Jakub Hrozek
Hi, a simple one-liner is attached. >From c7bd0b7e695d031258ab47d8c425c9d5843d4069 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Fri, 12 Aug 2016 13:23:16 +0200 Subject: [PATCH] CONFIG: full_name_format is an allowed option for all domains --- src/config/cfg_rules.ini | 1 + 1 file changed,

[SSSD] Re: [PATCH] Create pidfile after responders have started

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 11:48), Victor Tapia wrote: >Hi list, > >The attached patch covers ticket #3080 >(https://fedorahosted.org/sssd/ticket/3080). It's based on the following >commit: >https://git.fedorahosted.org/cgit/sssd.git/commit/?id=d19c4785215305e6eb5f2fa2fc503a2ba50d3f10 > > >Currently, systemd and

[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] Warn if ad_server contains IP address

2016-08-12 Thread Jakub Hrozek
On Wed, Aug 10, 2016 at 12:09:10PM -0400, Justin Stephenson wrote: > > On 08/09/2016 05:41 AM, Jakub Hrozek wrote: > > On Fri, Aug 05, 2016 at 12:09:27PM -0400, Justin Stephenson wrote: > > > Hi Lukas, > > > > > > I sent a response on July 6th but perhaps there was an issue with the > > > mailing

[SSSD] Re: [PATCH] Add support for disabling netlink use

2016-08-12 Thread Jakub Hrozek
On Tue, Aug 09, 2016 at 12:04:56PM -0400, Justin Stephenson wrote: > Hello, > > The attached patch fixes: https://fedorahosted.org/sssd/ticket/2860 > > This provides an option to allow sssd testing to ignore network status > changes from the netlink interface(remaining in offline mode, for exampl

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

2016-08-12 Thread Lukas Slebodnik
On (12/08/16 11:29), Jakub Hrozek wrote: >On Thu, Aug 11, 2016 at 11:46:23PM +0900, Muneyuki Noguchi wrote: >> Hi, >> >> The attached patch fixes Japanese translation bug >> by replacing "kiken", which means danger in Japanese, with "kigen". >> This bug was introduced on 19 Jul 2012. > >Hi Muneyuk

[SSSD] [PATCH] Create pidfile after responders have started

2016-08-12 Thread Victor Tapia
Hi list, The attached patch covers ticket #3080 (https://fedorahosted.org/sssd/ticket/3080). It's based on the following commit: https://git.fedorahosted.org/cgit/sssd.git/commit/?id=d19c4785215305e6eb5f2fa2fc503a2ba50d3f10 Currently, systemd and upstart set the service as 'started' before the r

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

2016-08-12 Thread Pavel Březina
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:55 PM, Pavel Březina wrote: On 07/20/2016 03:03 PM, Jakub Hroz

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

2016-08-12 Thread Jakub Hrozek
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:55 PM, Pavel Březina wrote: > > > > On 07/20/2016 03:03 PM, Jakub Hrozek wrote: > > > > > On Wed, Ju

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

2016-08-12 Thread Jakub Hrozek
On Thu, Aug 11, 2016 at 11:46:23PM +0900, Muneyuki Noguchi wrote: > Hi, > > The attached patch fixes Japanese translation bug > by replacing "kiken", which means danger in Japanese, with "kigen". > This bug was introduced on 19 Jul 2012. Hi Muneyuki-san, I don't speak Japanese, so I really don't

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

2016-08-12 Thread Jakub Hrozek
On Wed, Aug 10, 2016 at 08:54:25AM +0200, Petr Cech wrote: > Sorry, I experienced some issue with mailing list. > So I send it again. > > Forwarded Message > Subject: Re: [SSSD] Re: [PATCH SET] AD_PROVIDER: ad_enabled_domains > Date: Tue, 9 Aug 2016 17:29:38 +0200 > From: Petr Ce

[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] Remove old DP interface

2016-08-12 Thread Pavel Březina
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 depends on the sssctl failover patches due to changes in attaching dbus

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

2016-08-12 Thread Petr Cech
bump -- Petr^4 Čech ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

[SSSD] Re: [PATCH] LDAP: Fixing of removing netgroup from cache

2016-08-12 Thread Petr Cech
On 08/11/2016 08:53 AM, Petr Cech wrote: On 08/03/2016 12:34 PM, Michal Židek wrote: Two nitpicks, see inline. On 07/22/2016 02:34 PM, Petr Cech wrote: +static errno_t add_to_missing_attrs (TALLOC_CTX * mem_ctx, + struct sysdb_attrs *attrs, +