Skip to content

Commit 100ceb5

Browse files
committed
Tidy up by separating all ansible tasks with a new line and remove a commented out line.
1 parent a7076b3 commit 100ceb5

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

etc/kayobe/ansible/stackhpc-cloud-tests.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@
7272
ansible.builtin.stat:
7373
path: "{{ repo_tmpdir.path }}/stackhpc_cloud_tests/monitoring"
7474
register: stackhpc_cloud_monitoring_tests
75+
7576
- name: Run StackHPC Cloud monitoring tests
7677
ansible.builtin.command:
7778
cmd: >
@@ -121,10 +122,11 @@
121122
# remote test execution? That would place all results in a single file
122123
# and allow us to execute all tests from a single host.
123124
# https://testinfra.readthedocs.io/en/latest/backends.html#connection-backends
124-
- name: Check for StackHPC Cloud monitoring tests
125+
- name: Check for StackHPC Cloud host tests
125126
ansible.builtin.stat:
126127
path: "{{ repo_tmpdir.path }}/stackhpc_cloud_tests/host"
127128
register: stackhpc_cloud_host_tests
129+
128130
- name: Run StackHPC Cloud host tests
129131
ansible.builtin.command:
130132
cmd: >
@@ -134,7 +136,6 @@
134136
--timeout {{ sct_timeout }}
135137
-vv
136138
"{{ repo_tmpdir.path }}/stackhpc_cloud_tests/host"
137-
# chmod 644 {{ results_tmpdir.path }}/host-{{ inventory_hostname }}.html
138139
environment:
139140
DOCKER_VERSION_MIN: "{{ sct_docker_version_min }}"
140141
DOCKER_VERSION_MAX: "{{ sct_docker_version_max }}"
@@ -150,6 +151,7 @@
150151
# Some host checks may need to run as root
151152
become: true
152153
when: "'overcloud' in group_names and stackhpc_cloud_host_tests.stat.exists"
154+
153155
# Host test results will be owned by root - we need to read and delete them
154156
- name: Change permissions on SCT host test results
155157
ansible.builtin.command:
@@ -167,6 +169,7 @@
167169
recursive: true
168170
# For jump host
169171
use_ssh_args: true
172+
170173
- name: Write a file containing failed test runs
171174
ansible.builtin.copy:
172175
content: |-

0 commit comments

Comments
 (0)