Changing CI conditional to use success() instead

Signed-off-by: Julio Faracco <[email protected]>
discovery-unicamp · Jun 28, 2024 · ce888c8 · ce888c8
1 parent 801bdb4
commit ce888c8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -92,10 +92,10 @@ jobs:
         run: |
           ssh -T ${{ secrets.CLUSTER_USER }}@${{ secrets.CLUSTER_ADDRESS }} -p 8686 > /dev/null
       - name: Pull latest docker image
-        if: !failure() && steps.test_connection.outcome != 'failure'
+        if: success() || steps.test_connection.conclusion == 'success'
         run: |
           ssh ${{ secrets.CLUSTER_USER }}@${{ secrets.CLUSTER_ADDRESS }} -p 8686 "ssh ${{ secrets.CLUSTER_GPU_ID }} \"docker pull jcfaracco/dasf:gpu\""
       - name: Run test cases remotely
-        if: !failure() && steps.test_connection.outcome != 'failure'
+        if: success() || steps.test_connection.conclusion == 'success'
         run: |
           ssh ${{ secrets.CLUSTER_USER }}@${{ secrets.CLUSTER_ADDRESS }} -p 8686 "ssh ${{ secrets.CLUSTER_GPU_ID }} \"docker run --rm --gpus all jcfaracco/dasf:gpu sh -c 'pip3 install pytest parameterized pytest-cov pip --upgrade && rm -rf dasf-core && git clone https://github.com/discovery-unicamp/dasf-core.git && cd dasf-core/ && pip3 uninstall dasf -y && pip3 install . && pytest --cov dasf/ tests/ && cd -'\""