windmill/ansible-role-nodepool
Code Issues Proposed changes

Refactor distro vars include

Browse Source 
We can simplify this logic a little.

Change-Id: Ib7429b7409e05af5b0b82e47c04633fe8e731cb8
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2019-01-27 12:05:23 -05:00
parent 531d0960e8
commit fa67753f17
2 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tasks/main.yaml
View File

@ -13,12 +13,7 @@
# under the License. # under the License.
--- ---
- name: Include OS-specific variables - name: Include OS-specific variables
include_vars: "{{ ansible_distribution|lower }}.yaml" include_vars: "{{ ansible_os_family | lower }}.yaml"
when: ansible_os_family == 'Debian'
- name: Include OS-specific variables
include_vars: "{{ ansible_os_family|lower }}.yaml"
when: ansible_os_family == 'RedHat'
- include: "{{ nodepool_task }}.yaml" - include: "{{ nodepool_task }}.yaml"
with_items: "{{ nodepool_task_manager }}" with_items: "{{ nodepool_task_manager }}"

0
vars/ubuntu.yaml → vars/debian.yaml
View File