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

Compat-plugin related fixes #711

Closed
wants to merge 4 commits into from
Closed

Compat-plugin related fixes #711

wants to merge 4 commits into from

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented Apr 13, 2017

The compat plugin was causing deadlocks with the topology plugin. Move
its setup at the end of the installation and remove the
cn=topology,cn=ipa,cn=etc subtree from its scope.

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

Due to LDAP connection refactoring, compat-manage would have behaved
differently for root and for other users even though it requires
the directory manager password. This is caused by it trying to do
external bind when it does not have the DIRMAN password which was
previously not supplied.

https://pagure.io/freeipa/issue/6821
@stlaz stlaz changed the title Move the compat plugin setup at the end of install Compat-plugin related fixes Apr 21, 2017
@stlaz
Copy link
Contributor Author

stlaz commented Apr 21, 2017

The latest patchset fixes also problems with ipa-compat-manage which would behave differently for root/other-users (check the commit message), and updates the link to slapi-nis project in the compat plugin update file to the correct one.

The compat plugin was causing deadlocks with the topology plugin. Move
its setup at the end of the installation and remove the
cn=topology,cn=ipa,cn=etc subtree from its scope.

https://pagure.io/freeipa/issue/6821
The entries in cn=topology,cn=ipa,cn=etc should not be taken in
account for the compat plugin.

https://pagure.io/freeipa/issue/6821
@MartinBasti MartinBasti self-assigned this Apr 21, 2017
@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Apr 24, 2017
@MartinBasti
Copy link
Contributor

master:

  • 0c0af8c compat-manage: behave the same for all users
  • ddbbb1c Move the compat plugin setup at the end of install
  • 6456159 compat: ignore cn=topology,cn=ipa,cn=etc subtree
  • 68c8ddf compat plugin: Update link to slapi-nis project

ipa-4-5:

  • 4fa7718 compat-manage: behave the same for all users
  • 7364c13 Move the compat plugin setup at the end of install
  • e691877 compat: ignore cn=topology,cn=ipa,cn=etc subtree
  • efe0960 compat plugin: Update link to slapi-nis project

@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Apr 24, 2017
@stlaz stlaz deleted the dsfixup branch September 11, 2017 10:46
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