gha - move PR creation out of main workflow

This way we can set the correct permissions for the PR creation step

Author: cderv

.github/workflows/test-smokes.yml

@@ -38,7 +38,7 @@ on:
   schedule:
     - cron: "0 6 * * *"
 
-# this is require to cleanup julia cache
+# this is required to cleanup julia cache
 permissions:
   actions: write
   contents: read
@@ -289,35 +289,9 @@ jobs:
         working-directory: tests
         shell: pwsh
 
-      - name: Create Pull Request for new tests timing
-        if: matrix.time-test
-        id: cpr
-        uses: peter-evans/create-pull-request@v6
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          add-paths: tests/timing-for-ci.txt
-          commit-message: |
-            timing for new tests [auto PR]
-          branch: updates/timing-for-ci
-          delete-branch: true
-          assignees: cderv
-          reviewers: cderv
-          title: |
-            [CI] Update timing file
-          body: |
-            This PR was created automatically with new timing test file for our parallel smoke tests. 
-            Please review and merge to trigger a new build of the website.
-
-      - name: Check auto PR outputs
-        if: steps.cpr.outcome == 'success'
-        run: |
-          echo "Pull Request Number - ${{ steps.cpr.outputs.pull-request-number }}"
-          echo "Pull Request URL - ${{ steps.cpr.outputs.pull-request-url }}"
-          echo "Pull Request Action Performed - ${{ steps.cpr.outputs.pull-request-operation }}"
-
       - name: Upload test timing file
         uses: actions/upload-artifact@v4
-        if: matrix.time-test && ( failure() || cancelled())
+        if: matrix.time-test
         with:
           name: timed test file
           path: tests/timing-for-ci.txt

.github/workflows/update-test-timing.yml

@@ -6,11 +6,52 @@ on:
     - cron: "13 3 * * 1"
 
 name: Update timing test file
-
 jobs:
   run-timed-tests:
     name: Running Tests to produce timed file
     uses: ./.github/workflows/test-smokes.yml
     with:
       buckets: ""
       time-test: true
+
+  create-pr:
+    name: Create PR with updated timing file
+    needs: run-timed-tests
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      pull-requests: write
+    steps:
+      - name: Checkout Repo
+        uses: actions/checkout@v4
+
+      - name: Download timing file artifact
+        uses: actions/download-artifact@v4
+        with:
+          name: timed test file
+          path: tests/
+
+      - name: Create Pull Request for new tests timing
+        id: cpr
+        uses: peter-evans/create-pull-request@v7
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+          add-paths: tests/timing-for-ci.txt
+          commit-message: |
+            timing for new tests [auto PR]
+          branch: updates/timing-for-ci
+          delete-branch: true
+          assignees: cderv
+          reviewers: cderv
+          title: |
+            [CI] Update timing file
+          body: |
+            This PR was created automatically with new timing test file for our parallel smoke tests.
+            Please review and merge to trigger a new build of the website.
+
+      - name: Check auto PR outputs
+        if: steps.cpr.outcome == 'success'
+        run: |
+          echo "Pull Request Number - ${{ steps.cpr.outputs.pull-request-number }}"
+          echo "Pull Request URL - ${{ steps.cpr.outputs.pull-request-url }}"
+          echo "Pull Request Action Performed - ${{ steps.cpr.outputs.pull-request-operation }}"