[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-04-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute jhrozek commented: """ * master: 0f6b5b02afb35caae774ff4d52854a844d49f52e """ See the full comment at https://github.com/SSSD/sssd/pull/544#issuecomment-378590221 __

[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-04-04 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute fidencio commented: """ CI got stuck due to rawhide failures, but those have been already reported :-/ There was also a failure on find_uid tests, which fails every now and then. The job was 875

[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-04-03 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute pbrezina commented: """ LGTM """ See the full comment at https://github.com/SSSD/sssd/pull/544#issuecomment-378205992 ___ sssd-devel mailing list --

[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute jhrozek commented: """ I pushed a slightly simplified version. """ See the full comment at https://github.com/SSSD/sssd/pull/544#issuecomment-377339384 _

[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-03-29 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute jhrozek commented: """ > On 29 Mar 2018, at 10:24, fidencio wrote: > > @jhrozek, the patch itself looks good. > Do we have downstream tests covering this situation? I don’t think so > Would

[SSSD] [sssd PR#544][comment] IPA: Qualify the externalUser sudo attribute

2018-03-29 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/544 Title: #544: IPA: Qualify the externalUser sudo attribute fidencio commented: """ @jhrozek, the patch itself looks good. Do we have downstream tests covering this situation? Wouldn't be worth it to open a RHEL bug for this issue? """ See the full comm