Skip to content

Commit 857ba8c

Browse files
authored
Replace with_items with loop (#282)
`loop` is now the recommended way to loop through lists instead of `with_items`
1 parent 6ac3cba commit 857ba8c

File tree

6 files changed

+7
-8
lines changed

6 files changed

+7
-8
lines changed

tasks/modules/install-geoip.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
path: /etc/nginx/nginx.conf
2323
insertbefore: BOF
2424
line: "{{ item }}"
25-
with_items:
25+
loop:
2626
- load_module modules/ngx_http_geoip_module.so;
2727
- load_module modules/ngx_stream_geoip_module.so;
2828
when: not nginx_main_template_enable

tasks/modules/install-njs.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
path: /etc/nginx/nginx.conf
1717
insertbefore: BOF
1818
line: "{{ item }}"
19-
with_items:
19+
loop:
2020
- load_module modules/ngx_http_js_module.so;
2121
- load_module modules/ngx_stream_js_module.so;
2222
when: not nginx_main_template_enable

tasks/opensource/setup-debian.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,7 @@
1616
- name: "(Install: Debian/Ubuntu) Add NGINX Repository"
1717
apt_repository:
1818
repo: "{{ item }}"
19-
with_items:
20-
- "{{ repository }}"
19+
loop: "{{ repository }}"
2120

2221
- name: "(Install: Debian/Ubuntu) Install NGINX"
2322
apt:

tasks/plus/setup-license.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
src: "{{ item }}"
1313
dest: /etc/ssl/nginx
1414
decrypt: yes
15-
with_items:
15+
loop:
1616
- "{{ nginx_license.certificate }}"
1717
- "{{ nginx_license.key }}"
1818

tasks/unit/install-modules.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@
33
package:
44
name: "{{ item }}"
55
state: present
6-
with_items: "{{ nginx_unit_modules }}"
6+
loop: "{{ nginx_unit_modules }}"
77
when: ansible_os_family != "FreeBSD"
88
notify: "(Handler: Debian/Ubuntu/CentOS/RedHat) Start NGINX Unit"
99

1010
- name: "(Install: FreeBSD) Install NGINX Unit Modules"
1111
portinstall:
1212
name: "{{ item }}"
1313
state: present
14-
with_items: "{{ nginx_unit_modules }}"
14+
loop: "{{ nginx_unit_modules }}"
1515
when: ansible_os_family == "FreeBSD"
1616
notify: "(Handler: FreeBSD) Start NGINX Unit"

tasks/unit/setup-debian.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@
22
- name: "(Install: Debian/Ubuntu) Add NGINX Unit Repository"
33
apt_repository:
44
repo: "{{ item }}"
5-
with_items:
5+
loop:
66
- deb [arch=amd64] https://packages.nginx.org/unit/{{ ansible_distribution|lower }}/ {{ ansible_distribution_release }} unit
77
- deb-src https://packages.nginx.org/unit/{{ ansible_distribution|lower }}/ {{ ansible_distribution_release }} unit

0 commit comments

Comments
 (0)