[SSSD] [sssd PR#247][comment] Subdomain inherit

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/247 Title: #247: Subdomain inherit lslebodn commented: """ Bump. https://pagure.io/SSSD/sssd/issue/3337 is planed for 1.15.4. So it would be good to work on this sooner then later. """ See the full comment at

[SSSD] [sssd PR#335][comment] Revert "LDAP: Fix nesting level comparison"

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/335 Title: #335: Revert "LDAP: Fix nesting level comparison" lslebodn commented: """ On (24/07/17 05:38), fidencio wrote: >I agree with your approach, @lslebodn (and even suggested it during this >morning on IRC). > >But I also would like to know exactly

[SSSD] [sssd PR#4][+Rejected] Added small tweaks to enable SSSD to be compiled with the musl libc

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/4 Title: #4: Added small tweaks to enable SSSD to be compiled with the musl libc Label: +Rejected ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#4][-Changes requested] Added small tweaks to enable SSSD to be compiled with the musl libc

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/4 Title: #4: Added small tweaks to enable SSSD to be compiled with the musl libc Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#346][comment] intg: Disable add_remove tests

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/346 Title: #346: intg: Disable add_remove tests lslebodn commented: """ >I would prefer to have a comment in front of each disabled test saying it is >disabled and briefly explaining why (including a reference to the issue >perhaps). Reference to issue

[SSSD] [sssd PR#4][comment] Added small tweaks to enable SSSD to be compiled with the musl libc

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/4 Title: #4: Added small tweaks to enable SSSD to be compiled with the musl libc lslebodn commented: """ Patches does not apply to git master and it does not look like @lejonet would like to continue in this PR

[SSSD] [sssd PR#4][closed] Added small tweaks to enable SSSD to be compiled with the musl libc

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/4 Author: lejonet Title: #4: Added small tweaks to enable SSSD to be compiled with the musl libc Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/4/head:pr4 git checkout pr4

[SSSD] [sssd PR#128][comment] Fix group renaming issue when "id_provider = ldap" is set

2017-08-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/128 Title: #128: Fix group renaming issue when "id_provider = ldap" is set fidencio commented: """ CI passed: http://vm-058-233.${abc}/logs/job/73/09/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/128#issuecomment-321674948

[SSSD] [sssd PR#128][synchronized] Fix group renaming issue when "id_provider = ldap" is set

2017-08-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/128 Author: fidencio Title: #128: Fix group renaming issue when "id_provider = ldap" is set Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/128/head:pr128 git checkout pr128 From

[SSSD] [sssd PR#128][comment] Fix group renaming issue when "id_provider = ldap" is set

2017-08-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/128 Title: #128: Fix group renaming issue when "id_provider = ldap" is set jhrozek commented: """ So I was thinking about this PR a bit more and I'm no longer sure we can rename groups at will. I think we can only support that if we support multiple

[SSSD] [sssd PR#346][+Changes requested] intg: Disable add_remove tests

2017-08-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/346 Title: #346: intg: Disable add_remove tests Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#346][comment] intg: Disable add_remove tests

2017-08-10 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/346 Title: #346: intg: Disable add_remove tests spbnick commented: """ This is a good move, and looks fine to me, except I would prefer to have a comment in front of each disabled test saying it is disabled and briefly explaining why (including a

[SSSD] [sssd PR#346][opened] intg: Disable add_remove tests

2017-08-10 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/346 Author: lslebodn Title: #346: intg: Disable add_remove tests Action: opened PR body: """ add_remove fails quite often in enumeration test. The reason of failures is not obvious and will take some time to investigate it. Failures blocks testing of

[SSSD] [sssd PR#334][comment] Print a warning when enumeration is requested but disabled

2017-08-10 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/334 Title: #334: Print a warning when enumeration is requested but disabled amitkumar50 commented: """ @lslebodn I believe you want to log message on return path when Backend says Nss responder to write data in Memcache. Something this is there in

[SSSD] [sssd PR#326][-Changes requested] IPA: check if IPA hostname is a FQDN

2017-08-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/326 Title: #326: IPA: check if IPA hostname is a FQDN Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#342][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-08-10 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/342 Title: #342: SELINUX: Use getseuserbyname to get IPA seuser mzidek-rh commented: """ The code LGTM and my ad-hoc testing was so far successful. There is one more test I want to try manually. We do not have upstream tests that would cover this part of

[SSSD] [sssd PR#326][synchronized] IPA: check if IPA hostname is a FQDN

2017-08-10 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/326 Author: amitkumar50 Title: #326: IPA: check if IPA hostname is a FQDN Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/326/head:pr326 git checkout pr326 From

[SSSD] [sssd PR#326][synchronized] IPA: check if IPA hostname is a FQDN

2017-08-10 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/326 Author: amitkumar50 Title: #326: IPA: check if IPA hostname is a FQDN Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/326/head:pr326 git checkout pr326 From

[SSSD] Re: Shall we freeze the development till #3463 is solved?

2017-08-10 Thread Fabiano FidĂȘncio
On Thu, Aug 10, 2017 at 10:15 AM, Lukas Slebodnik wrote: > On (09/08/17 14:43), Jakub Hrozek wrote: >> >>> On 8 Aug 2017, at 16:51, Fabiano FidĂȘncio wrote: >>> >>> People, >>> >>> There's a test, part of our internal CI, recurrently failing in the >>>

[SSSD] Re: Shall we freeze the development till #3463 is solved?

2017-08-10 Thread Lukas Slebodnik
On (09/08/17 14:43), Jakub Hrozek wrote: > >> On 8 Aug 2017, at 16:51, Fabiano FidĂȘncio wrote: >> >> People, >> >> There's a test, part of our internal CI, recurrently failing in the >> past few weeks: >> >> === FAILURES >>