From ebd03d604b9cce1d54e6aed54e31354d2eb8d891 Mon Sep 17 00:00:00 2001 From: Paul Hinze Date: Fri, 13 May 2022 16:23:52 -0500 Subject: [PATCH] ci: Reorder spectesting to fix success notify Turns out you can't run a script in the workdir after you `rm -rf` the workspace! --- .github/workflows/spectesting.yml | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/.github/workflows/spectesting.yml b/.github/workflows/spectesting.yml index 6f8389bed..5c0b81bb7 100644 --- a/.github/workflows/spectesting.yml +++ b/.github/workflows/spectesting.yml @@ -84,6 +84,15 @@ jobs: name: vagrant-spec-${{matrix.providers}}.log path: ${{ github.workspace }}/vagrant-spec.log + notify-on-success: + runs-on: self-hosted + name: Notify on Success + needs: spec-tests + if: success() + steps: + - name: Notify on Success + run: ./.ci/spec/notify-success.sh + cleanup: runs-on: self-hosted name: Cleanup Post Vagrant-Spec Tests @@ -95,11 +104,3 @@ jobs: - name: Clean Workspace run: rm -rf ${{ github.workspace }} - notify-on-success: - runs-on: self-hosted - name: Notify on Success - needs: spec-tests - if: success() - steps: - - name: Notify on Success - run: ./.ci/spec/notify-success.sh