test.yml: upload failed-logs as an artifact if test-templates.sh fails and failed-logs exists.

Signed-off-by: Norio Nomura <[email protected]>

silence yamllint

Signed-off-by: Norio Nomura <[email protected]>

test:yml: switch to `if: always()`, if the previous step uses `retry`

Signed-off-by: Norio Nomura <[email protected]>

test.yml: change to `failure-logs`

Signed-off-by: Norio Nomura <[email protected]>

Author: norio-nomura

.github/actions/upload_failure_logs_if_exists/action.yml

@@ -0,0 +1,15 @@
+name: 'upload failure-logs if exists'
+description: 'upload failure-logs if exists'
+runs:
+  using: "composite"
+  steps:
+  - name: "Check if failure-logs exists"
+    if: always()
+    id: check-if-failure-logs-exists
+    run: echo "exists=$([ -d "failure-logs" ] && echo "true" || echo "false")" >> "$GITHUB_OUTPUT"
+  - name: "Upload failure-logs"
+    if: steps.check-if-failure-logs-exists.outputs.exists == 'true'
+    uses: actions/upload-artifact@v4
+    with:
+      name: failure-logs
+      path: failure-logs/

.github/workflows/test.yml

@@ -194,6 +194,8 @@ jobs:
     # GHA macOS is slow and flaky, so we only test default.yaml here.
     # Other yamls are tested on Linux instances.
     #
+    - if: always()
+      uses: ./.github/actions/upload_failure_logs_if_exists
     - name: "Show cache"
       if: always()
       run: ./hack/debug-cache.sh
@@ -258,6 +260,8 @@ jobs:
         retry_on: error
         max_attempts: 3
         command: ./hack/test-templates.sh templates/${{ matrix.template }}
+    - if: always()
+      uses: ./.github/actions/upload_failure_logs_if_exists
     - name: "Show cache"
       run: ./hack/debug-cache.sh
 
@@ -357,6 +361,8 @@ jobs:
         retry_on: error
         max_attempts: 3
         command: ./hack/test-templates.sh templates/vmnet.yaml
+    - if: always()
+      uses: ./.github/actions/upload_failure_logs_if_exists
 
   upgrade:
     name: "Upgrade test"
@@ -425,3 +431,5 @@ jobs:
       env:
         LIMACTL_CREATE_ARGS: "--vm-type vz --mount-type virtiofs --rosetta --network vzNAT"
       run: ./hack/test-templates.sh templates/${{ matrix.template }}
+    - if: failure()
+      uses: ./.github/actions/upload_failure_logs_if_exists

hack/test-templates.sh

@@ -97,7 +97,9 @@ function diagnose() {
 	tail "$HOME/.lima/${NAME}"/*.log
 	limactl shell "$NAME" systemctl --no-pager status
 	limactl shell "$NAME" systemctl --no-pager
-	limactl shell "$NAME" sudo cat /var/log/cloud-init-output.log
+	mkdir -p failure-logs
+	cp -pf "$HOME/.lima/${NAME}"/*.log failure-logs/
+	limactl shell "$NAME" sudo cat /var/log/cloud-init-output.log | tee failure-logs/cloud-init-output.log
 	set +x -e
 }