Skip to content

Commit 5d3f46f

Browse files
committed
playbooks: ansible-lint fix fqcn
Fix fqcn ansible-lint rule violations Fixed fqcn rule violations across 175 file(s). Affected files include: - playbooks/monitor-results.yml - playbooks/roles/blktests/handlers/main.yml - playbooks/roles/blktests/tasks/install-deps/debian/main.yml - playbooks/roles/blktests/tasks/install-deps/main.yml - playbooks/roles/blktests/tasks/install-deps/redhat/main.yml ... and 170 more files Applied using the ansible-lint-fix-rules.py script with --fix=fqcn. Generated-by: Ansible Lint (ansible-lint-fix-rules.py) Signed-off-by: Daniel Gomez <[email protected]>
1 parent 4b4591b commit 5d3f46f

File tree

175 files changed

+1405
-1430
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

175 files changed

+1405
-1430
lines changed

playbooks/monitor-results.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
become_method: sudo
66
tasks:
77
- name: Import optional extra_args file
8-
include_vars: "{{ item }}"
8+
ansible.builtin.include_vars: "{{ item }}"
99
ignore_errors: true
1010
with_first_found:
1111
- files:
@@ -16,7 +16,7 @@
1616
tags: vars
1717

1818
- name: Collect monitoring results without stopping monitoring
19-
include_role:
19+
ansible.builtin.include_role:
2020
name: monitoring
2121
tasks_from: monitor_collect_only
2222
when:
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
---
22
# SPDX-License-Identifier GPL-2.0+ OR MIT
33
- name: Restart watchdog daemon
4-
service: name="{{watchdog_service_name}}" state=restarted
4+
ansible.builtin.service: name="{{watchdog_service_name}}" state=restarted

playbooks/roles/blktests/tasks/install-deps/debian/main.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,22 +2,22 @@
22
- name: Update apt cache
33
become: true
44
become_method: sudo
5-
apt:
5+
ansible.builtin.apt:
66
update_cache: true
77
tags: linux
88

99
- name: Install nvme tools
1010
become: true
1111
become_method: sudo
12-
apt:
12+
ansible.builtin.apt:
1313
name:
1414
- nvme-cli
1515
update_cache: true
1616

1717
- name: Install blktests build dependencies
1818
become: true
1919
become_method: sudo
20-
apt:
20+
ansible.builtin.apt:
2121
name:
2222
- bison
2323
- flex
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
---
2-
- include_role:
2+
- ansible.builtin.include_role:
33
name: pkg
44

55
# tasks to install dependencies for oscheck
66
- name: oscheck distribution ospecific setup
7-
import_tasks: tasks/install-deps/debian/main.yml
7+
ansible.builtin.import_tasks: tasks/install-deps/debian/main.yml
88
when: ansible_facts['os_family']|lower == 'debian'
9-
- import_tasks: tasks/install-deps/suse/main.yml
9+
- ansible.builtin.import_tasks: tasks/install-deps/suse/main.yml
1010
when: ansible_facts['os_family']|lower == 'suse'
11-
- import_tasks: tasks/install-deps/redhat/main.yml
11+
- ansible.builtin.import_tasks: tasks/install-deps/redhat/main.yml
1212
when: ansible_facts['os_family']|lower == 'redhat'

playbooks/roles/blktests/tasks/install-deps/redhat/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
- name: Install build dependencies for blktests
33
become: true
44
become_method: sudo
5-
yum:
5+
ansible.builtin.dnf:
66
update_cache: true
77
name: "{{ packages }}"
88
vars:

playbooks/roles/blktests/tasks/install-deps/suse/main.yml

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
---
22
- name: Set generic SUSE specific distro facts
3-
set_fact:
3+
ansible.builtin.set_fact:
44
is_sle: '{{ (ansible_distribution == "SLES") or (ansible_distribution == "SLED") }}'
55
is_leap: '{{ "Leap" in ansible_distribution }}'
66
is_tumbleweed: '{{ "openSUSE Tumbleweed" == ansible_distribution }}'
77

88
- name: Set SLE specific version labels to make checks easier
9-
set_fact:
9+
ansible.builtin.set_fact:
1010
is_sle10: '{{ ansible_distribution_major_version == "10" }}'
1111
is_sle11: '{{ ansible_distribution_major_version == "11" }}'
1212
is_sle12: '{{ ansible_distribution_major_version == "12" }}'
@@ -24,7 +24,7 @@
2424
- is_sle|bool
2525

2626
- name: Set SLE specific version labels to make checks easier when not SLE
27-
set_fact:
27+
ansible.builtin.set_fact:
2828
is_sle10: false
2929
is_sle11: false
3030
is_sle12: false
@@ -42,22 +42,22 @@
4242
- not is_sle|bool
4343

4444
- name: By default we assume we have figured out how to add repos on a release
45-
set_fact:
45+
ansible.builtin.set_fact:
4646
repos_present: true
4747

4848
- name: Lets us disable things which require a zypper repo present
49-
set_fact:
49+
ansible.builtin.set_fact:
5050
repos_present: false
5151
when:
5252
- is_sle|bool
5353
- is_sle10|bool or is_sle11|bool
5454

5555
- name: The default is to assume all distros supports nvme-utils
56-
set_fact:
56+
ansible.builtin.set_fact:
5757
lacks_nvme_utils: false
5858

5959
- name: Does this release lack nvme-utils
60-
set_fact:
60+
ansible.builtin.set_fact:
6161
lacks_nvme_utils: true
6262
when:
6363
- is_sle|bool
@@ -167,35 +167,35 @@
167167
- is_sle15sp2|bool or is_sle15sp3|bool
168168

169169
- name: The default is to assume we do not want to add the benchmark repo
170-
set_fact:
170+
ansible.builtin.set_fact:
171171
add_benchmark_repo: false
172172

173173
- name: Does this release require the benchmark repo
174-
set_fact:
174+
ansible.builtin.set_fact:
175175
add_benchmark_repo: true
176176
when:
177177
- is_sle15sp2|bool or is_sle15sp3|bool or is_sle15sp4|bool or is_sle12sp5|bool
178178

179179
- name: Add benchmark sles12sp5 opensuse repo when needed
180180
become: true
181181
become_method: sudo
182-
script:
182+
ansible.builtin.script:
183183
cmd: "{{ role_path }}/scripts/add-suse-repo-if-not-found.sh http://download.opensuse.org/repositories/benchmark/SLE_12_SP5/ benchmark benchmark"
184184
when:
185185
- is_sle12sp5|bool
186186

187187
- name: Add benchmark sles15sp2 opensuse repo when needed
188188
become: true
189189
become_method: sudo
190-
script:
190+
ansible.builtin.script:
191191
cmd: "{{ role_path }}/scripts/add-suse-repo-if-not-found.sh http://download.opensuse.org/repositories/benchmark/SLE_15_SP2/ benchmark benchmark"
192192
when:
193193
- is_sle15sp2|bool
194194

195195
- name: Add benchmark sles15sp3 opensuse repo when needed
196196
become: true
197197
become_method: sudo
198-
script:
198+
ansible.builtin.script:
199199
cmd: "{{ role_path }}/scripts/add-suse-repo-if-not-found.sh http://download.opensuse.org/repositories/benchmark/SLE_15_SP3/ benchmark benchmark"
200200
when:
201201
- is_sle15sp3|bool or is_sle15sp4|bool
@@ -211,11 +211,11 @@
211211
- add_benchmark_repo|bool
212212

213213
- name: The default is to assume we do not want to to compile nbd manually
214-
set_fact:
214+
ansible.builtin.set_fact:
215215
compile_nbd: false
216216

217217
- name: Set when we need to compile nbd
218-
set_fact:
218+
ansible.builtin.set_fact:
219219
compile_nbd: true
220220
when:
221221
- is_sle15sp2|bool or is_sle15sp3|bool or is_sle15sp4|bool or is_leap|bool
@@ -232,7 +232,7 @@
232232
- compile_nbd|bool
233233

234234
- name: git clone nbd
235-
git:
235+
ansible.builtin.git:
236236
repo: "{{ nbd_git }}"
237237
dest: "{{ nbd_data }}"
238238
update: true
@@ -243,7 +243,7 @@
243243
tags: ["git", "blktests", "nbd"]
244244

245245
- name: Skip documentation for nbd as we lack the requirement of docbook2man
246-
file:
246+
ansible.builtin.file:
247247
path: "{{ nbd_data }}/man/{{ item }}"
248248
state: touch
249249
with_items:
@@ -260,33 +260,33 @@
260260
- compile_nbd|bool
261261

262262
- name: Run autogen for nbd
263-
command: "./autogen.sh"
263+
ansible.builtin.command: "./autogen.sh"
264264
tags: ["blktests", "nbd"]
265265
args:
266266
chdir: "{{ nbd_data }}"
267267

268268
- name: Run configure for nbd
269-
command: "./configure"
269+
ansible.builtin.command: "./configure"
270270
tags: ["blktests", "nbd"]
271271
args:
272272
chdir: "{{ nbd_data }}"
273273

274274
- name: Get nproc
275-
command: "{{ num_jobs }}"
275+
ansible.builtin.command: "{{ num_jobs }}"
276276
tags: ["git", "blktests", "nbd"]
277277
register: nproc
278278

279279
- name: Build nbd
280280
tags: ["git", "blktests", "nbd"]
281-
make:
281+
community.general.make:
282282
chdir: "{{ nbd_data }}"
283283
jobs: "{{ nproc.stdout }}"
284284

285285
- name: Install nbd
286286
become: true
287287
become_flags: "su - -c"
288288
become_method: sudo
289-
command: "{{ make }} install"
289+
ansible.builtin.command: "{{ make }} install"
290290
tags: ["git", "blktests", "nbd"]
291291
args:
292292
chdir: "{{ nbd_data }}"

0 commit comments

Comments
 (0)