diff --git a/roles/container-engine/containerd/tasks/containerd_repo.yml b/roles/container-engine/containerd/tasks/containerd_repo.yml index 81971e7f3389671c4a5515afb3c5eab26befe9e7..64c360d1b84250fee0e95b2a9d981b1a18dc8f77 100644 --- a/roles/container-engine/containerd/tasks/containerd_repo.yml +++ b/roles/container-engine/containerd/tasks/containerd_repo.yml @@ -13,14 +13,11 @@ when: ansible_pkg_mgr == 'apt' - name: ensure containerd repository is enabled - action: "{{ containerd_repo_info.pkg_repo }}" - args: + apt_repository: repo: "{{ item }}" state: present with_items: "{{ containerd_repo_info.repos }}" - when: - - ansible_os_family in ['Ubuntu', 'Debian'] - - containerd_repo_info.repos|length > 0 + when: ansible_pkg_mgr == 'apt' - name: Configure containerd repository on Fedora template: diff --git a/roles/container-engine/containerd/vars/debian.yml b/roles/container-engine/containerd/vars/debian.yml index 270a5e11fcbefcd0fcdbd2e837a252f1ddf14f82..7b73083daccc43f99f0d83075216eeb42aff28b0 100644 --- a/roles/container-engine/containerd/vars/debian.yml +++ b/roles/container-engine/containerd/vars/debian.yml @@ -9,7 +9,6 @@ containerd_repo_key_info: - '{{ containerd_debian_repo_repokey }}' containerd_repo_info: - pkg_repo: apt_repository repos: - > deb {{ containerd_debian_repo_base_url }} diff --git a/roles/container-engine/containerd/vars/redhat.yml b/roles/container-engine/containerd/vars/redhat.yml index 25cb7247046e22388b76a0467d740ff8fb87b131..e51f2c89c317a3a629638dbc5e629df03e14ed2c 100644 --- a/roles/container-engine/containerd/vars/redhat.yml +++ b/roles/container-engine/containerd/vars/redhat.yml @@ -3,7 +3,3 @@ containerd_package_info: enablerepo: "docker-ce" pkgs: - "{{ containerd_versioned_pkg[containerd_version | string] }}" - -containerd_repo_info: - pkg_repo: '' - repos: [] diff --git a/roles/container-engine/containerd/vars/suse.yml b/roles/container-engine/containerd/vars/suse.yml index e32c249ab95f78366051435a09e52921e19324d1..fb45f9ca8f16d8861c166636672842e421bd2429 100644 --- a/roles/container-engine/containerd/vars/suse.yml +++ b/roles/container-engine/containerd/vars/suse.yml @@ -5,7 +5,3 @@ containerd_package: containerd containerd_package_info: pkgs: - "{{ containerd_package }}" - -containerd_repo_info: - pkg_repo: '' - repos: [] diff --git a/roles/container-engine/containerd/vars/ubuntu.yml b/roles/container-engine/containerd/vars/ubuntu.yml index 94fa56a2dec4c7347e9f327682b2fb390a9ceea0..a43797e65215f2eedade15cc5a8b516a2e27d2f4 100644 --- a/roles/container-engine/containerd/vars/ubuntu.yml +++ b/roles/container-engine/containerd/vars/ubuntu.yml @@ -9,7 +9,6 @@ containerd_repo_key_info: - '{{ containerd_ubuntu_repo_repokey }}' containerd_repo_info: - pkg_repo: apt_repository repos: - > deb {{ containerd_ubuntu_repo_base_url }} diff --git a/roles/container-engine/docker/tasks/main.yml b/roles/container-engine/docker/tasks/main.yml index 325b70f02759e21cfae93d315c80116659523df4..ce7b39480ebd25231450b6a3cd9b517b8433137d 100644 --- a/roles/container-engine/docker/tasks/main.yml +++ b/roles/container-engine/docker/tasks/main.yml @@ -67,12 +67,11 @@ when: ansible_pkg_mgr == 'apt' - name: ensure docker-ce repository is enabled - action: "{{ docker_repo_info.pkg_repo }}" - args: + apt_repository: repo: "{{ item }}" state: present with_items: "{{ docker_repo_info.repos }}" - when: not (ansible_os_family in ["Flatcar Container Linux by Kinvolk", "RedHat", "Suse", "ClearLinux"] or is_ostree) and (docker_repo_info.repos|length > 0) + when: ansible_pkg_mgr == 'apt' - name: Configure docker repository on Fedora template: diff --git a/roles/container-engine/docker/vars/debian.yml b/roles/container-engine/docker/vars/debian.yml index 097f3c753a6a24d85c5d65f7adba6189da24fc2d..ad7abb5e2037f1e4d1ed4df55ed9cf0850d3b9f5 100644 --- a/roles/container-engine/docker/vars/debian.yml +++ b/roles/container-engine/docker/vars/debian.yml @@ -26,7 +26,6 @@ docker_repo_key_info: - 9DC858229FC7DD38854AE2D88D81803C0EBFCD88 docker_repo_info: - pkg_repo: apt_repository repos: - > deb {{ docker_debian_repo_base_url }} diff --git a/roles/container-engine/docker/vars/redhat.yml b/roles/container-engine/docker/vars/redhat.yml index 25c790b57de104627c0f8bde44d5baf016bd912e..6551b71a86ea5acd9a0e173f263de580f5db460b 100644 --- a/roles/container-engine/docker/vars/redhat.yml +++ b/roles/container-engine/docker/vars/redhat.yml @@ -22,7 +22,3 @@ docker_package_info: - "{{ containerd_versioned_pkg[containerd_version | string] }}" - "{{ docker_cli_versioned_pkg[docker_cli_version | string] }}" - "{{ docker_versioned_pkg[docker_version | string] }}" - -docker_repo_info: - pkg_repo: '' - repos: [] diff --git a/roles/container-engine/docker/vars/suse.yml b/roles/container-engine/docker/vars/suse.yml index fb7b91ffee7480a1d0820caf43b465a53f8a2052..2d9fbf0214c5d8102821f9d61d4739e35b4da2f2 100644 --- a/roles/container-engine/docker/vars/suse.yml +++ b/roles/container-engine/docker/vars/suse.yml @@ -4,7 +4,3 @@ docker_package_info: pkgs: - docker - containerd - -docker_repo_info: - pkg_repo: '' - repos: [] diff --git a/roles/container-engine/docker/vars/ubuntu.yml b/roles/container-engine/docker/vars/ubuntu.yml index 128510b12ecea6038392ad62da9ac0ca6bff78ac..21ccc066526b89008549a94c5fed2f1ae8fad541 100644 --- a/roles/container-engine/docker/vars/ubuntu.yml +++ b/roles/container-engine/docker/vars/ubuntu.yml @@ -26,7 +26,6 @@ docker_repo_key_info: - 9DC858229FC7DD38854AE2D88D81803C0EBFCD88 docker_repo_info: - pkg_repo: apt_repository repos: - > deb [arch={{ host_architecture }}] {{ docker_ubuntu_repo_base_url }}