From 5a30aa39ca45065b464287e133dfaf17af3b01f0 Mon Sep 17 00:00:00 2001 From: Vicente Cheng Date: Thu, 11 Jul 2024 15:46:12 +0800 Subject: [PATCH] ci: fix the get log/destroy VM condition - should use `steps..conclusion == 'success'` Signed-off-by: Vicente Cheng --- .github/workflows/basic-ci.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/basic-ci.yaml b/.github/workflows/basic-ci.yaml index 304d5cf9..a1b89364 100644 --- a/.github/workflows/basic-ci.yaml +++ b/.github/workflows/basic-ci.yaml @@ -77,11 +77,11 @@ jobs: echo Running integration tests NDM_HOME=`pwd` go test -v ./tests/... - name: "Get NDM logs" - if: steps.vm_deploy.outcome == 'success' + if: steps.vm_deploy.conclusion == 'success' run: | ./ci/scripts/get-debug-info.sh - name: "Tear Down / Cleanup" - if: steps.vm_deploy.outcome == 'success' + if: steps.vm_deploy.conclusion == 'success' run: | pushd ndm-vagrant-rancherd vagrant destroy -f --parallel