bugfix: transfer of report.yaml must work with pod as well (#755)

* bugfix: transfer of report.yaml must work with pod as well * fix output directory Signed-off-by: Matthias Büchse <[email protected]> Co-authored-by: Artem Goncharov <[email protected]>
SovereignCloudStack · Sep 19, 2024 · 7fcc8e9 · 7fcc8e9
1 parent e1fcbb1
commit 7fcc8e9
Showing 1 changed file with 6 additions and 10 deletions.
diff --git a/playbooks/compliance_check.yaml b/playbooks/compliance_check.yaml
@@ -3,20 +3,16 @@
   hosts: all
   tasks:
     - name: Run compliance script
+      # write report.yaml into the proper directory so it will be transferred back by base job
+      # -- this then works regardless of VM/pod
       ansible.builtin.command:
-        cmd: python3 Tests/scs-test-runner.py --config Tests/config.toml --debug run --preset {{ preset }} --output report.yaml
+        cmd: >
+          python3 Tests/scs-test-runner.py --config Tests/config.toml --debug
+          run --preset {{ preset }}
+          --output "{{ ansible_user_dir }}/zuul-output/artifacts/report.yaml"
         chdir: "{{ ansible_user_dir }}/{{ zuul.project.src_dir }}"
       changed_when: true
 
-    - name: Copy result YAML
-      ansible.builtin.synchronize:
-        dest: "{{ zuul.executor.log_root }}/report.yaml"
-        mode: pull
-        src: "{{ ansible_user_dir }}/{{ zuul.project.src_dir }}/report.yaml"
-        verify_host: true
-        owner: no
-        group: no
-
     - name: Return artifact URL
       zuul_return:
         data: