From ce888c82c586e5b78433923fa203fcc9ead8b7c8 Mon Sep 17 00:00:00 2001 From: Julio Faracco Date: Fri, 28 Jun 2024 14:40:44 -0300 Subject: [PATCH] Changing CI conditional to use success() instead Signed-off-by: Julio Faracco --- .github/workflows/ci.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 7f8656c..78e47cc 100644 --- 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 -'\""