openstack
/
openstack-ansible-os_keystone
Code Issues Proposed changes

Ensure that LDAP config is deployed on all keystone hosts 

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 3a6a55b8d0)
This commit is contained in:
Jesse Pretorius 2018-11-28 11:43:14 +00:00 committed by Jesse Pretorius (odyssey4me)
parent 5a54cc6ba5
commit fc3d2fe4b6
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tasks/keystone_ldap_setup.yml
View File

@ -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']) }}

1
tasks/main.yml
View File

@ -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