[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ * master: * 861ab44e8148208425b67c4711bc8fade10fd3ed * 3e39806177e1cd383743ff596cb96df44a6ce8c9 * ed0cdfcacc44e4e13e1524e254efa744610a87c2 *

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ the new PR just amends the manpage description of the non-POSIX domains """ See the full comment at https://github.com/SSSD/sssd/pull/215#issuecomment-290368513

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ On Thu, Mar 30, 2017 at 02:53:18AM -0700, sumit-bose wrote: > I tested the patches with a plain LDAP setup and with and AD. In general they > work as expected and since I think

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups sumit-bose commented: """ I tested the patches with a plain LDAP setup and with and AD. In general they work as expected and since I think the current code is ok I would ACK the patches so that the

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups pbrezina commented: """ On 03/30/2017 11:23 AM, Jakub Hrozek wrote: > On Thu, Mar 30, 2017 at 01:59:20AM -0700, Pavel Březina wrote: >> I got error in enumeration as well with my secrets patch

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups fidencio commented: """ On Thu, Mar 30, 2017 at 11:23 AM, Jakub Hrozek wrote: > On Thu, Mar 30, 2017 at 01:59:20AM -0700, Pavel Březina wrote: > > I got error in

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ On Thu, Mar 30, 2017 at 01:59:20AM -0700, Pavel Březina wrote: > I got error in enumeration as well with my secrets patch (definitely not > related), but on debian and in

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-30 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups pbrezina commented: """ I got error in enumeration as well with my secrets patch (definitely not related), but on debian and in different test:

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ CI is happier now: http://sssd-ci.duckdns.org/logs/job/66/49/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/215#issuecomment-290205579

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/66/47/summary.html there is a RHEL6 failure in the enumeration code. Because the test only failed on RHEL-6, I don't think it's related

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/215#issuecomment-290113327 ___ sssd-devel mailing

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ I fixed the minor issues in comments and the man pages. I also fixed the issue in the Kerberos provider with the following hunk: ``` diff --git a/src/providers/krb5/krb5_auth.c

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-29 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups pbrezina commented: """ ```xml POSIX domains are reachable by all services. Application domains are only reachable

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-28 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups fidencio commented: """ On Tue, Mar 28, 2017 at 11:14 PM, Jakub Hrozek wrote: > *@jhrozek* commented on this pull request. > -- > > In

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups sumit-bose commented: """ Please find my comments in-line. @pbrezina, I wonder if you can have a look at the CACHE_REQ patch to see if it is ok from the design point of view. """ See the full

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ the second iteration of patches should fix a coverity warning """ See the full comment at https://github.com/SSSD/sssd/pull/215#issuecomment-289871457

[SSSD] [sssd PR#215][comment] Support for non-POSIX users and groups

2017-03-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/215 Title: #215: Support for non-POSIX users and groups jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/65/97/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/215#issuecomment-289805144