diff options
author | Michael Gugino <mgugino@redhat.com> | 2017-12-05 15:14:49 -0500 |
---|---|---|
committer | Michael Gugino <mgugino@redhat.com> | 2017-12-07 09:25:31 -0500 |
commit | 2249ba3d08d1e6c55bf008609c9e4eace16bd917 (patch) | |
tree | 849e51214d2bd93e3fe8f9f4537d8f843416d4d8 /playbooks | |
parent | 4bcb53654635de775c60d0176e3aeeed4856fc86 (diff) | |
download | openshift-2249ba3d08d1e6c55bf008609c9e4eace16bd917.tar.gz openshift-2249ba3d08d1e6c55bf008609c9e4eace16bd917.tar.bz2 openshift-2249ba3d08d1e6c55bf008609c9e4eace16bd917.tar.xz openshift-2249ba3d08d1e6c55bf008609c9e4eace16bd917.zip |
Implement container_runtime playbooks and changes
This commit refactors some duplicate code, removes
usage of set_fact where not needed, and reorganizes
container_runtime role to use include_role.
Diffstat (limited to 'playbooks')
-rw-r--r-- | playbooks/container-runtime/config.yml | 6 | ||||
-rw-r--r-- | playbooks/container-runtime/private/config.yml | 28 | ||||
l--------- | playbooks/container-runtime/private/roles | 1 | ||||
-rw-r--r-- | playbooks/prerequisites.yml | 10 |
4 files changed, 37 insertions, 8 deletions
diff --git a/playbooks/container-runtime/config.yml b/playbooks/container-runtime/config.yml new file mode 100644 index 000000000..f15aa771f --- /dev/null +++ b/playbooks/container-runtime/config.yml @@ -0,0 +1,6 @@ +--- +- import_playbook: ../init/main.yml + vars: + skip_verison: True + +- import_playbook: private/config.yml diff --git a/playbooks/container-runtime/private/config.yml b/playbooks/container-runtime/private/config.yml new file mode 100644 index 000000000..67445edeb --- /dev/null +++ b/playbooks/container-runtime/private/config.yml @@ -0,0 +1,28 @@ +--- +- hosts: "{{ l_containerized_host_groups }}" + vars: + l_chg_temp: "{{ openshift_containerized_host_groups | default([]) }}" + l_containerized_host_groups: "{{ (['oo_nodes_to_config'] | union(l_chg_temp)) | join(':') }}" + # role: container_runtime is necessary here to bring role default variables + # into the play scope. + roles: + - role: container_runtime + tasks: + - include_role: + name: container_runtime + tasks_from: package_docker.yml + when: + - not openshift_docker_use_system_container | bool + - not openshift_use_crio_only | bool + - include_role: + name: container_runtime + tasks_from: systemcontainer_docker.yml + when: + - openshift_docker_use_system_container | bool + - not openshift_use_crio_only | bool + - include_role: + name: container_runtime + tasks_from: systemcontainer_crio.yml + when: + - openshift_use_crio | bool + - openshift_docker_is_node_or_master | bool diff --git a/playbooks/container-runtime/private/roles b/playbooks/container-runtime/private/roles new file mode 120000 index 000000000..148b13206 --- /dev/null +++ b/playbooks/container-runtime/private/roles @@ -0,0 +1 @@ +../../roles/
\ No newline at end of file diff --git a/playbooks/prerequisites.yml b/playbooks/prerequisites.yml index 7dd59c5d8..0cc5fcef8 100644 --- a/playbooks/prerequisites.yml +++ b/playbooks/prerequisites.yml @@ -1,12 +1,6 @@ --- -- include: init/main.yml +- import_playbook: init/main.yml vars: skip_verison: True -- hosts: "{{ l_containerized_host_groups }}" - vars: - l_chg_temp: "{{ openshift_containerized_host_groups | default([]) }}" - l_containerized_host_groups: "{{ (['oo_nodes_to_config'] | union(l_chg_temp)) | join(':') }}" - tasks: - - include_role: - name: container_runtime +- import_playbook: container-runtime/private/config.yml |