From a5b89b132a5b681ce0e81e46216443d0307c789f Mon Sep 17 00:00:00 2001 From: Pawel Wieczorek Date: Thu, 30 Jul 2020 12:20:02 +0200 Subject: Replace deprecated "with_items" with loops As of Ansible 2.5+, the recommended way to perform loops is to use the new "loop" keyword instead of "with_*" style loops [1]. This issue was reported by: Bartek Grzybowski [1] https://docs.ansible.com/ansible/latest/user_guide/playbooks_loops.html Issue-ID: INT-1601 Change-Id: Icf9079fc5c22ac034631397ea46d2b03fb4298ab Signed-off-by: Pawel Wieczorek --- .../ansible/roles/openstack/create_devstack_hosts/tasks/main.yml | 3 +-- .../ansible/roles/openstack/create_devstack_network/tasks/main.yml | 2 +- .../roles/openstack/create_devstack_securitygroup/tasks/main.yml | 2 +- .../ansible/roles/openstack/destroy_devstack_hosts/tasks/main.yml | 3 +-- .../ansible/roles/openstack/destroy_devstack_keypair/tasks/main.yml | 2 +- .../ansible/roles/openstack/destroy_devstack_network/tasks/main.yml | 2 +- .../roles/openstack/destroy_devstack_securitygroup/tasks/main.yml | 2 +- 7 files changed, 7 insertions(+), 9 deletions(-) (limited to 'deployment/noheat') diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_hosts/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_hosts/tasks/main.yml index f79314c72..731bca04f 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_hosts/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_hosts/tasks/main.yml @@ -1,4 +1,3 @@ --- - include: create_host.yml host={{ item }} - with_items: - - "{{ hosts }}" + loop: "{{ hosts }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_network/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_network/tasks/main.yml index 5c8af745f..5e3ef67f5 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_network/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_network/tasks/main.yml @@ -1,4 +1,4 @@ --- - include: create_network.yml net={{ item }} - with_items: + loop: - "{{ network }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_securitygroup/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_securitygroup/tasks/main.yml index da125cbc4..3ce0e182b 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_securitygroup/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/create_devstack_securitygroup/tasks/main.yml @@ -1,4 +1,4 @@ --- - include: create_securitygroup.yml secgrp={{ item }} - with_items: + loop: - "{{ securitygroup }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_hosts/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_hosts/tasks/main.yml index eddf09736..5ce130d44 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_hosts/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_hosts/tasks/main.yml @@ -1,4 +1,3 @@ --- - include: destroy_host.yml host={{ item }} - with_items: - - "{{ hosts }}" + loop: "{{ hosts }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_keypair/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_keypair/tasks/main.yml index b106ee0cd..6025b82b3 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_keypair/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_keypair/tasks/main.yml @@ -7,6 +7,6 @@ file: state: absent path: "{{ item }}" - with_items: + loop: - "~/.ssh/{{ keypair.name }}.pub" - "~/.ssh/{{ keypair.name }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_network/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_network/tasks/main.yml index 1bfab0d00..e52dcbdb8 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_network/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_network/tasks/main.yml @@ -1,4 +1,4 @@ --- - include: destroy_network.yml net={{ item }} - with_items: + loop: - "{{ network }}" diff --git a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_securitygroup/tasks/main.yml b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_securitygroup/tasks/main.yml index 586e18067..de098afec 100644 --- a/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_securitygroup/tasks/main.yml +++ b/deployment/noheat/infra-openstack/ansible/roles/openstack/destroy_devstack_securitygroup/tasks/main.yml @@ -1,4 +1,4 @@ --- - include: destroy_securitygroup.yml secgrp={{ item }} - with_items: + loop: - "{{ securitygroup }}" -- cgit 1.2.3-korg