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 mailing list
sssd-devel
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
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
>
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 LOCAL_CO
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") from
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-devel@lists.fedorahosted.o
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
>> https://lists.fedorahosted.org/admin/lists/sssd-
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 state and not in local
>>
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
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
_
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
https://lists.fedoraho
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
https://lists.fedo
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
_
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
__
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
https://lists.fedorahosted
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
https://lists.fedorahosted.org/
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
https://lists.fedora
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
https://l
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:
>
> https://github.com/SSSD/sssd/blob/master/src/pr
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 looks good and all
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
lslebodn commented on a pull request
"""
On (12/09/16 10:58), mzidek-rh wrote:
>We recently fixed this with normal groups. The MPG code path was still using
>shortname in sysdb search operation.
>
>Upstream ticket:
>https://fedorahosted.org/sssd/ticket/3184
ACK
http://sssd-ci.duckdns.org/logs/jo
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.
>
master:
* 1c72723cde8bea0d390b928c7cd29e48e7a7deab
* 07e7683f5a86991feaa764e2055116554ada1b93
LS
"""
See the full comme
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
http://sssd-ci.duckdns.org/logs/job/53/24/summary.html
LS
"""
See the full comment at
https://github.com/SSSD/sss
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
https://github.com/SSSD/sssd/pull/20#issuecomment-246
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
https://l
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-devel@lists.fedorahosted.or
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 check
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
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
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
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 at
32 matches
Mail list logo