Re: [SSSD] [PATCH] SDAP: rem warning - sizelimit exceeded in POSIX check

2015-10-11 Thread Jakub Hrozek
On Fri, Oct 09, 2015 at 08:02:08PM +0200, Michal Židek wrote: > On 10/09/2015 02:05 PM, Lukas Slebodnik wrote: > >On (09/10/15 13:56), Jakub Hrozek wrote: > >>On Tue, Oct 06, 2015 at 01:25:33PM +0200, Pavel Reichl wrote: > >>> > >>> > >>>On 10/06/2015 11:21 AM, Jakub Hrozek wrote: > > I

[SSSD] [PATCH] local overrides: don't concact server with overriden name/id

2015-10-11 Thread Pavel Březina
https://fedorahosted.org/sssd/ticket/2833 We can't search of overridden name/id in LDAP when using local overrides. These patches fix this for nss and sudo responder, please let me know if there is any other place. Similar fix needs to be done in cache_req interface, but I'll send it as

Re: [SSSD] [PATCH] local overrides: don't concact server with overriden name/id

2015-10-11 Thread Pavel Březina
On 10/11/2015 06:03 PM, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2833 We can't search of overridden name/id in LDAP when using local overrides. These patches fix this for nss and sudo responder, please let me know if there is any other place. Similar fix needs to be done in

Re: [SSSD] [PATCH] intg: Do not use non-existent pre-increment

2015-10-11 Thread Jakub Hrozek
On Fri, Oct 09, 2015 at 08:16:27PM +0200, Michal Židek wrote: > On 10/09/2015 05:10 PM, Nikolai Kondrashov wrote: > >Hi everyone, > > > >This patch fixes a hang of integration tests upon a failure to start slapd > >(which is a separate issue), introduced by my misunderstanding of Python. > > >

[SSSD] [PATCH] SDAP: add missing header files

2015-10-11 Thread Pavel Reichl
Hello, while messing around with some code analyses tool I noticed that some header files are not including directly header files which they use and thus are dependent on being included in particular order. Please see simple patch attached that fixes some of such files (but definitely not

[SSSD] [PATCHSET] remove unused params

2015-10-11 Thread Pavel Reichl
Hello, please see simple patch set attached. >From 85deb8e78f6fdbcc064d6184115483381c0174c6 Mon Sep 17 00:00:00 2001 From: Pavel Reichl Date: Sun, 11 Oct 2015 22:33:08 +0200 Subject: [PATCH 2/4] sudo: remove unused param. in ldap_get_sudo_options ---

Re: [SSSD] I would like to move these tickets out of 1.13

2015-10-11 Thread Jakub Hrozek
On Thu, Oct 08, 2015 at 11:22:38PM +0200, Jakub Hrozek wrote: > Hi, > > because we had quite a few tickets targetting 1.13 in this week's triage > bucket, I would like to move some out of the 1.13 buckets. > > Here are my proposals, feel free to go through 1.13.2/1.13.3 and propose > your own: >

[SSSD] [PATCH] Use refcount to keep track of server structures returned from failover

2015-10-11 Thread Jakub Hrozek
Hi, the attached patches are my proposal to fix https://fedorahosted.org/sssd/ticket/2829 I haven't tested them past make check yet, because I'm not sure I like them myself :) but at the same time I can't see a better way to keep track of the servers and let callers set state of servers. The

Re: [SSSD] [PATCH] Use refcount to keep track of server structures returned from failover

2015-10-11 Thread Lukas Slebodnik
On (11/10/15 22:00), Jakub Hrozek wrote: >Hi, > >the attached patches are my proposal to fix >https://fedorahosted.org/sssd/ticket/2829 > >I haven't tested them past make check yet, because I'm not sure I like >them myself :) but at the same time I can't see a better way to keep >track of the

Re: [SSSD] [PATCHSET] remove unused params

2015-10-11 Thread Lukas Slebodnik
On (11/10/15 23:17), Pavel Reichl wrote: >Hello, please see simple patch set attached. >From 85deb8e78f6fdbcc064d6184115483381c0174c6 Mon Sep 17 00:00:00 2001 >From: Pavel Reichl >Date: Sun, 11 Oct 2015 22:33:08 +0200 >Subject: [PATCH 2/4] sudo: remove unused param. in

Re: [SSSD] [PATCH] BUILD: Avoid symlinks with python modules

2015-10-11 Thread Lukas Slebodnik
On (05/10/15 10:56), Lukas Slebodnik wrote: >ehlo, > >I had a discussion with debian maintainer on IRC: >04:07 < tjaalton> hmm, is adding symlinks to python modules a fedora thing? >like pysss.so -> _py2sss.so >04:08 < tjaalton> looks like current debian python tools rename them, breaking >the

Re: [SSSD] [PATCH] SDAP: add missing header files

2015-10-11 Thread Lukas Slebodnik
On (11/10/15 23:11), Pavel Reichl wrote: >Hello, > >while messing around with some code analyses tool I noticed that some header >files are not including directly header files which they use and thus are >dependent on being included in particular order. > >Please see simple patch attached that

Re: [SSSD] [PATCH] intg: Do not use non-existent pre-increment

2015-10-11 Thread Lukas Slebodnik
On (11/10/15 21:27), Jakub Hrozek wrote: >On Fri, Oct 09, 2015 at 08:16:27PM +0200, Michal Židek wrote: >> On 10/09/2015 05:10 PM, Nikolai Kondrashov wrote: >> >Hi everyone, >> > >> >This patch fixes a hang of integration tests upon a failure to start slapd >> >(which is a separate issue),

Re: [SSSD] [PATCH] SDAP: rem warning - sizelimit exceeded in POSIX check

2015-10-11 Thread Lukas Slebodnik
On (09/10/15 20:02), Michal Židek wrote: >On 10/09/2015 02:05 PM, Lukas Slebodnik wrote: >>On (09/10/15 13:56), Jakub Hrozek wrote: >>>On Tue, Oct 06, 2015 at 01:25:33PM +0200, Pavel Reichl wrote: On 10/06/2015 11:21 AM, Jakub Hrozek wrote: > >I personally don't care about