[CI] Fix typo in GitHub context variable in documentation deploy pipe…

…line (#1194) * fix typo in github context variable * split between test and doc runners for Linux
aramis-lab · May 27, 2024 · ebe1b7a · ebe1b7a
1 parent de492f9
commit ebe1b7a
Show file tree

Hide file tree

Showing 9 changed files with 15 additions and 7 deletions.
diff --git a/.github/workflows/build_and_deploy.yml b/.github/workflows/build_and_deploy.yml
@@ -16,14 +16,13 @@ concurrency:
 env:
   POETRY_VERSION: '1.6.1'
   PYTHON_VERSION: '3.10'
-  BRANCH_NAME: "PR-${{ github.event.number }}"
 
 jobs:
   build-and-deploy-doc:
     runs-on:
       - self-hosted
       - Linux
-      - doc
+      - doc-runner
     steps:
       - uses: actions/checkout@v4
       - uses: snok/install-poetry@v1
@@ -38,14 +37,13 @@ jobs:
           make doc
       - name: Deploy documentation
         run: |
-          echo "${GITHUB_REF}"
-          echo "${PR_NUMBER}"
-          echo "${{ github.ref_name }}"
-          echo "${{ github.head_ref }}"
-          if [[ ! -z "${{ github.event_number }}" ]];
+          if [[ -z "${{ github.event.number }}" ]];
           then
             BRANCH_NAME="${{ github.ref_name }}"
+          else
+            BRANCH_NAME="PR-${{ github.event.number }}"
           fi
+          echo $BRANCH_NAME
           mv site "${BRANCH_NAME}"
           ls "${BRANCH_NAME}"
           scp -r "${BRANCH_NAME}" "aramislab:/srv/local/clinica/docs/public/${BRANCH_NAME}"

diff --git a/.github/workflows/test_converters.yml b/.github/workflows/test_converters.yml
@@ -51,6 +51,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     steps:
       - uses: actions/checkout@v4
       - uses: snok/install-poetry@v1

diff --git a/.github/workflows/test_instantiation.yml b/.github/workflows/test_instantiation.yml
@@ -49,6 +49,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     steps:
       - uses: actions/checkout@v4
       - uses: snok/install-poetry@v1

diff --git a/.github/workflows/test_non_regression_fast.yml b/.github/workflows/test_non_regression_fast.yml
@@ -53,6 +53,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     steps:
       - uses: actions/checkout@v4
       - uses: snok/install-poetry@v1

diff --git a/.github/workflows/test_pipelines_anat.yml b/.github/workflows/test_pipelines_anat.yml
@@ -67,6 +67,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 120
     steps:
       - uses: actions/checkout@v4
@@ -99,6 +100,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 720
     steps:
       - uses: actions/checkout@v4

diff --git a/.github/workflows/test_pipelines_anat_freesurfer.yml b/.github/workflows/test_pipelines_anat_freesurfer.yml
@@ -41,6 +41,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 1440
     steps:
       - uses: actions/checkout@v4

diff --git a/.github/workflows/test_pipelines_dwi.yml b/.github/workflows/test_pipelines_dwi.yml
@@ -43,6 +43,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 720
     steps:
       - uses: actions/checkout@v4

diff --git a/.github/workflows/test_pipelines_dwi_preprocessing.yml b/.github/workflows/test_pipelines_dwi_preprocessing.yml
@@ -66,6 +66,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 720
     steps:
       - uses: actions/checkout@v4
@@ -98,6 +99,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 720
     steps:
       - uses: actions/checkout@v4

diff --git a/.github/workflows/test_pipelines_pet.yml b/.github/workflows/test_pipelines_pet.yml
@@ -37,6 +37,7 @@ jobs:
     runs-on:
       - self-hosted
       - Linux
+      - test-runner
     timeout-minutes: 720
     steps:
       - uses: actions/checkout@v4