Skip to content

Commit

Permalink
Ensure that LDAP config is deployed on all keystone hosts
Browse files Browse the repository at this point in the history
In I66ed21cdcf42d0c2012062c8cf74305fecbec312 the condition meant
for the setup of the domain was mistakenly applied to all tasks
including laying down the template.

This patch moves the conditional which ensures the domain is setup
on the first host to the task in question to ensure that everything
is good and well with the world again.

Change-Id: Icb7c2556306d459534e6791f16c7013d0e9fcaf5
Closes-Bug: #1804827
(cherry picked from commit 3a6a55b)
  • Loading branch information
Jesse Pretorius committed Dec 11, 2018
1 parent 5a54cc6 commit fc3d2fe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 2 additions & 0 deletions tasks/keystone_ldap_setup.yml
Expand Up @@ -26,6 +26,8 @@
delay: 10
with_dict: "{{ keystone_ldap }}"
delegate_to: "{{ keystone_service_setup_host }}"
when:
- "inventory_hostname == ((groups['keystone_all'] | intersect(ansible_play_hosts)) | list)[-1]"
vars:
ansible_python_interpreter: >-
{{ (keystone_service_setup_host == 'localhost') | ternary(ansible_playbook_python, ansible_python['executable']) }}
Expand Down
1 change: 0 additions & 1 deletion tasks/main.yml
Expand Up @@ -165,7 +165,6 @@

- include_tasks: keystone_ldap_setup.yml
when:
- "inventory_hostname == ((groups['keystone_all'] | intersect(ansible_play_hosts)) | list)[-1]"
- "keystone_service_setup | bool"
tags:
- keystone-config
Expand Down

0 comments on commit fc3d2fe

Please sign in to comment.