[SSSD] Re: [PATCH] CONFIGURE: Inform about optional build dependencies

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 06:06:26PM +0200, Jakub Hrozek wrote: > On Thu, Jul 07, 2016 at 01:05:39PM +0200, Lukas Slebodnik wrote: > > On (05/07/16 07:59), Jakub Hrozek wrote: > > >On Fri, Jul 01, 2016 at 08:45:53AM +0200, Lukas Slebodnik wrote: > > >> ehlo, > &g

[SSSD] Re: [PATCH] CONFIGURE: Inform about optional build dependencies

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 01:05:39PM +0200, Lukas Slebodnik wrote: > On (05/07/16 07:59), Jakub Hrozek wrote: > >On Fri, Jul 01, 2016 at 08:45:53AM +0200, Lukas Slebodnik wrote: > >> ehlo, > >> > >> We usually inform about optional build dependencies at conf

[SSSD] Fwd: Re: [PATCH] LDAP: Lookup services by all protocols unless a protocol is specified

2016-07-07 Thread Jakub Hrozek
sssd-devel seems to be eating our mails lately, resending - Forwarded Message - From: "Jakub Hrozek" To: sssd-devel@lists.fedorahosted.org Sent: Thursday, July 7, 2016 4:44:30 PM Subject: Re: [SSSD] Re: [PATCH] LDAP: Lookup services by all protocols unless a protocol is spe

[SSSD] Re: [PATCH] sudo: solve problems with fully qualified names

2016-07-07 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 06:20:00PM +0200, Jakub Hrozek wrote: > On Wed, Jul 06, 2016 at 03:23:26PM +0200, Jakub Hrozek wrote: > > On Wed, Jun 01, 2016 at 11:52:44AM +0200, Pavel Březina wrote: > > > On 05/31/2016 01:44 PM, Jakub Hrozek wrote: > > > > On Fri, May 27, 2

[SSSD] Re: [PATCH] LDAP: Lookup services by all protocols unless a protocol is specified

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 12:39:28PM +0200, Pavel Březina wrote: > On 07/07/2016 12:34 PM, Jakub Hrozek wrote: > > On Thu, Jul 07, 2016 at 10:16:03AM +0200, Sumit Bose wrote: > > > resend > > > - Forwarded message from Sumit Bose - > > > > &

[SSSD] Re: [PATCH][PUSHED] MAN: Include idmap_sss.8.xml in the manpage sources

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 01:06:31PM +0200, Lukas Slebodnik wrote: > On (07/07/16 13:05), Pavel Březina wrote: > >On 06/30/2016 03:37 PM, Lukas Slebodnik wrote: > >> On (30/06/16 09:28), Sumit Bose wrote: > >> > On Wed, Jun 29, 2016 at 11:23:55PM +0200, Jakub Hrozek

[SSSD] Re: [PATCH] PAM/KRB5: optional otp and password prompting

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 12:39:01PM +0200, Jakub Hrozek wrote: > The prompts were changes as Nathaniel suggested and the basic sanity > checks worked with this patch. I'm going to push with Nathaniel's RB. * master: 78027feeb56d6fe216f699be

[SSSD] Re: [PATCH] PAM/KRB5: optional otp and password prompting

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 10:30:21AM +0200, Sumit Bose wrote: > On Wed, Jul 06, 2016 at 10:03:12AM -0400, Nathaniel McCallum wrote: > > On Wed, 2016-07-06 at 15:55 +0200, Sumit Bose wrote: > > > On Wed, Jul 06, 2016 at 09:03:45AM -0400, Nathaniel McCallum wrote: > > > > > > > > Patch WFM and looks c

[SSSD] Re: [PATCH] LDAP: Lookup services by all protocols unless a protocol is specified

2016-07-07 Thread Jakub Hrozek
line > In-Reply-To: <20160705103025.GB24232@hendrix> > User-Agent: Mutt/1.6.1 (2016-04-27) > > On Tue, Jul 05, 2016 at 12:30:25PM +0200, Jakub Hrozek wrote: > > Hi, > > > > the attached patch makes service lookups great again. > > > > To repr

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-07 Thread Jakub Hrozek
On Thu, Jul 07, 2016 at 11:45:42AM +0200, Lukas Slebodnik wrote: > On (07/07/16 11:39), Pavel Březina wrote: > >On 07/06/2016 11:21 PM, Sumit Bose wrote: > >> > >> ok, CI passed http://sssd-ci.duckdns.org/logs/job/47/41/summary.html > >> > >> ACK > > > >I went through the code changes and it look

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 06:34:32PM +0200, Jakub Hrozek wrote: > On Wed, Jul 06, 2016 at 11:13:02AM +0200, Lukas Slebodnik wrote: > > On (03/07/16 12:50), Jakub Hrozek wrote: > > >On Fri, Jul 01, 2016 at 11:01:53PM +0200, Jakub Hrozek wrote: > > >> On Fri, Jul 01, 2

[SSSD] RFC: Release notes for the 1.14.0 release

2016-07-06 Thread Jakub Hrozek
Hi, I prepared the release notes for the 1.14.0 release here: https://fedorahosted.org/sssd/wiki/Releases/Notes-1.14.0 please comment or edit them. The notes are written with the assumption that several pending patches (including prompting patches, enterprise principals or the sysdb refactorin

[SSSD] Re: [PATCHES] krb5 config snippets

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 06:24:08PM +0200, Jakub Hrozek wrote: > On Tue, Jul 05, 2016 at 12:54:34PM +0200, Sumit Bose wrote: > > Hi, > > > > the following 4 patches include fixes for > > ttps://fedorahosted.org/sssd/ticket/2788 and > > https://fedorahosted.o

[SSSD] Re: IPA: enable enterprise principals if server supports them

2016-07-06 Thread Jakub Hrozek
On Tue, Jul 05, 2016 at 10:06:15PM +0200, Sumit Bose wrote: > On Tue, Jul 05, 2016 at 07:15:03PM +0200, Jakub Hrozek wrote: > > On Tue, Jul 05, 2016 at 12:37:22PM +0200, Sumit Bose wrote: > > > Hi, > > > > > > this patch set should solve https://fedorahosted.o

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 11:13:02AM +0200, Lukas Slebodnik wrote: > On (03/07/16 12:50), Jakub Hrozek wrote: > >On Fri, Jul 01, 2016 at 11:01:53PM +0200, Jakub Hrozek wrote: > >> On Fri, Jul 01, 2016 at 03:24:39PM +0200, Jakub Hrozek wrote: > >> > On Wed, Jun 29, 2

[SSSD] Re: [PATCHES] krb5 config snippets

2016-07-06 Thread Jakub Hrozek
On Tue, Jul 05, 2016 at 12:54:34PM +0200, Sumit Bose wrote: > Hi, > > the following 4 patches include fixes for > ttps://fedorahosted.org/sssd/ticket/2788 and > https://fedorahosted.org/sssd/ticket/3041. They are not related but > depend on each other patch-wise, so I put them in a single mail. Pl

[SSSD] Re: [PATCH] test_sysdb_ts_cache: Do not use wrong pointer for output argument

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 01:04:49PM +0200, Sumit Bose wrote: > On Wed, Jul 06, 2016 at 12:23:08PM +0200, Sumit Bose wrote: > > On Mon, Jul 04, 2016 at 03:33:08PM +0200, Lukas Slebodnik wrote: > > > ehlo, > > > > > > Attached patches fix failures in test on big endian > > > http://s390.koji.fedorapr

[SSSD] Re: [PATCH] sudo: solve problems with fully qualified names

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 03:23:26PM +0200, Jakub Hrozek wrote: > On Wed, Jun 01, 2016 at 11:52:44AM +0200, Pavel Březina wrote: > > On 05/31/2016 01:44 PM, Jakub Hrozek wrote: > > > On Fri, May 27, 2016 at 11:54:20AM +0200, Pavel Březina wrote: > > > >

[SSSD] Re: [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 11:07:30AM +0200, Lukas Slebodnik wrote: > On (06/07/16 00:03), Jakub Hrozek wrote: > >On Tue, Jul 05, 2016 at 11:40:48PM +0200, Jakub Hrozek wrote: > >> On Tue, Jul 05, 2016 at 09:02:24PM +0200, Lukas Slebodnik wrote: > >> > On (05/07

[SSSD] Re: [PATCH] sudo: solve problems with fully qualified names

2016-07-06 Thread Jakub Hrozek
On Wed, Jun 01, 2016 at 11:52:44AM +0200, Pavel Březina wrote: > On 05/31/2016 01:44 PM, Jakub Hrozek wrote: > > On Fri, May 27, 2016 at 11:54:20AM +0200, Pavel Březina wrote: > > > See commit message for details. > > > > > > Two configurations

[SSSD] Re: [PATCH] PAM: Allow to configure pam services for smartcards

2016-07-06 Thread Jakub Hrozek
On Wed, Jul 06, 2016 at 11:04:56AM +0200, Lukas Slebodnik wrote: > Updated patch is attached. it is not :-) ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

[SSSD] Re: [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-07-05 Thread Jakub Hrozek
On Tue, Jul 05, 2016 at 11:40:48PM +0200, Jakub Hrozek wrote: > On Tue, Jul 05, 2016 at 09:02:24PM +0200, Lukas Slebodnik wrote: > > On (05/07/16 12:27), Jakub Hrozek wrote: > > >On Wed, Jun 29, 2016 at 05:38:55PM +0200, Lukas Slebodnik wrote: > > >> On (29/06

[SSSD] Re: [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-07-05 Thread Jakub Hrozek
On Tue, Jul 05, 2016 at 09:02:24PM +0200, Lukas Slebodnik wrote: > On (05/07/16 12:27), Jakub Hrozek wrote: > >On Wed, Jun 29, 2016 at 05:38:55PM +0200, Lukas Slebodnik wrote: > >> On (29/06/16 13:12), Jakub Hrozek wrote: > >> >On Wed, Jun 29, 2016 at 12:48:36P

[SSSD] Re: IPA: enable enterprise principals if server supports them

2016-07-05 Thread Jakub Hrozek
On Tue, Jul 05, 2016 at 12:37:22PM +0200, Sumit Bose wrote: > Hi, > > this patch set should solve https://fedorahosted.org/sssd/ticket/3018 > by looking up the additional UPN suffixes on the IPA server. If some > were found, enterprise principals are enabled if they are not explicitly > disabled i

[SSSD] [PATCH] LDAP: Lookup services by all protocols unless a protocol is specified

2016-07-05 Thread Jakub Hrozek
00:00:00 2001 From: Jakub Hrozek Date: Tue, 5 Jul 2016 12:23:23 +0200 Subject: [PATCH] LDAP: Lookup services by all protocols unless a protocol is specified The DP refactoring changed the way we handle strings from sbus. We no longer receive NULL strings, but empty strings instead. --- src/p

[SSSD] Re: [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-07-05 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 05:38:55PM +0200, Lukas Slebodnik wrote: > On (29/06/16 13:12), Jakub Hrozek wrote: > >On Wed, Jun 29, 2016 at 12:48:36PM +0200, Lukas Slebodnik wrote: > >> On (28/06/16 11:50), Jakub Hrozek wrote: > >> >From b493cee9976b8dd62bea3d8f09b88ce809a

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

2016-07-04 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 03:37:25PM +0200, Petr Cech wrote: > > > On 06/22/2016 06:25 AM, Petr Cech wrote: > > On 06/14/2016 05:21 PM, Petr Cech wrote: > > > Hello list, > > > > > > there is patch set attached resolving: > > > https://fedorahosted.org/sssd/ticket/2828 > > > > > > You can also se

[SSSD] Re: [PATCH] MAN: Remove leading spaces from elements programlisting

2016-07-04 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 08:59:15AM +0200, Lukas Slebodnik wrote: > ehlo, > > I noticed weird indentation of example for allow_missing_name. > It's longer that 80 columns which is not usual for man pages > and default terminal size (25x80). > > LS I agree this renders nicer, ACK _

[SSSD] Re: [PATCH] CONFIGURE: Inform about optional build dependencies

2016-07-04 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 08:45:53AM +0200, Lukas Slebodnik wrote: > ehlo, > > We usually inform about optional build dependencies at configure time > and which option can disable checking of this dependency. > > LS I agree with the intent, but I would just say we need such and such library, inste

[SSSD] Re: [PATCH] DOC: Fix few typos in doxygen comments

2016-07-04 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 10:51:13AM +0200, Lukas Slebodnik wrote: > ehlo, > > I noticed doxygen warnings when I was packaging 1.14beta into fedora. > > LS ACK, no more errors and make docs works fine. ___ sssd-devel mailing list sssd-devel@lists.fedorah

[SSSD] Re: [PATCH] sssctl: print a message when managing sssd

2016-07-04 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 01:24:35PM +0200, Pavel Březina wrote: > From 8b877579f3d1a9bbfa728a6e78ff829d936efbb1 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= > Date: Fri, 1 Jul 2016 13:23:57 +0200 > Subject: [PATCH] sssctl: print a message when managing sssd Shouldn't this be

[SSSD] Re: [PATCH] sssctl: start sssd in remove-cache command

2016-07-04 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 01:21:39PM +0200, Pavel Březina wrote: > From 87c7a20749bc48f97de0f18df77193ed36da6076 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= > Date: Fri, 1 Jul 2016 12:59:08 +0200 > Subject: [PATCH 1/2] sssctl: restart SSSD when removing cache > > Resolves: >

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-03 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 11:01:53PM +0200, Jakub Hrozek wrote: > On Fri, Jul 01, 2016 at 03:24:39PM +0200, Jakub Hrozek wrote: > > On Wed, Jun 29, 2016 at 07:00:13PM +0200, Jakub Hrozek wrote: > > > On Wed, Jun 29, 2016 at 04:36:23PM +0200, Sumit Bose wrote: > > > >

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-01 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 03:24:39PM +0200, Jakub Hrozek wrote: > On Wed, Jun 29, 2016 at 07:00:13PM +0200, Jakub Hrozek wrote: > > On Wed, Jun 29, 2016 at 04:36:23PM +0200, Sumit Bose wrote: > > > On Wed, Jun 29, 2016 at 12:05:42PM +0200, Lukas Slebodnik wrote: > > > &

[SSSD] Re: fully qualified sysdb names for users and groups

2016-07-01 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 07:00:13PM +0200, Jakub Hrozek wrote: > On Wed, Jun 29, 2016 at 04:36:23PM +0200, Sumit Bose wrote: > > On Wed, Jun 29, 2016 at 12:05:42PM +0200, Lukas Slebodnik wrote: > > > On (29/06/16 11:54), Pavel Březina wrote: > > > >On 06/28/201

[SSSD] Re: [PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.

2016-07-01 Thread Jakub Hrozek
On Fri, Jul 01, 2016 at 08:55:05AM +0200, Lukas Slebodnik wrote: > On (26/11/15 16:51), Jakub Hrozek wrote: > >On Thu, Nov 26, 2015 at 04:46:07PM +0100, Sumit Bose wrote: > >> On Thu, Nov 26, 2015 at 03:47:13PM +0100, Jakub Hrozek wrote: > >> > On Mon, Nov 23, 2

[SSSD] Announcing SSSD 1.14 Beta

2016-06-29 Thread Jakub Hrozek
ace the monitor ping with an in-process heartbeat #2957 Extend interface between DP and IFP #3070 Add infrastructure for socket-activated responders == Detailed Changelog == Christian Heimes (1): * Secrets: m4 macros for jansson and http-parser Jakub Hrozek (19): * Updating the

[SSSD] Re: [PATCH] BUILD: Fix detection of systemd

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 09:39:15PM +0200, Jakub Hrozek wrote: > > Previously it was part of libsystemd-daemon since systemd-32 :-) > > > > New version is attached > > > > LS > > ACK * master: * 7b308bfdac1f6cdc08502d69d49682a1f9aefe06 * 96a6

[SSSD] Re: [PATCH SET} A new Secrets service

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 10:39:17PM +0200, Jakub Hrozek wrote: > On Wed, Jun 29, 2016 at 06:49:05PM +0200, Jakub Hrozek wrote: > > On Wed, Jun 29, 2016 at 06:20:25PM +0200, Lukas Slebodnik wrote: > > > On (24/06/16 10:16), Jakub Hrozek wrote: > > > >On Thu, Jun 23,

[SSSD] [PATCH][PUSHED] MAN: Include idmap_sss.8.xml in the manpage sources

2016-06-29 Thread Jakub Hrozek
620c8ace8 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Wed, 29 Jun 2016 23:16:54 +0200 Subject: [PATCH] MAN: Include idmap_sss.8.xml in the manpage sources Reviewed-by: N/A, one-liner before release --- src/man/po/po4a.cfg | 1 + 1 file changed, 1 insertion(+) diff --git a/src/man/p

[SSSD] Re: [PATCH] BUILD: Fix detection of systemd

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 12:44:17PM +0200, Lukas Slebodnik wrote: > On (29/06/16 12:23), Jakub Hrozek wrote: > >On Wed, Jun 29, 2016 at 12:12:19PM +0200, Lukas Slebodnik wrote: > >> On (29/06/16 11:28), Jakub Hrozek wrote: > >> >On Tue, Jun 28, 2016 at 05:50:

[SSSD] Re: fully qualified sysdb names for users and groups

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 04:36:23PM +0200, Sumit Bose wrote: > On Wed, Jun 29, 2016 at 12:05:42PM +0200, Lukas Slebodnik wrote: > > On (29/06/16 11:54), Pavel Březina wrote: > > >On 06/28/2016 06:24 PM, Jakub Hrozek wrote: > > >> Hi, > > >> > > >

[SSSD] Re: [PATCH SET} A new Secrets service

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 06:20:25PM +0200, Lukas Slebodnik wrote: > On (24/06/16 10:16), Jakub Hrozek wrote: > >On Thu, Jun 23, 2016 at 04:23:12PM -0400, Simo Sorce wrote: > >> On Thu, 2016-06-23 at 21:36 +0200, Jakub Hrozek wrote: > >> > On Thu, Apr 21, 2016 at 09:21:

[SSSD] Re: [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 12:48:36PM +0200, Lukas Slebodnik wrote: > On (28/06/16 11:50), Jakub Hrozek wrote: > >From b493cee9976b8dd62bea3d8f09b88ce809a40980 Mon Sep 17 00:00:00 2001 > >From: Jakub Hrozek > >Date: Thu, 19 Nov 2015 10:40:39 +0100 > >Subject: [PATCH] LDAP:

[SSSD] Re: [PATCH] BUILD: Fix detection of systemd

2016-06-29 Thread Jakub Hrozek
On Wed, Jun 29, 2016 at 12:12:19PM +0200, Lukas Slebodnik wrote: > On (29/06/16 11:28), Jakub Hrozek wrote: > >On Tue, Jun 28, 2016 at 05:50:15PM +0200, Jakub Hrozek wrote: > >> On Tue, Jun 28, 2016 at 12:45:13PM +0200, Lukas Slebodnik wrote: > >> > ehlo, > &g

[SSSD] Re: [PATCH] BUILD: Fix detection of systemd

2016-06-29 Thread Jakub Hrozek
On Tue, Jun 28, 2016 at 05:50:15PM +0200, Jakub Hrozek wrote: > On Tue, Jun 28, 2016 at 12:45:13PM +0200, Lukas Slebodnik wrote: > > ehlo, > > > > @see commit message in attached patch. > > > > LS > > Thanks, the code reads much better and works on RHEL-6

[SSSD] Re: [PATCH] PAM: Allow to configure pam services for smartcards

2016-06-28 Thread Jakub Hrozek
On Mon, Feb 15, 2016 at 01:23:52PM +0100, Sumit Bose wrote: > On Wed, Jan 20, 2016 at 01:35:01PM +0100, Lukas Slebodnik wrote: > > ehlo, > > > > attached patch should fix ticket #2926 > > > > The default case (no adding, no removing services) is tested > > by pam-srv-tests. > > > > If the patch

[SSSD] Re: [PATCH] SSH: Do not print an error message if sss_ssh_authorizedkeys is asked for a local user

2016-06-28 Thread Jakub Hrozek
On Thu, Apr 28, 2016 at 10:40:58AM +0200, Jakub Hrozek wrote: > Hi, > > please see the attached patch. I didn't find any information in the ssh > man pages about the expected error codes from the AuthorizedKeysCommand, > but I think we should at least suppress the error mess

[SSSD] Re: [PATCH] IPA: Handle requests for netgroups from trusted domains gracefully

2016-06-28 Thread Jakub Hrozek
On Wed, May 11, 2016 at 01:59:42PM +0200, Jakub Hrozek wrote: > On Wed, May 11, 2016 at 01:57:53PM +0200, Pavel Březina wrote: > > On 05/06/2016 03:04 PM, Jakub Hrozek wrote: > > > To reproduce, just run: > > > getent netgroup some_name@trusted.domain > &g

[SSSD] fully qualified sysdb names for users and groups

2016-06-28 Thread Jakub Hrozek
Hi, here is my branch that implements using the fully qualified names in sysdb for users and groups: https://github.com/jhrozek/sssd/commits/sysdb-fqdn The SSSD code changes are done. There are some downstream tests that are failing (some sudo and some HBAC tests), but I think I can chase tho

[SSSD] Re: [PATCH] BUILD: Fix detection of systemd

2016-06-28 Thread Jakub Hrozek
On Tue, Jun 28, 2016 at 12:45:13PM +0200, Lukas Slebodnik wrote: > ehlo, > > @see commit message in attached patch. > > LS Thanks, the code reads much better and works on RHEL-6 and Fedora. I'm waiting for CI before pushing. But in the meantime, ACK. _

[SSSD] Re: [PATCH 2/2] UTIL: Revent connection handling in sssd_async_connect_send

2016-06-28 Thread Jakub Hrozek
On Tue, Jun 28, 2016 at 12:48:04PM +0200, Lukas Slebodnik wrote: > On (20/06/16 08:49), Simo Sorce wrote: > >On Mon, 2016-06-20 at 11:04 +0200, Lukas Slebodnik wrote: > >> On (19/06/16 15:27), Simo Sorce wrote: > >> >As the commit message says, nothing more. > >> >Isn't it right to wait for 6 secon

[SSSD] [PATCH] LDAP: Change the default rfc2307 autofs attribute mappings

2016-06-28 Thread Jakub Hrozek
, we should also file a ticket to remove the warnings in the next release to avoid warning forever. >From b493cee9976b8dd62bea3d8f09b88ce809a40980 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Thu, 19 Nov 2015 10:40:39 +0100 Subject: [PATCH] LDAP: Change the default rfc2307 autofs attrib

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-27 Thread Jakub Hrozek
On Fri, Jun 24, 2016 at 09:48:18AM +0200, Michal Židek wrote: > On 06/23/2016 10:38 PM, Michal Židek wrote: > > On 06/23/2016 10:22 PM, Jakub Hrozek wrote: > > > On Thu, Jun 23, 2016 at 10:19:49PM +0200, Michal Židek wrote: > > > > On 06/23/2016 10:17 PM, Jakub Hroze

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 07:02:19PM +0200, Jakub Hrozek wrote: > On Mon, Jun 27, 2016 at 05:58:49PM +0200, Michal Židek wrote: > > On 06/27/2016 04:18 PM, Jakub Hrozek wrote: > > > On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > > > > It depends on w

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-27 Thread Jakub Hrozek
On Fri, Jun 24, 2016 at 10:09:24AM +0200, Michal Židek wrote: > On 06/24/2016 09:56 AM, Jakub Hrozek wrote: > > On Thu, Jun 23, 2016 at 11:10:57AM +0200, Lukas Slebodnik wrote: > > > ehlo > > > > > > The first patch is sligtly modified version of Michal&#x

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 05:58:49PM +0200, Michal Židek wrote: > On 06/27/2016 04:18 PM, Jakub Hrozek wrote: > > On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > > > It depends on which patches will be pushed to master. > > > but I was able to apply

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:46:33AM +0200, Lukas Slebodnik wrote: > On (23/06/16 11:03), Lukas Slebodnik wrote: > >On (20/06/16 21:09), Jakub Hrozek wrote: > >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > >>> ehlo, > >>> >

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 10:57:02AM +0200, Jakub Hrozek wrote: > On Tue, Jun 21, 2016 at 10:25:41AM +0200, Lukas Slebodnik wrote: > > It looks like there is an agreement that we needn't change anything in > > libini_config for this use-case. So I will focus on other ding-libs p

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 06:05:50PM +0200, Jakub Hrozek wrote: > On Mon, Jun 27, 2016 at 01:26:09PM +0200, Pavel Březina wrote: > > Turned out I sent wrong patches. > > > > Thank you, Coverity came back clean, CI passed as well: > http://sssd-ci.duckdns.org/logs/job/46

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 01:26:09PM +0200, Pavel Březina wrote: > Turned out I sent wrong patches. > Thank you, Coverity came back clean, CI passed as well: http://sssd-ci.duckdns.org/logs/job/46/13/summary.html I did some more manual testing (because there are no tests so far..) and found:

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
On Mon, Jun 27, 2016 at 03:05:54PM +0200, Lukas Slebodnik wrote: > It depends on which patches will be pushed to master. > but I was able to apply patch from this thread with 3way merge > on Pavel's sssctl branch from fedorapeople; there weren't any conflicts The latest ones on the list. I will pu

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-27 Thread Jakub Hrozek
> > > On 06/24/2016 09:57 AM, Jakub Hrozek wrote: > > > > > > On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: > > > > > > > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: > > > > > > > > On Thu, Jun 23, 2016 at 09:10

[SSSD] Re: release notes for the 1.14 beta release

2016-06-27 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 10:21:34PM +0200, Jakub Hrozek wrote: > Hi, > > I prepared release notes for the beta release: > https://fedorahosted.org/sssd/wiki/Releases/Notes-1.14.0beta > I wanted to release the tarball today, but I would also like to push the > secrets respond

[SSSD] Re: [PATCH SET} A new Secrets service

2016-06-24 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 04:23:12PM -0400, Simo Sorce wrote: > On Thu, 2016-06-23 at 21:36 +0200, Jakub Hrozek wrote: > > On Thu, Apr 21, 2016 at 09:21:17AM +0200, Jakub Hrozek wrote: > > > Given that Lukas says "http_parser can provide pkgconfig in future", I > >

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-24 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 10:04:21PM +0200, Michal Židek wrote: > On 06/23/2016 09:40 PM, Jakub Hrozek wrote: > > On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: > > > On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > > > > On (23/06

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-24 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:10:57AM +0200, Lukas Slebodnik wrote: > ehlo > > The first patch is sligtly modified version of Michal's patch. > It depends on patch for config snippet. Because config > validation is optional if it isn't supported in libini_config. > And detection for new libini_config

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 10:19:49PM +0200, Michal Židek wrote: > On 06/23/2016 10:17 PM, Jakub Hrozek wrote: > > On Thu, Jun 23, 2016 at 10:12:21PM +0200, Michal Židek wrote: > > > On 06/23/2016 10:08 PM, Jakub Hrozek wrote: > > > > On Thu, Jun 23, 2016 at 11:10:57A

[SSSD] release notes for the 1.14 beta release

2016-06-23 Thread Jakub Hrozek
Hi, I prepared release notes for the beta release: https://fedorahosted.org/sssd/wiki/Releases/Notes-1.14.0beta I wanted to release the tarball today, but I would also like to push the secrets responder and wait for CI to finish..let's try tomorrow.

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 10:12:21PM +0200, Michal Židek wrote: > On 06/23/2016 10:08 PM, Jakub Hrozek wrote: > > On Thu, Jun 23, 2016 at 11:10:57AM +0200, Lukas Slebodnik wrote: > > > @@ -217,6 +216,14 @@ int confdb_init_db(const char *config_file, const >

[SSSD] Re: [PATCH] confdb: Check for config file errors on sssd startup

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:10:57AM +0200, Lukas Slebodnik wrote: > @@ -217,6 +216,14 @@ int confdb_init_db(const char *config_file, const char > *config_dir, > goto done; > } > > +/* FIXME: Do not hardcode the path */ > +ret = sss_ini_call_validators(init_data, > +

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 09:10:34PM +0200, Jakub Hrozek wrote: > On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > > On (23/06/16 18:59), Michal Židek wrote: > > >On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: > > >> On (23/06/16 18:24), Michal Židek

[SSSD] Re: [PATCH SET} A new Secrets service

2016-06-23 Thread Jakub Hrozek
On Thu, Apr 21, 2016 at 09:21:17AM +0200, Jakub Hrozek wrote: > Given that Lukas says "http_parser can provide pkgconfig in future", I > read his mail as a preference to keep the pkg-check test. And actually I > agree, it doesn't hurt, let's keep it in. I wanted to

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 07:29:13PM +0200, Lukas Slebodnik wrote: > On (23/06/16 18:59), Michal Židek wrote: > >On 06/23/2016 06:52 PM, Lukas Slebodnik wrote: > >> On (23/06/16 18:24), Michal Židek wrote: > >> > On 06/23/2016 05:50 PM, Michal Židek wrote: > >> > > Lukas, > >> > > > >> > > did you h

[SSSD] I renamed the 1.15 milestone to 1.16, the new 1.15 tracks Fedora 25 development

2016-06-23 Thread Jakub Hrozek
Hi, ssia, mostly. The previous 1.15 bucket was just a huge ball of everything we want to work on for the next release. But we're looking at some enhancements sooner, in time for Fedora 25, so I just shifted the version numbers and will add some tickets that we would like to include in Fedora 25 to

[SSSD] Re: [PATCH] Improve sysdb write performance

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 12:40:28PM +0200, Sumit Bose wrote: > On Thu, Jun 23, 2016 at 09:52:46AM +0200, Jakub Hrozek wrote: > > On Tue, Jun 21, 2016 at 10:28:52PM +0200, Lukas Slebodnik wrote: > > > On (21/06/16 22:08), Jakub Hrozek wrote: > > > >On Tue, Jun 21, 2

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 01:40:50PM +0200, Lukas Slebodnik wrote: > On (23/06/16 13:23), Jakub Hrozek wrote: > >On Wed, Jun 22, 2016 at 03:23:28PM +0200, Jakub Hrozek wrote: > >> On Tue, Jun 21, 2016 at 04:44:05PM +0200, Jakub Hrozek wrote: > >> > On Tue, Jun 21, 2

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-23 Thread Jakub Hrozek
On Thu, Jun 23, 2016 at 11:46:33AM +0200, Lukas Slebodnik wrote: > On (23/06/16 11:03), Lukas Slebodnik wrote: > >On (20/06/16 21:09), Jakub Hrozek wrote: > >>On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > >>> ehlo, > >>> >

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-22 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 04:44:05PM +0200, Jakub Hrozek wrote: > On Tue, Jun 21, 2016 at 04:15:06PM +0200, Pavel Březina wrote: > > On 06/21/2016 03:50 PM, Jakub Hrozek wrote: > > > On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > > > > Hi, > > &g

[SSSD] Announcing ding-libs 0.6.0

2016-06-22 Thread Jakub Hrozek
A new version of ding-libs was released today! ding-libs, or "Ding is not GLib" is a a set of helpful libraries used by projects such as SSSD or gss-proxy. The tarball can be downloaded from: https://fedorahosted.org/sssd/wiki/Releases == Highlights == * libini_config * libini now suppor

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 11:48:20AM +0200, Lukas Slebodnik wrote: > Thank you for review. > > Feel free to change/add comments before pushing patches. I just added a simple comment that the structure field is unused at the moment. All patches are pushed to ding-libs master: * a62db8ede4deba1

[SSSD] release notes for the ding-libs 0.6.0 release

2016-06-22 Thread Jakub Hrozek
Hi, I prepared very simple release notes for ding-libs 0.6.0: https://fedorahosted.org/sssd/wiki/Releases/DingNotes-0.6.0 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorah

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 10:52:34AM +0200, Lukas Slebodnik wrote: > On (22/06/16 10:39), Jakub Hrozek wrote: > >On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > >> ehlo, > >> > >> The first 4 patches are slightly modified version > >

[SSSD] Re: [ding-libs] [PATCH] ini: Add infrastructure for validators

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 09:28:26AM +0200, Lukas Slebodnik wrote: > ehlo, > > The first 4 patches are slightly modified version > of Michal's patches. Mostly coding style issues, fixed reports from static > analyzers + fixed small issues which would be fixed as part of review process. > > Last two

[SSSD] Re: [PATCHES] gpo and ding libs

2016-06-22 Thread Jakub Hrozek
On Wed, Jun 22, 2016 at 09:40:47AM +0200, Lukas Slebodnik wrote: > On (05/05/16 16:28), Michal Židek wrote: > >New patches are attached. > > > >SSSD patch changes: > >-I changed the behavior to only try > >parsing with the new flag when it > >previously failed as proposed by Sumit. > > > >-Existenc

[SSSD] Re: [PATCH] Improve sysdb write performance

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 11:46:51AM +0200, Jakub Hrozek wrote: > On Mon, Jun 20, 2016 at 06:57:55PM +0200, Jakub Hrozek wrote: > > On Fri, Jun 10, 2016 at 01:33:49PM +0200, Jakub Hrozek wrote: > > > On Fri, Jun 03, 2016 at 12:43:41PM +0200, Jakub Hrozek wrote: > > >

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 04:15:06PM +0200, Pavel Březina wrote: > On 06/21/2016 03:50 PM, Jakub Hrozek wrote: > > On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > > > Hi, > > > > > > Pavel's sssctl tool is at: > > > >

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 04:10:41PM +0200, Jakub Hrozek wrote: > On Tue, Jun 21, 2016 at 03:50:17PM +0200, Jakub Hrozek wrote: > > On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > > > Hi, > > > > > > Pavel's sssctl tool is at: > > >

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 03:50:17PM +0200, Jakub Hrozek wrote: > On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > > Hi, > > > > Pavel's sssctl tool is at: > > https://fedorapeople.org/cgit/pbrezina/public_git/sssd.git/log/?h=sssctl > >

[SSSD] Re: [PATCH] Pavel's sssctl work

2016-06-21 Thread Jakub Hrozek
On Mon, Jun 20, 2016 at 07:13:10PM +0200, Jakub Hrozek wrote: > Hi, > > Pavel's sssctl tool is at: > https://fedorapeople.org/cgit/pbrezina/public_git/sssd.git/log/?h=sssctl > > I'll try to have a first pass at review today in the evening, but I > thought

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-21 Thread Jakub Hrozek
On Tue, Jun 21, 2016 at 10:25:41AM +0200, Lukas Slebodnik wrote: > It looks like there is an agreement that we needn't change anything in > libini_config for this use-case. So I will focus on other ding-libs patches. > > BTW I need to update manual pages to reflect this change. > Please file a t

[SSSD] Announcing SSSD 1.14 Alpha

2016-06-20 Thread Jakub Hrozek
ssage for SSL errors. Jakub Hrozek (72): * Updating the version to track 1.14 development * MAN: Clarify pam_trusted_users option description * MAN: proxy and krb5 are valid access control modules * contrib: Add a pre-push hook to warn about commits without Reviewed-By * AD: Provide co

[SSSD] Re: [PATCH] confd: Make it possible to use config snippets

2016-06-20 Thread Jakub Hrozek
On Mon, Jun 20, 2016 at 08:54:18PM +0200, Lukas Slebodnik wrote: > ehlo, > > Attached is a sligtly modified version of Michal's patch. The same patch is attached twice. Was it by accident or did you mean to send two patches? > I fixed few coding style issues + added missing creation of directory

[SSSD] [PATCH] Pavel's sssctl work

2016-06-20 Thread Jakub Hrozek
Hi, Pavel's sssctl tool is at: https://fedorapeople.org/cgit/pbrezina/public_git/sssd.git/log/?h=sssctl I'll try to have a first pass at review today in the evening, but I thought I'll pass on the branch in case anyone else is interested.. ___ sssd-

[SSSD] Re: [PATCH] Improve sysdb write performance

2016-06-20 Thread Jakub Hrozek
On Fri, Jun 10, 2016 at 01:33:49PM +0200, Jakub Hrozek wrote: > On Fri, Jun 03, 2016 at 12:43:41PM +0200, Jakub Hrozek wrote: > > Hi, > > > > the attached patchset improves SSSD by performance by adding a second > > ldb cache file that is opened with LDB_FLG_NOSYNC and

[SSSD] Re: [PATCHES] Data provider refactoring

2016-06-20 Thread Jakub Hrozek
On Sat, Jun 18, 2016 at 04:11:30PM +0200, Pavel Březina wrote: > On 06/17/2016 08:33 PM, Lukas Slebodnik wrote: > > On (17/06/16 16:54), Lukas Slebodnik wrote: > > > On (14/06/16 15:30), Pavel Březina wrote: > > > > On 05/16/2016 02:00 PM, Pavel Březina wrote: > > > > > Hi, > > > > > the patches ar

[SSSD] RFC: 1.14 alpha release notes

2016-06-20 Thread Jakub Hrozek
Hi, I would like to release 1.14 alpha today, so I prepared the release notes: https://fedorahosted.org/sssd/wiki/Releases/Notes-1.14.0alpha I will also move the tickets that didn't make the Alpha release out of the Trac milestone. Please comment or edit the notes directly. For convenience,

[SSSD] Re: [PATCH] pam-srv-tests: Fix warning unused-function

2016-06-16 Thread Jakub Hrozek
On Thu, Jun 16, 2016 at 07:31:46PM +0200, Lukas Slebodnik wrote: > ehlo, > > attached small patch fix tiny complier warning when building with openssl > instead of nss. > > LS ack CI: http://sssd-ci.duckdns.org/logs/job/45/33/summary.html ___ sssd-dev

[SSSD] Re: [PATCH] SPEC: Run sssd as privileged user

2016-06-16 Thread Jakub Hrozek
On Thu, Jun 16, 2016 at 10:17:26PM +0200, Lukas Slebodnik wrote: > On (16/06/16 22:11), Jakub Hrozek wrote: > >On Thu, Jun 16, 2016 at 08:04:36PM +0200, Lukas Slebodnik wrote: > >> ehlo, > >> > >> attached patch makes my life easier. > >> I needn&#x

[SSSD] Re: [PATCH] SPEC: Run sssd as privileged user

2016-06-16 Thread Jakub Hrozek
On Thu, Jun 16, 2016 at 08:04:36PM +0200, Lukas Slebodnik wrote: > ehlo, > > attached patch makes my life easier. > I needn't patch spec file before building > sssd for downstream tests. SSSD still has some > issues with non-root mode and complicated providers. > > I would appreciate review. I t

<    1   2   3   4   5   6   7   8   9   10   >