openstack/openstack-ansible-os_keystone
Code Issues Proposed changes

Ensure that LDAP config is deployed on all keystone hosts

Browse Source 
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
This commit is contained in:
Jesse Pretorius 2018-11-28 11:43:14 +00:00
parent 7b56f1d9b7
commit 3a6a55b8d0
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

@ -142,7 +142,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