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

Support for ansible 2.3.2.0 #25

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JMLX42
Copy link

@JMLX42 JMLX42 commented Aug 22, 2017

No description provided.

The password is properly set in the conf. But the conf was not properly taken into account because the slapd was not restarted.
The service was not restarted because it was already started upon install even if it was not supposed to.
To avoid starting the service upon install, the RUNLEVEL env variable must be set to 1. That's what the `vars/main.yml` is supposed to do. But it was not used properly in tasks/install_ldap.yml and was ignored.
@JMLX42 JMLX42 changed the title Fix/ansible 2 3 2 0 Support for ansible 2.3.2.0 Aug 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant