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

2016-08-16 Thread Justin Stephenson
On 08/16/2016 10:00 AM, Stephen Gallagher wrote: On 08/12/2016 11:26 AM, Justin Stephenson wrote: 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,

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

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 04:29:43PM +0200, Petr Cech wrote: > On 08/16/2016 03:58 PM, Stephen Gallagher wrote: > > On 08/16/2016 09:26 AM, Jakub Hrozek wrote: > > > > On Tue, Aug 16, 2016 at 03:17:19PM +0200, Petr Cech wrote: > > > > > > From 24d32d0eb12ddc433e64ffd6411e9e13f0067b35 Mon Sep 17

[SSSD] Re: [PATCH] SPEC: Own the secrets DB path

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 05:01:55PM +0200, Jakub Hrozek wrote: > To test, run: > rpm -qf /var/lib/sss/secrets/ > the directory was unowned before and should be owned by sssd-common now. CI: http://sssd-ci.duckdns.org/logs/job/51/74/summary.html By the way Fabiano asked on IRC why I owned the

[SSSD] Re: [PATCH] Warn if ad_server contains IP address

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 05:50:19PM +0200, Jakub Hrozek wrote: > On Fri, Aug 12, 2016 at 10:08:19AM -0400, Justin Stephenson wrote: > > Thanks Jakub for your help, updated patch set attached. > > ACK to both, I'll push the patches when CI finishes. CI:

[SSSD] Re: [PATCH] Warn if ad_server contains IP address

2016-08-16 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 10:08:19AM -0400, Justin Stephenson wrote: > Thanks Jakub for your help, updated patch set attached. ACK to both, I'll push the patches when CI finishes. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

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

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 04:16:08PM +0200, Petr Cech wrote: > On 08/16/2016 04:06 PM, Justin Stephenson wrote: > > Updated patch attached. > > > > Kind regards, > > Justin Stephenson > > Thanks, Justin. > > Obviously ACK. Umm..I'm by no means a UI designer, but does HINT in ALL CAPS look

[SSSD] [PATCH] SPEC: Own the secrets DB path

2016-08-16 Thread Jakub Hrozek
To test, run: rpm -qf /var/lib/sss/secrets/ the directory was unowned before and should be owned by sssd-common now. >From a398c91ef196be680f6c2d5b2d5251e060005ec0 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 16 Aug 2016 16:45:36 +0200 Subject: [PATCH] SPEC: Own

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

2016-08-16 Thread Fabiano Fidêncio
On Mon, Aug 15, 2016 at 4:17 PM, Fabiano Fidêncio wrote: > 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] Re: [PATCH SET] AD_PROVIDER: ad_enabled_domains

2016-08-16 Thread Petr Cech
On 08/16/2016 03:58 PM, Stephen Gallagher wrote: On 08/16/2016 09:26 AM, Jakub Hrozek wrote: > On Tue, Aug 16, 2016 at 03:17:19PM +0200, Petr Cech wrote: From 24d32d0eb12ddc433e64ffd6411e9e13f0067b35 Mon Sep 17 00:00:00 2001 From: Petr Cech Date: Fri, 13 May

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

2016-08-16 Thread Petr Cech
On 08/16/2016 04:06 PM, Justin Stephenson wrote: Updated patch attached. Kind regards, Justin Stephenson Thanks, Justin. Obviously ACK. On 08/16/2016 02:05 AM, Lukas Slebodnik wrote: On (12/08/16 12:24), Justin Stephenson wrote: Simple error message patch, resolves

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

2016-08-16 Thread Justin Stephenson
Updated patch attached. Kind regards, Justin Stephenson On 08/16/2016 02: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

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

2016-08-16 Thread Stephen Gallagher
On 08/12/2016 11:26 AM, Justin Stephenson wrote: > 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,

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

2016-08-16 Thread Stephen Gallagher
On 08/16/2016 09:26 AM, Jakub Hrozek wrote: > On Tue, Aug 16, 2016 at 03:17:19PM +0200, Petr Cech wrote: From 24d32d0eb12ddc433e64ffd6411e9e13f0067b35 Mon Sep 17 00:00:00 2001 From: Petr Cech Date: Fri, 13 May 2016 05:21:07 -0400 Subject: [PATCH 1/5]

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

2016-08-16 Thread Justin Stephenson
Thanks for the info, yes please go ahead and squash them. Kind regards, Justin Stephenson On 08/16/2016 09:32 AM, Jakub Hrozek wrote: On Fri, Aug 12, 2016 at 11:26:15AM -0400, Justin Stephenson wrote: code patch and man page attached, also added the PATCH: prefix to the commit message for the

[SSSD] Re: [PATCH] LDAP: Log autofs rfc2307 config changes only with enabled responder

2016-08-16 Thread Petr Cech
On 08/16/2016 01:41 PM, Lukas Slebodnik wrote: ehlo, attached patch should fix annoying message with disabled autofs responder. LS Hi Lukas, LGTM, I am waiting for CI. Regards -- Petr^4 Čech ___ sssd-devel mailing list

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

2016-08-16 Thread Jakub Hrozek
On Fri, Aug 12, 2016 at 11:26:15AM -0400, Justin Stephenson wrote: > code patch and man page attached, also added the PATCH: prefix to the commit > message for the code patch. You don't have to add the PATCH prefix, git adds that automatically when you format the patch with "git format-patch".

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

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 03:17:19PM +0200, Petr Cech wrote: > > > From 24d32d0eb12ddc433e64ffd6411e9e13f0067b35 Mon Sep 17 00:00:00 2001 > > > From: Petr Cech > > > Date: Fri, 13 May 2016 05:21:07 -0400 > > > Subject: [PATCH 1/5] AD_PROVIDER: Add ad_enabled_domains option > > >

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

2016-08-16 Thread Petr Cech
On 08/16/2016 02:52 PM, Jakub Hrozek wrote: On Mon, Aug 15, 2016 at 04:03:17PM +0200, Petr Cech wrote: 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:

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

2016-08-16 Thread Lukas Slebodnik
On (16/08/16 14:57), Lukas Slebodnik wrote: >On (15/08/16 11:09), Lukas Slebodnik wrote: >>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

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

2016-08-16 Thread Lukas Slebodnik
On (15/08/16 11:09), Lukas Slebodnik wrote: >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

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

2016-08-16 Thread Lukas Slebodnik
On (16/08/16 14:21), Lukas Slebodnik wrote: >On (16/08/16 13:30), Pavel Březina wrote: >>On 08/16/2016 12:55 PM, Lukas Slebodnik wrote: >>> On (16/08/16 11:32), Pavel Březina wrote: >>> > On 08/15/2016 01:21 PM, Lukas Slebodnik wrote: >>> > > On (15/08/16 11:49), Pavel Březina wrote: >>> > > > On

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-08-16 Thread Petr Cech
On 08/16/2016 01:22 PM, Petr Cech wrote: On 08/16/2016 01:02 PM, Lukas Slebodnik wrote: On (16/08/16 12:52), Petr Cech wrote: On 08/16/2016 10:15 AM, Jakub Hrozek wrote: On Tue, Aug 16, 2016 at 09:50:19AM +0200, Petr Cech wrote: Hello list, I am solving ticket [1] now. There are three

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

2016-08-16 Thread Lukas Slebodnik
On (16/08/16 13:30), Pavel Březina wrote: >On 08/16/2016 12:55 PM, Lukas Slebodnik wrote: >> On (16/08/16 11:32), Pavel Březina wrote: >> > On 08/15/2016 01:21 PM, Lukas Slebodnik wrote: >> > > On (15/08/16 11:49), Pavel Březina wrote: >> > > > On 08/15/2016 11:21 AM, Lukas Slebodnik wrote: >> > >

[SSSD] [PATCH] LDAP: Log autofs rfc2307 config changes only with enabled responder

2016-08-16 Thread Lukas Slebodnik
ehlo, attached patch should fix annoying message with disabled autofs responder. LS >From 7eaeb08fe25938734cd69e3a0e625fa5befa6329 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Tue, 16 Aug 2016 13:32:06 +0200 Subject: [PATCH] LDAP: Log autofs rfc2307 config changes

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

2016-08-16 Thread Pavel Březina
On 08/16/2016 12:55 PM, Lukas Slebodnik wrote: On (16/08/16 11:32), Pavel Březina wrote: On 08/15/2016 01:21 PM, Lukas Slebodnik wrote: 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, +

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-08-16 Thread Lukas Slebodnik
On (16/08/16 12:52), Petr Cech wrote: >On 08/16/2016 10:15 AM, Jakub Hrozek wrote: >> On Tue, Aug 16, 2016 at 09:50:19AM +0200, Petr Cech wrote: >> > Hello list, >> > >> > I am solving ticket [1] now. There are three >> > points mentioned. A have prepared patches for >> > the first two. I would

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

2016-08-16 Thread Lukas Slebodnik
On (16/08/16 11:32), Pavel Březina wrote: >On 08/15/2016 01:21 PM, Lukas Slebodnik wrote: >> 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, >> > > > +

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-08-16 Thread Petr Cech
On 08/16/2016 10:15 AM, Jakub Hrozek wrote: On Tue, Aug 16, 2016 at 09:50:19AM +0200, Petr Cech wrote: Hello list, I am solving ticket [1] now. There are three points mentioned. A have prepared patches for the first two. I would like to ask anybody it is right or if I miss something. The

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

2016-08-16 Thread Lukas Slebodnik
On (12/08/16 16:30), Lukas Slebodnik wrote: >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:

[SSSD] Re: [PATCH] DP: Add log message for get account info

2016-08-16 Thread Jakub Hrozek
On Tue, Aug 16, 2016 at 10:04:27AM +0200, Petr Cech wrote: > On 08/16/2016 09:20 AM, Petr Cech wrote: > > On 08/16/2016 09:17 AM, Lukas Slebodnik wrote: > > > ehlo, > > > > > > Petr improved debug messages in > > >

[SSSD] Re: [PATCH] DP: Add log message for get account info

2016-08-16 Thread Petr Cech
On 08/16/2016 09:20 AM, Petr Cech wrote: On 08/16/2016 09:17 AM, Lukas Slebodnik wrote: ehlo, Petr improved debug messages in https://git.fedorahosted.org/cgit/sssd.git/commit/?id=376eaf187c13c2a1eaea0ffbdd970b6b563ab74c but it was removed as part of DP refactoring. LS Hi Lukas, thanks.

[SSSD] [PATCH] DP: Add log message for get account info

2016-08-16 Thread Lukas Slebodnik
ehlo, Petr improved debug messages in https://git.fedorahosted.org/cgit/sssd.git/commit/?id=376eaf187c13c2a1eaea0ffbdd970b6b563ab74c but it was removed as part of DP refactoring. LS >From 9c9004acd9bffaf23efdf5d6b5290adaafe2331e Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik

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

2016-08-16 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

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

2016-08-16 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,