345ecbf55e
This might fix some hidden bugs where the check tasks forgot to include params important for the service. We also get a nice optimisation by using a filtered loop instead of task skipping per service with 'when'. As proven in https://review.opendev.org/c/openstack/kolla-ansible/+/914997 This refactoring allows for further optimisation and fixing work to proceed with much less hassle. Including getting rid of many notify statements as the restarts are now safely handled by check-containers. Some notifies had to stay, because of special edge cases eg. in rolling upgrades and loadbalancer config. One downside is we remove the little optimisation for Zun that ignored config change for copying loopback but this is an acceptable tradeoff considering the benefits above. Co-Authored-By: Roman Krček <roman.krcek@tietoevry.com> Change-Id: I855dfef33aa0f3fd1301295bb8ede3e587e7162a Partially-Implements: blueprint performance-improvements
24 lines
746 B
YAML
24 lines
746 B
YAML
---
|
|
- name: Copying over ceph.conf for Zun
|
|
copy:
|
|
src: "{{ node_custom_config }}/zun/zun-compute/{{ ceph_cluster }}.conf"
|
|
dest: "{{ node_config_directory }}/zun-compute/"
|
|
mode: "0660"
|
|
become: true
|
|
|
|
- name: Copy over Ceph keyring files for zun-compute
|
|
copy:
|
|
src: "{{ node_custom_config }}/zun/zun-compute/{{ ceph_cluster }}.client.{{ ceph_cinder_user }}.keyring"
|
|
dest: "{{ node_config_directory }}/zun-compute/"
|
|
mode: "0660"
|
|
become: true
|
|
when: external_ceph_cephx_enabled | bool
|
|
|
|
- name: Ensuring config directory has correct owner and permission
|
|
become: true
|
|
file:
|
|
path: "{{ node_config_directory }}/zun-compute"
|
|
recurse: yes
|
|
owner: "{{ config_owner_user }}"
|
|
group: "{{ config_owner_group }}"
|