[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

On 09/13/2016 04:27 PM, Lukas Slebodnik wrote: On (13/09/16 16:24), Lukas Slebodnik wrote: On (13/09/16 14:11), Fabiano Fidêncio wrote: On Mon, Sep 12, 2016 at 9:40 AM, Petr Cech wrote: Bump. -- Petr^4 Čech ___ sssd-devel

[SSSD] Re: [PATCH] ini_augment: Use full path when reporting pattern mismatch

On 09/13/2016 04:53 PM, Lukas Slebodnik wrote: On (01/09/16 17:35), Michal Židek wrote: On 09/01/2016 05:26 PM, Dmitri Pal wrote: Hello, I do not like either of the versions of the patch. It is OK to use path_concat instead of snprintf. The whole point of not using it was to simplify the code

[SSSD] Re: [PATCH] ini_augment: Use full path when reporting pattern mismatch

On (01/09/16 17:35), Michal Židek wrote: >On 09/01/2016 05:26 PM, Dmitri Pal wrote: >> Hello, >> >> I do not like either of the versions of the patch. >> It is OK to use path_concat instead of snprintf. The whole point of not >> using it was to simplify the code and not have to check yet another

[SSSD] Re: [PATCH] SECRETS: Search by the right type when checking containers

On Tue, Sep 13, 2016 at 04:30:57PM +0200, Jakub Hrozek wrote: > On Tue, Aug 30, 2016 at 11:08:48AM +0200, Fabiano Fidêncio wrote: > > We've been searching for the wrong type ("simple") in > > local_db_check_containers(), which always gives us a NULL result. > > > > Let's introduce the new

[SSSD] Re: [PATCH] SECRETS: Search by the right type when checking containers

On Tue, Aug 30, 2016 at 11:08:48AM +0200, Fabiano Fidêncio wrote: > We've been searching for the wrong type ("simple") in > local_db_check_containers(), which always gives us a NULL result. > > Let's introduce the new LOCAL_CONTAINER_FILTER and do the search for the > right type ("container")

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

On (13/09/16 16:24), Lukas Slebodnik wrote: >On (13/09/16 14:11), Fabiano Fidêncio wrote: >>On Mon, Sep 12, 2016 at 9:40 AM, Petr Cech wrote: >>> Bump. >>> >>> >>> -- >>> Petr^4 Čech >>> ___ >>> sssd-devel mailing list >>>

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

On (13/09/16 14:11), Fabiano Fidêncio wrote: >On Mon, Sep 12, 2016 at 9:40 AM, Petr Cech wrote: >> Bump. >> >> >> -- >> Petr^4 Čech >> ___ >> sssd-devel mailing list >> sssd-devel@lists.fedorahosted.org >>

[SSSD] Re: [PATCH] SDAP: Fix settig paging attribute in sdap_get_generic_ext_send

On (09/09/16 13:09), Sumit Bose wrote: >On Fri, Sep 09, 2016 at 12:48:47PM +0200, Lukas Slebodnik wrote: >> On (31/08/16 07:45), Fabiano Fidêncio wrote: >> >On Tue, Aug 30, 2016 at 5:26 PM, Lukas Slebodnik >> >wrote: >> >> ehlo, >> >> >> >> We should set pagging flag in

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

On 09/12/2016 10:01 AM, Lukas Slebodnik wrote: On (11/09/16 23:49), Jakub Hrozek wrote: On Thu, Sep 08, 2016 at 12:56:08PM +0200, Lukas Slebodnik wrote: Let me explain why wrappers are not good idea in production. There was introduced new wrapper(#1991) for ldb_search SSS_LDB_SEARCH. It should

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (closed)

jhrozek's pull request #15: "Avoid returning System Error on clock skew" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/15/head:pr15 git checkout pr15

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (+Pushed)

jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (-Accepted)

jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Accepted* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

lslebodn commented on a pull request """ On (13/09/16 06:07), sumit-bose wrote: >I think it is still an ACK. > master: * d3348f49260998880bb7cd3b2fb72d562b1b7a64 LS """ See the full comment at https://github.com/SSSD/sssd/pull/15#issuecomment-246680797

[SSSD] [sssd PR#23] sss_groupshow does not work with mpg (closed)

mzidek-rh's pull request #23: "sss_groupshow does not work with mpg" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/23 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/23/head:pr23 git checkout pr23

[SSSD] [sssd PR#23] sss_groupshow does not work with mpg (+Pushed)

mzidek-rh's pull request #23: "sss_groupshow does not work with mpg" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/23 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

sumit-bose commented on a pull request """ I think it is still an ACK. """ See the full comment at https://github.com/SSSD/sssd/pull/15#issuecomment-246674999 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (+Accepted)

jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (-Changes requested)

jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Changes requested* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

lslebodn commented on a pull request """ On (07/09/16 10:13), Jakub Hrozek wrote: >On Tue, Sep 06, 2016 at 06:09:58AM -0700, Jakub Hrozek wrote: >> good idea > >ah, only when I started to implement this I realized it's already done :) > >See: > >

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

On Mon, Sep 12, 2016 at 9:40 AM, Petr Cech wrote: > Bump. > > > -- > Petr^4 Čech > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org Patch

[SSSD] [sssd PR#23] sss_groupshow does not work with mpg (comment)

lslebodn commented on a pull request """ master: * bb14556c1df503314644fc424fbbf95759791db9 * 812bed08943df8bf3fd1ff9eabcaf5bedc635c92 """ See the full comment at https://github.com/SSSD/sssd/pull/23#issuecomment-246659465 ___ sssd-devel mailing list

[SSSD] [sssd PR#20] sss_override fails to export (comment)

lslebodn commented on a pull request """ On (09/09/16 05:50), mzidek-rh wrote: >See the updated tests. It uses export and import and checks if override works >after import. > ACK for test; just waiting for CI LS """ See the full comment at

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (+Accepted)

jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (-Changes requested)

jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Changes requested* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (synchronize)

fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" was synchronize See the full pull-request at https://github.com/SSSD/sssd/pull/11 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/11/head:pr11 git

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

lslebodn commented on a pull request """ On (13/09/16 01:33), mzidek-rh wrote: >Makes sense, but I would prefer to add new "else if" branch with >ADMINLIMIT_EXCEEDED specific debug message. > >It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT >into something more generic, like

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (+Changes requested)

jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Changes requested* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

mzidek-rh commented on a pull request """ Makes sense, but I would prefer to add new "else if" branch with ADMINLIMIT_EXCEEDED specific debug message. It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT into something more generic, like SDAP_SRCH_FLG_SILENT because we want to

[SSSD] Re: [PATCH] Unit tests for pam_sss using pam_wrapper (need help with CI..)

On (09/08/16 10:18), Jakub Hrozek wrote: >On Tue, Aug 09, 2016 at 08:04:38AM +0200, Lukas Slebodnik wrote: >> On (09/05/16 10:07), Jakub Hrozek wrote: >> >On Wed, May 04, 2016 at 11:36:57PM +0200, Lukas Slebodnik wrote: >> >> On (27/04/16 10:51), Jakub Hrozek wrote: >> >> >Hi, >> >> > >> >> >the