Explicit loop variable names

This commit is contained in:
Jérémy Lecour 2022-06-08 15:38:48 +02:00 committed by Jérémy Lecour
parent cbe7985814
commit e6ea44ff29
4 changed files with 24 additions and 10 deletions

View file

@ -27,16 +27,18 @@
- kvm_mem - kvm_mem
notify: restart munin-node notify: restart munin-node
- name: Enable redis munin plugin - name: Enable Munin plugins
file: file:
src: "/usr/local/share/munin/plugins/{{item}}" src: "/usr/local/share/munin/plugins/{{ plugin_name }}"
dest: "/etc/munin/plugins/{{item}}" dest: "/etc/munin/plugins/{{ plugin_name }}"
state: link state: link
force: yes force: yes
loop: loop:
- kvm_cpu - kvm_cpu
- kvm_io - kvm_io
- kvm_mem - kvm_mem
loop_control:
loop_var: plugin_name
notify: restart munin-node notify: restart munin-node
- name: Copy Munin plugins conf - name: Copy Munin plugins conf

View file

@ -39,7 +39,7 @@
- name: Enable redis munin plugin - name: Enable redis munin plugin
file: file:
src: /usr/local/share/munin/plugins/redis_ src: /usr/local/share/munin/plugins/redis_
dest: "/etc/munin/plugins/redis_{{item}}" dest: "/etc/munin/plugins/redis_{{ plugin_name }}"
state: link state: link
loop: loop:
- connected_clients - connected_clients
@ -48,6 +48,8 @@
- per_sec - per_sec
- used_keys - used_keys
- used_memory - used_memory
loop_control:
loop_var: plugin_name
notify: restart munin-node notify: restart munin-node
when: not ansible_check_mode when: not ansible_check_mode
tags: tags:

View file

@ -39,7 +39,7 @@
- name: Enable redis munin plugin - name: Enable redis munin plugin
file: file:
src: /usr/local/share/munin/plugins/redis_ src: /usr/local/share/munin/plugins/redis_
dest: "/etc/munin/plugins/{{ redis_instance_name }}_redis_{{item}}" dest: "/etc/munin/plugins/{{ redis_instance_name }}_redis_{{ plugin_name }}"
state: link state: link
loop: loop:
- connected_clients - connected_clients
@ -48,6 +48,8 @@
- per_sec - per_sec
- used_keys - used_keys
- used_memory - used_memory
loop_control:
loop_var: plugin_name
notify: restart munin-node notify: restart munin-node
when: not ansible_check_mode when: not ansible_check_mode
tags: tags:

View file

@ -38,7 +38,7 @@
- name: "Instance '{{ redis_instance_name }}' config hooks directories are present" - name: "Instance '{{ redis_instance_name }}' config hooks directories are present"
file: file:
dest: "{{ item }}" dest: "{{ _dir }}"
mode: "0755" mode: "0755"
owner: "root" owner: "root"
group: "root" group: "root"
@ -49,6 +49,8 @@
- "{{ redis_conf_dir }}/redis-server.post-up.d" - "{{ redis_conf_dir }}/redis-server.post-up.d"
- "{{ redis_conf_dir }}/redis-server.pre-down.d" - "{{ redis_conf_dir }}/redis-server.pre-down.d"
- "{{ redis_conf_dir }}/redis-server.post-down.d" - "{{ redis_conf_dir }}/redis-server.post-down.d"
loop_control:
loop_var: _dir
when: when:
- ansible_distribution == "Debian" - ansible_distribution == "Debian"
- ansible_distribution_major_version is version('9', '=') - ansible_distribution_major_version is version('9', '=')
@ -56,14 +58,16 @@
- redis - redis
- name: "Instance '{{ redis_instance_name }}' hooks examples are present" - name: "Instance '{{ redis_instance_name }}' hooks examples are present"
command: "cp -a /etc/redis/{{ item }}/00_example {{ redis_conf_dir }}/{{ item }}" command: "cp -a /etc/redis/{{ _dir }}/00_example {{ redis_conf_dir }}/{{ _dir }}"
args: args:
creates: "{{ redis_conf_dir }}/{{ item }}/00_example" creates: "{{ redis_conf_dir }}/{{ _dir }}/00_example"
loop: loop:
- "redis-server.pre-up.d" - "redis-server.pre-up.d"
- "redis-server.post-up.d" - "redis-server.post-up.d"
- "redis-server.pre-down.d" - "redis-server.pre-down.d"
- "redis-server.post-down.d" - "redis-server.post-down.d"
loop_control:
loop_var: _dir
when: when:
- ansible_distribution == "Debian" - ansible_distribution == "Debian"
- ansible_distribution_major_version is version('9', '=') - ansible_distribution_major_version is version('9', '=')
@ -72,7 +76,7 @@
- name: "Instance '{{ redis_instance_name }}' socket/pid directories are present" - name: "Instance '{{ redis_instance_name }}' socket/pid directories are present"
file: file:
dest: "{{ item }}" dest: "{{ _dir }}"
mode: "0755" mode: "0755"
owner: "redis-{{ redis_instance_name }}" owner: "redis-{{ redis_instance_name }}"
group: "redis-{{ redis_instance_name }}" group: "redis-{{ redis_instance_name }}"
@ -81,12 +85,14 @@
loop: loop:
- "{{ redis_pid_dir }}" - "{{ redis_pid_dir }}"
- "{{ redis_socket_dir }}" - "{{ redis_socket_dir }}"
loop_control:
loop_var: _dir
tags: tags:
- redis - redis
- name: "Instance '{{ redis_instance_name }}' data/log directories are present" - name: "Instance '{{ redis_instance_name }}' data/log directories are present"
file: file:
dest: "{{ item }}" dest: "{{ _dir }}"
mode: "0751" mode: "0751"
owner: "redis-{{ redis_instance_name }}" owner: "redis-{{ redis_instance_name }}"
group: "redis-{{ redis_instance_name }}" group: "redis-{{ redis_instance_name }}"
@ -95,6 +101,8 @@
loop: loop:
- "{{ redis_data_dir }}" - "{{ redis_data_dir }}"
- "{{ redis_log_dir }}" - "{{ redis_log_dir }}"
loop_control:
loop_var: _dir
tags: tags:
- redis - redis