[SSSD] Re: [PATCH] DEBUG: Add `debug` alias for debug_level

2016-04-27 Thread Stephen Gallagher
On 04/27/2016 06:44 AM, Petr Cech wrote: > On 04/27/2016 08:47 AM, Petr Cech wrote: >> On 04/26/2016 05:08 PM, Stephen Gallagher wrote: >>> Our users constantly make the mistake of typing `debug = 9` in the >>> sssd.conf >>> instead of `debug_level = 9` as would be correct. This happens >>> frequen

[SSSD] Re: [PATCH] DEBUG: Add `debug` alias for debug_level

2016-04-27 Thread Stephen Gallagher
On 04/27/2016 05:57 AM, Pavel Březina wrote: > On 04/26/2016 05:08 PM, Stephen Gallagher wrote: >> Our users constantly make the mistake of typing `debug = 9` in the sssd.conf >> instead of `debug_level = 9` as would be correct. This happens >> frequently-enough >> that we should just alias it rat

[SSSD] Re: [PATCH] DEBUG: Add `debug` alias for debug_level

2016-04-27 Thread Petr Cech
On 04/27/2016 08:47 AM, Petr Cech wrote: On 04/26/2016 05:08 PM, Stephen Gallagher wrote: Our users constantly make the mistake of typing `debug = 9` in the sssd.conf instead of `debug_level = 9` as would be correct. This happens frequently-enough that we should just alias it rather than continu

[SSSD] [PATCH] MAN: Remove references to the obsolete PubkeyAgent ssh option

2016-04-27 Thread Jakub Hrozek
Hi, please see the attached trivial patch. The issue was reported by adelton on IRC. >From 979353eb20849f036522ce4f5edf28f5a989f886 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Wed, 27 Apr 2016 12:18:15 +0200 Subject: [PATCH] MAN: Remove references to the obsolete PubkeyAgent ssh option The

[SSSD] Re: [PATCH] DEBUG: Add `debug` alias for debug_level

2016-04-27 Thread Pavel Březina
On 04/26/2016 05:08 PM, Stephen Gallagher wrote: Our users constantly make the mistake of typing `debug = 9` in the sssd.conf instead of `debug_level = 9` as would be correct. This happens frequently-enough that we should just alias it rather than continue to have people make mistakes. Resolves:

[SSSD] Re: [PATCH] Do not crash if GetUserAttrs cannot be parsed

2016-04-27 Thread Pavel Březina
On 04/27/2016 11:13 AM, Jakub Hrozek wrote: On Wed, Apr 20, 2016 at 01:20:14PM +0200, Pavel Březina wrote: On 04/20/2016 11:56 AM, Jakub Hrozek wrote: Hi Pavel, can you check if this is the right thing to do for methods that parse arguments on their own? To reproduce, it was enough to run:

[SSSD] Re: [PATCH] Do not crash if GetUserAttrs cannot be parsed

2016-04-27 Thread Jakub Hrozek
On Wed, Apr 20, 2016 at 01:20:14PM +0200, Pavel Březina wrote: > On 04/20/2016 11:56 AM, Jakub Hrozek wrote: > > Hi Pavel, > > > > can you check if this is the right thing to do for methods that parse > > arguments on their own? > > > > To reproduce, it was enough to run: > > sudo dbus-send

[SSSD] Re: [PATCH] MAN: Drop the reference to IPAv2 in the man page

2016-04-27 Thread Jakub Hrozek
On Mon, Mar 21, 2016 at 09:37:45AM +0100, Lukas Slebodnik wrote: > On (17/03/16 22:59), Jakub Hrozek wrote: > >a man page one liner :) > > > >No need to explicitly mention IPAv2, I hope there are no deployments of > >IPAv1 left anymore :) > > > >This change was suggested by Rob in > >https://fedora

[SSSD] Re: [PATCH] LDAP: Print port in sdap_print_server

2016-04-27 Thread Jakub Hrozek
On Fri, Apr 22, 2016 at 05:04:21PM +0200, Lukas Slebodnik wrote: > On (22/04/16 15:57), Jakub Hrozek wrote: > >On Mon, Apr 11, 2016 at 03:35:52PM +0200, Jakub Hrozek wrote: > >> On Mon, Apr 11, 2016 at 02:58:06PM +0200, Lukas Slebodnik wrote: > >> > On (11/04/16 13:39), Jakub Hrozek wrote: > >> > >

[SSSD] Re: Running tests with different environment

2016-04-27 Thread Jakub Hrozek
On Thu, Jan 28, 2016 at 05:12:40PM +0100, Jakub Hrozek wrote: > On Thu, Jan 28, 2016 at 04:19:25PM +0100, Lukas Slebodnik wrote: > > On (27/10/15 22:35), Lukas Slebodnik wrote: > > >On (27/10/15 17:57), Jakub Hrozek wrote: > > >>On Tue, Oct 27, 2015 at 05:42:29PM +0100, Jakub Hrozek wrote: > > >>>

[SSSD] Re: [PATCHES] PAM: refactor pam_reply

2016-04-27 Thread Jakub Hrozek
On Mon, May 11, 2015 at 06:28:35PM +0200, Lukas Slebodnik wrote: > On (11/05/15 17:36), Pavel Reichl wrote: > >Rebased patch set is attached. > > Code coverage of function pam_reply was quite high > but it covered just part which was necessary for Sumit's work. > > I thought you would increase co