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.

Sign up for GitHub

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

Fixes for #9, #22 and #24 #27

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

Fixes for #9, #22 and #24 #27

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5227bec
Fix for issue #24 and #22
grisu48 Nov 7, 2018
440c7d3
Fixed preview of examples
grisu48 Nov 7, 2018
25dae02
Fixed readme
grisu48 Nov 7, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ them are as follows:
Examples
--------

1) Configure an OpenLDAP server without SSL:
Configure an OpenLDAP server without SSL:

- hosts: all
sudo: true
Expand All @@ -38,7 +38,8 @@ Examples
openldap_server_rootpw: passme
openldap_server_enable_ssl: false

2) Configure an OpenLDAP server with SSL:

Configure an OpenLDAP server with SSL:

- hosts: all
sudo: true
Expand Down Expand Up @@ -66,5 +67,3 @@ Author Information
------------------

Benno Joy


1 change: 1 addition & 0 deletions meta/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ galaxy_info:
versions:
- 5
- 6
- 7
- name: Fedora
versions:
- 16
Expand Down
2 changes: 1 addition & 1 deletion tasks/configure_ldap.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,4 +44,4 @@
register: result

- name: add the base domain
shell: ldapadd -x -D "cn=Manager,dc={{ openldap_server_domain_name.split('.')[0] }},dc={{ openldap_server_domain_name.split('.')[1] }}" -w {{ openldap_server_rootpw }} -f {{ result.dest|default(result.path) }} && touch {{ openldap_server_app_path }}/rootdn_created creates={{ openldap_server_app_path }}/rootdn_created
shell: ldapadd -x -h localhost -D "cn=Manager,dc={{ openldap_server_domain_name.split('.')[0] }},dc={{ openldap_server_domain_name.split('.')[1] }}" -w {{ openldap_server_rootpw }} -f {{ result.dest|default(result.path) }} && touch {{ openldap_server_app_path }}/rootdn_created creates={{ openldap_server_app_path }}/rootdn_created
4 changes: 2 additions & 2 deletions tasks/install_ldap.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,13 @@

- name: Install the openldap and required Packages for RedHat
yum: name={{ item }} state=installed
with_items: openldap_server_pkgs
with_items: "{{ openldap_server_pkgs }}"
when: ansible_os_family == 'RedHat'


- name: Install the openldap and required Packages for Ubuntu
apt: name={{ item }} state=installed update_cache=yes
with_items: openldap_server_pkgs
with_items: "{{ openldap_server_pkgs }}"
environment: env
when: ansible_os_family == 'Debian'

Expand Down