From 2b3a0468711a065acbcf7c1c12ab91b97924bf61 Mon Sep 17 00:00:00 2001 From: caoyuan <cao.yuan@99cloud.net> Date: Wed, 27 Jun 2018 16:59:28 +0800 Subject: [PATCH] Add the role enabled check for some projects 1. Add the role enabled check for some projects 2. adjust the file created positon for keystone to keep consistence with others Change-Id: Id2b893ba546b3adf41d97927f8d20dca403a0457 --- ansible/roles/congress/tasks/config.yml | 4 +++- ansible/roles/keystone/tasks/config.yml | 26 ++++++++++++------------- ansible/roles/mistral/tasks/config.yml | 4 +++- ansible/roles/tacker/tasks/config.yml | 2 +- ansible/roles/zun/tasks/config.yml | 4 +++- 5 files changed, 23 insertions(+), 17 deletions(-) diff --git a/ansible/roles/congress/tasks/config.yml b/ansible/roles/congress/tasks/config.yml index 75fcffa09..7d09577f7 100644 --- a/ansible/roles/congress/tasks/config.yml +++ b/ansible/roles/congress/tasks/config.yml @@ -7,7 +7,9 @@ group: "{{ config_owner_group }}" mode: "0770" become: true - when: inventory_hostname in groups[item.value.group] + when: + - inventory_hostname in groups[item.value.group] + - item.value.enabled | bool with_dict: "{{ congress_services }}" - name: Check if policies shall be overwritten diff --git a/ansible/roles/keystone/tasks/config.yml b/ansible/roles/keystone/tasks/config.yml index 047b65856..0c26075f7 100644 --- a/ansible/roles/keystone/tasks/config.yml +++ b/ansible/roles/keystone/tasks/config.yml @@ -1,4 +1,17 @@ --- +- 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: "{{ keystone_services }}" + - name: Check if policies shall be overwritten local_action: stat path="{{ item }}" run_once: True @@ -21,19 +34,6 @@ run_once: True register: keystone_domain_directory -- 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: "{{ keystone_services }}" - - name: Creating Keystone Domain directory vars: keystone: "{{ keystone_services.keystone }}" diff --git a/ansible/roles/mistral/tasks/config.yml b/ansible/roles/mistral/tasks/config.yml index be213b52a..c16dc4b72 100644 --- a/ansible/roles/mistral/tasks/config.yml +++ b/ansible/roles/mistral/tasks/config.yml @@ -7,7 +7,9 @@ group: "{{ config_owner_group }}" mode: "0770" become: true - when: inventory_hostname in groups[item.value.group] + when: + - inventory_hostname in groups[item.value.group] + - item.value.enabled | bool with_dict: "{{ mistral_services }}" - name: Check if policies shall be overwritten diff --git a/ansible/roles/tacker/tasks/config.yml b/ansible/roles/tacker/tasks/config.yml index b60b8ed51..abc96248d 100644 --- a/ansible/roles/tacker/tasks/config.yml +++ b/ansible/roles/tacker/tasks/config.yml @@ -9,7 +9,7 @@ become: true when: - inventory_hostname in groups[item.value.group] - - item.value.enabled + - item.value.enabled | bool with_dict: "{{ tacker_services }}" - name: Check if policies shall be overwritten diff --git a/ansible/roles/zun/tasks/config.yml b/ansible/roles/zun/tasks/config.yml index c9e799755..0aac546f3 100644 --- a/ansible/roles/zun/tasks/config.yml +++ b/ansible/roles/zun/tasks/config.yml @@ -7,7 +7,9 @@ group: "{{ config_owner_group }}" mode: "0770" become: true - when: inventory_hostname in groups[item.value.group] + when: + - inventory_hostname in groups[item.value.group] + - item.value.enabled | bool with_dict: "{{ zun_services }}" - name: Check if policies shall be overwritten -- GitLab