Skip to content

Commit bb724bf

Browse files
committed
test.yml: use upload_failure_logs_if_exists to "Upgrade test" job
Signed-off-by: Norio Nomura <[email protected]>
1 parent 6366227 commit bb724bf

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -394,6 +394,8 @@ jobs:
394394
retry_on: error
395395
max_attempts: 3
396396
command: ./hack/test-upgrade.sh ${{ matrix.oldver }} ${{ github.sha }}
397+
- if: always()
398+
uses: ./.github/actions/upload_failure_logs_if_exists
397399

398400
vz:
399401
name: "vz"

hack/test-upgrade.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,9 @@ function uninstall_lima() {
4949

5050
function show_lima_log() {
5151
tail -n 100 ~/.lima/"${LIMA_INSTANCE}"/*.log || true
52+
mkdir -p failure-logs
53+
cp -pf ~/.lima/"${LIMA_INSTANCE}"/*.log failure-logs/ || true
54+
limactl shell "${LIMA_INSTANCE}" sudo cat /var/log/cloud-init-output.log | tee failure-logs/cloud-init-output.log || true
5255
}
5356

5457
INFO "Uninstalling lima"
@@ -89,7 +92,7 @@ INFO "Installing the new Lima ${NEWVER}"
8992
install_lima "${NEWVER}"
9093

9194
INFO "Restarting the instance"
92-
limactl start --tty=false "${LIMA_INSTANCE}"
95+
limactl start --tty=false "${LIMA_INSTANCE}" || show_lima_log
9396
lima nerdctl info
9497

9598
INFO "Confirming that the host filesystem is still mounted"

0 commit comments

Comments
 (0)