[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain lslebodn commented: """ master: * 45e322191c7aa9390798b64ccb158ee800489945 """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-326624250 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain fidencio commented: """ Added "Accepted" label per @pbrezina's review, """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-326550705 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-326546558 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain jhrozek commented: """ Sure, just type in a wrong password when authenticating as a user from an IPA-AD trust. With the patch, you will see the migration being tried as well, without the patch, the PAM request should just shortcut. """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-318285876 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain jhrozek commented: """ On Wed, Jun 14, 2017 at 07:07:29AM -0700, fidencio wrote: > @jhrozek: While the code does look simple enough I'd like to ask you whether > you know in which commit the regression was introduced (that would be good to > have mentioned in the commit message). Done. > > You have an ACK from me but I didn't run any tests with this patch (IOW, > would be really nice to get at least a second opinion on this one). Thank you for checking the code nonetheless. """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-308490666 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
[SSSD] [sssd PR#307][comment] IPA: Only attempt migration for the joined domain
URL: https://github.com/SSSD/sssd/pull/307 Title: #307: IPA: Only attempt migration for the joined domain fidencio commented: """ @jhrozek: While the code does look simple enough I'd like to ask you whether you know in which commit the regression was introduced (that would be good to have mentioned in the commit message). You have an ACK from me but I didn't run any tests with this patch (IOW, would be really nice to get at least a second opinion on this one). """ See the full comment at https://github.com/SSSD/sssd/pull/307#issuecomment-308442392 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org