[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-11-02 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider jhrozek commented: """ On Wed, Nov 02, 2016 at 06:18:57AM -0700, Justin Stephenson wrote: > Thanks for your comments Jakub, please see below: > > 1) Yes, I will go ahead and submit a

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-11-02 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider justin-stephenson commented: """ Thanks for your comments Jakub, please see below: 1) Yes, I will go ahead and submit a different patch in this same PR if it's okay. 2) I agree about

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-11-02 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider jhrozek commented: """ Thank you for the patch. I checked it by inspecting the values from ad_opts.c and krb5_opts.c and ldap_opts.c with vimdiff. I only have two questions: 1) Should we do

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-10-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider sumit-bose commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/68#issuecomment-256867038 ___ sssd-devel

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-10-27 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/68#issuecomment-256778568

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-10-27 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/68#issuecomment-256778566