Skip to content

Commit 9bfbc66

Browse files
committed
refactor: make task names unique; fix indentation
the new ansible-lint wants all task names to be unique, and is more strict about indentation Signed-off-by: Rich Megginson <[email protected]>
1 parent f7dca14 commit 9bfbc66

46 files changed

Lines changed: 436 additions & 436 deletions

Some content is hidden

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

tests/test-verify-pool-members.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,13 @@
4545
if storage_test_pool.encryption else 'disk' }}"
4646
when: storage_test_pool.type == 'lvm'
4747

48-
- name: Set expected pv type
48+
- name: Set expected pv type - 2
4949
set_fact:
5050
_storage_test_expected_pv_type: "{{ 'partition'
5151
if storage_use_partitions | d(false) else 'disk' }}"
5252
when: storage_test_pool.type == 'lvm' and not storage_test_pool.encryption
5353

54-
- name: Set expected pv type
54+
- name: Set expected pv type - 3
5555
set_fact:
5656
_storage_test_expected_pv_type: "{{ storage_test_pool.raid_level }}"
5757
when:

tests/test-verify-volume-device.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
Incorrect device node presence for volume {{ storage_test_volume.name }}
1414
when: _storage_test_volume_present or storage_test_volume.type == 'disk'
1515

16-
- name: Verify the presence/absence of the device node
16+
- name: Verify the presence/absence of the device node - 2
1717
assert:
1818
that: not storage_test_dev.stat.exists
1919
msg: >-

tests/test-verify-volume-size.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@
127127
- storage_test_volume.thin_pool_size is not none
128128
- "'%' not in storage_test_volume.thin_pool_size"
129129

130-
- name: Calculate the expected size based on pool size and percentage value
130+
- name: Calculate the expected size based on pool size and percentage value - 2
131131
set_fact:
132132
storage_test_expected_thin_pool_size: "{{
133133
(_storage_test_max_thin_pool_size.bytes *
@@ -158,7 +158,7 @@
158158
debug:
159159
var: storage_test_actual_size
160160

161-
- name: Show expected size
161+
- name: Show expected size - 2
162162
debug:
163163
var: storage_test_expected_size
164164

tests/tests_change_disk_fs.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
fs_type: "{{ fs_type_after }}"
5555
disks: "{{ unused_disks }}"
5656

57-
- name: Verify role results
57+
- name: Verify role results - 2
5858
include_tasks: verify-role-results.yml
5959

6060
- name: Repeat the previous invocation to verify idempotence
@@ -68,7 +68,7 @@
6868
fs_type: "{{ fs_type_after }}"
6969
disks: "{{ unused_disks }}"
7070

71-
- name: Verify role results
71+
- name: Verify role results - 3
7272
include_tasks: verify-role-results.yml
7373

7474
- name: Clean up
@@ -82,5 +82,5 @@
8282
mount_point: "{{ mount_location }}"
8383
state: absent
8484

85-
- name: Verify role results
85+
- name: Verify role results - 4
8686
include_tasks: verify-role-results.yml

tests/tests_change_disk_mount.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
mount_point: "{{ mount_location_after }}"
5151
disks: "{{ unused_disks }}"
5252

53-
- name: Verify role results
53+
- name: Verify role results - 2
5454
include_tasks: verify-role-results.yml
5555

5656
- name: Repeat the previous invocation to verify idempotence
@@ -63,7 +63,7 @@
6363
mount_point: "{{ mount_location_after }}"
6464
disks: "{{ unused_disks }}"
6565

66-
- name: Verify role results
66+
- name: Verify role results - 3
6767
include_tasks: verify-role-results.yml
6868

6969
- name: Clean up
@@ -77,5 +77,5 @@
7777
disks: "{{ unused_disks }}"
7878
state: absent
7979

80-
- name: Verify role results
80+
- name: Verify role results - 4
8181
include_tasks: verify-role-results.yml

tests/tests_change_fs.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
fs_type: "{{ fs_after }}"
6161
mount_point: "{{ mount_location }}"
6262

63-
- name: Verify role results
63+
- name: Verify role results - 2
6464
include_tasks: verify-role-results.yml
6565

6666
- name: Re-run the role on the same volume without specifying fs_type
@@ -82,7 +82,7 @@
8282
- blivet_output.pools[0].volumes[0].fs_type == fs_after
8383
msg: Failed to preserve omitted fs_type on existing lvm volume
8484

85-
- name: Verify role results
85+
- name: Verify role results - 3
8686
include_tasks: verify-role-results.yml
8787

8888
- name: Repeat the previous invocation to verify idempotence
@@ -98,7 +98,7 @@
9898
fs_type: "{{ fs_after }}"
9999
mount_point: "{{ mount_location }}"
100100

101-
- name: Verify role results
101+
- name: Verify role results - 4
102102
include_tasks: verify-role-results.yml
103103

104104
- name: Remove the FS
@@ -113,7 +113,7 @@
113113
size: "{{ volume_size }}"
114114
fs_type: unformatted
115115

116-
- name: Verify role results
116+
- name: Verify role results - 5
117117
include_tasks: verify-role-results.yml
118118

119119

@@ -131,5 +131,5 @@
131131
fs_type: "{{ fs_after }}"
132132
mount_point: "{{ mount_location }}"
133133

134-
- name: Verify role results
134+
- name: Verify role results - 6
135135
include_tasks: verify-role-results.yml

tests/tests_change_fs_use_partitions.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
fs_type: "{{ fs_type_after }}"
6262
mount_point: "{{ mount_location }}"
6363

64-
- name: Verify role results
64+
- name: Verify role results - 2
6565
include_tasks: verify-role-results.yml
6666

6767
- name: Repeat the previous invocation to verify idempotence
@@ -77,7 +77,7 @@
7777
fs_type: "{{ fs_type_after }}"
7878
mount_point: "{{ mount_location }}"
7979

80-
- name: Verify role results
80+
- name: Verify role results - 3
8181
include_tasks: verify-role-results.yml
8282

8383
- name: Clean up
@@ -95,5 +95,5 @@
9595
mount_point: "{{ mount_location }}"
9696
state: absent
9797

98-
- name: Verify role results
98+
- name: Verify role results - 4
9999
include_tasks: verify-role-results.yml

tests/tests_change_mount.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@
5656
size: "{{ volume_size }}"
5757
mount_point: "{{ mount_location_after }}"
5858

59-
- name: Verify role results
59+
- name: Verify role results - 2
6060
include_tasks: verify-role-results.yml
6161

6262
- name: Repeat the previous invocation to verify idempotence
@@ -71,7 +71,7 @@
7171
size: "{{ volume_size }}"
7272
mount_point: "{{ mount_location_after }}"
7373

74-
- name: Verify role results
74+
- name: Verify role results - 3
7575
include_tasks: verify-role-results.yml
7676

7777
- name: Clean up
@@ -87,5 +87,5 @@
8787
size: "{{ volume_size }}"
8888
mount_point: "{{ mount_location_after }}"
8989

90-
- name: Verify role results
90+
- name: Verify role results - 4
9191
include_tasks: verify-role-results.yml

tests/tests_create_disk_then_remove.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
- blivet_output.volumes[0].fs_type == 'ext4'
6565
msg: File system not preserved on existing partition volume.
6666

67-
- name: Verify role results
67+
- name: Verify role results - 2
6868
include_tasks: verify-role-results.yml
6969

7070
- name: Remove the disk device created above
@@ -78,5 +78,5 @@
7878
mount_point: "{{ mount_location }}"
7979
state: absent
8080

81-
- name: Verify role results
81+
- name: Verify role results - 3
8282
include_tasks: verify-role-results.yml

tests/tests_create_lv_size_equal_to_vg.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,5 @@
5959
- name: test1
6060
mount_point: "{{ mount_location }}"
6161

62-
- name: Verify role results
62+
- name: Verify role results - 2
6363
include_tasks: verify-role-results.yml

0 commit comments

Comments
 (0)