ci: fix the get log/destroy VM condition

- should use `steps.<step id>.conclusion == 'success'` Signed-off-by: Vicente Cheng <[email protected]>
harvester · Jul 19, 2024 · 5a30aa3 · 5a30aa3
1 parent 4e6e62f
commit 5a30aa3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git 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