Comment 2 for bug 1933116

Revision history for this message
Ɓukasz Zemczak (sil2100) wrote :

I don't know the sssd codebase so it's hard for me to assess if the huge patch that's being added as part of this SRU indeed only touches the GPO support or not. I will trust in your validation, but let's make sure that no other functionality of sssd is affected by this change. Would it be possible to do some exploratory testing at least, just to be sure? As said, maybe for those that know the code it's 'obvious' that it's not possible to introduce a regression, but seeing the diffstat there seem to be files updated that are not only related to gpo.

Anyway, let's do this, but proceed with caution. Also, remember that by pulling in such a huge set of changes, I think you need to coordinate with the server team if they are willing to maintain the patch going forward (or making sure the desktop team does).