9702d4c3c3
Including tasks has a performance penalty when compared with importing tasks. If the include has a condition associated with it, then the overhead of the include may be lower than the overhead of skipping all imported tasks. In the case of the check-containers.yml include, the included file only has a single task, so the overhead of skipping this task will not be greater than the overhead of the task import. It therefore makes sense to switch to use import_tasks there. Partially-Implements: blueprint performance-improvements Change-Id: I65d911670649960708b9f6a4c110d1a7df1ad8f7
36 lines
966 B
YAML
36 lines
966 B
YAML
---
|
|
- name: Ensuring config directories exist
|
|
file:
|
|
path: "{{ node_config_directory }}/{{ item.key }}"
|
|
state: "directory"
|
|
owner: "{{ config_owner_user }}"
|
|
group: "{{ config_owner_group }}"
|
|
mode: "0770"
|
|
become: true
|
|
when:
|
|
- inventory_hostname in groups[item.value.group]
|
|
- item.value.enabled | bool
|
|
with_dict: "{{ vmtp_services }}"
|
|
|
|
- name: Copying over configuration file for vmtp
|
|
vars:
|
|
service: "{{ vmtp_services['vmtp'] }}"
|
|
merge_yaml:
|
|
sources:
|
|
- "{{ role_path }}/templates/{{ item }}.j2"
|
|
- "{{ node_custom_config }}/{{ item }}"
|
|
- "{{ node_custom_config }}/vmtp/{{ item }}"
|
|
dest: "{{ node_config_directory }}/vmtp/{{ item }}"
|
|
mode: "0660"
|
|
become: true
|
|
when:
|
|
- inventory_hostname in groups[service.group]
|
|
- service.enabled | bool
|
|
with_items:
|
|
- "cfg.default.yaml"
|
|
notify:
|
|
- Restart vmtp container
|
|
|
|
- import_tasks: check-containers.yml
|
|
when: kolla_action != "config"
|