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

[master] gracefully handle setting replica bind dn group on old masters #319

Closed
wants to merge 1 commit into from

Conversation

martbab
Copy link
Contributor

@martbab martbab commented Dec 7, 2016

This is the version of #315 for master
branch only

https://fedorahosted.org/freeipa/ticket/6532

Pre-3.3 masters do not support setting 'nsds5replicabinddngroup'
attribute on existing replica entry during setup of initial replication.
In this case UNWILLING_TO_PERFORM is returned. The code can interpret
this error as an indication of old master and fall back to just adding
its LDAP principal to entry's 'nsds5replicabinddn' attribute.

https://fedorahosted.org/freeipa/ticket/6532
@martbab
Copy link
Contributor Author

martbab commented Dec 9, 2016

PR #322 was closed, it was agreed that this fix is better.

@martbab
Copy link
Contributor Author

martbab commented Dec 9, 2016

@flo-renaud I was able to reproduce the issue you observed when trying to test the patch. I have opened an upstream ticket for it https://fedorahosted.org/freeipa/ticket/6549. I think it is a regression caused by installer refactoring and is unrelated to this PR.

@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Dec 12, 2016
@flo-renaud
Copy link
Contributor

Hi,
thanks for the patch. It works as expected.

@martbab martbab added the pushed Pull Request has already been pushed label Dec 12, 2016
@martbab
Copy link
Contributor Author

martbab commented Dec 12, 2016

@martbab martbab closed this Dec 12, 2016
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