Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport][ipa-4-6] Disable Schema Compat plugin during server upgrade #2002

Closed
wants to merge 1 commit into from

Conversation

rcritten
Copy link
Contributor

@rcritten rcritten commented Jun 7, 2018

This PR was opened automatically because PR #1971 was pushed to master and backport to ipa-4-6 is required.

If this is enabled it can cause a deadlock with SSSD trying
to look up entries and it trying to get data on AD users
from SSSD.

When reading the entry from LDIF try to get the camel-case
nsslapd-pluginEnabled and fall back to the all lower-case
nsslapd-pluginenabled if that is not found. It would be nice
if the fetch function was case sensitive but this is likely
overkill as it is, but better safe than blowing up.

Upon restoring it will always write the camel-case version.

https://pagure.io/freeipa/issue/6721

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
@rcritten rcritten added the ack Pull Request approved, can be merged label Jun 7, 2018
@rcritten
Copy link
Contributor Author

rcritten commented Jun 7, 2018

PR was ACKed automatically because this is backport of PR #1971. Wait for CI to finish before pushing. In case of questions or problems contact @rcritten who is author of the original PR.

@stlaz
Copy link
Contributor

stlaz commented Jun 8, 2018

ipa-4-6:

  • 36e95d2 Disable Schema Compat plugin during server upgrade

@stlaz stlaz added the pushed Pull Request has already been pushed label Jun 8, 2018
@stlaz stlaz closed this Jun 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants