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

Jump to bottom

masters DS<1.3.3 do not support bind group #322

Closed
wants to merge 1 commit into from
Closed

masters DS<1.3.3 do not support bind group #322

wants to merge 1 commit into from

Conversation

tbordaz
Copy link
Contributor

@tbordaz tbordaz commented Dec 8, 2016

Check the instance version before setting nsds5replicabbinddngroup and
nsds5replicabinddngroupcheckinterval

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

@tbordaz
masters DS<1.3.3 do not support bind group
f7f759a 
Check the instance version before setting nsds5replicabbinddngroup and
nsds5replicabinddngroupcheckinterval

https://fedorahosted.org/freeipa/ticket/6532
@MartinBasti MartinBasti mentioned this pull request Dec 8, 2016
Closed
@flo-renaud
Copy link
Contributor

Hi,
there is already an open PR for this issue:
#319 for master and #315 for ipa-4-4.
You may want to synchronize with @martbab and decide which solution to implement.

@tbordaz
Copy link
Contributor Author

tbordaz commented Dec 9, 2016

Oppss I missed that thanks for the heads up.
PR #319 and #315 are better fixes for this issue.
#322 should be discard

@martbab martbab added the rejected Pull Request has been rejected label Dec 9, 2016
@martbab
Copy link
Contributor

martbab commented Dec 9, 2016

OK closing and marking as rejected.

@martbab martbab closed this Dec 9, 2016
@martbab martbab mentioned this pull request Dec 9, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
rejected Pull Request has been rejected
Projects
None yet
Milestone
No milestone
3 participants
@tbordaz @flo-renaud @martbab