From f1f44661bb6e78fdc6301a16d40fcb84eee1b6b5 Mon Sep 17 00:00:00 2001 From: Florian Sihler Date: Tue, 19 Sep 2023 08:26:16 +0200 Subject: [PATCH 1/4] ci-fix: use release token so that the qa can publish the benchmark results --- .github/workflows/qa.yaml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/qa.yaml b/.github/workflows/qa.yaml index 168e32be37..b3bcc36ea7 100644 --- a/.github/workflows/qa.yaml +++ b/.github/workflows/qa.yaml @@ -86,6 +86,8 @@ jobs: steps: - name: "Checkout Repository" uses: actions/checkout@v4 + with: + token: ${{ secrets.RELEASE_TOKEN }} - name: "⬇️ Get benchmark Artifacts" uses: actions/download-artifact@v3 @@ -125,7 +127,7 @@ jobs: uses: ad-m/github-push-action@master with: branch: main - github_token: ${{ secrets.GITHUB_TOKEN }} + github_token: ${{ secrets.RELEASE_TOKEN }} force: true full-test: From 9feeb4e30e50b9af661783856d9cde9e429286a9 Mon Sep 17 00:00:00 2001 From: Florian Sihler Date: Tue, 19 Sep 2023 16:33:05 +0200 Subject: [PATCH 2/4] ci-fix: pull before re-push --- .github/workflows/qa.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/qa.yaml b/.github/workflows/qa.yaml index b3bcc36ea7..0066ce089c 100644 --- a/.github/workflows/qa.yaml +++ b/.github/workflows/qa.yaml @@ -95,8 +95,9 @@ jobs: name: benchmark-results path: benchmark/ - - name: Name and Email for Git (config) + - name: Name and Email for Git (config) and update run: | + git pull git config --local user.email "action@github.com" git config --local user.name "GitHub Action" From 1ab71a4565ae53b3f8d6e60b8765c4e5241f325d Mon Sep 17 00:00:00 2001 From: Florian Sihler Date: Tue, 19 Sep 2023 16:33:53 +0200 Subject: [PATCH 3/4] ci-fix: restrict upload runs to 1 at a time --- .github/workflows/qa.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/qa.yaml b/.github/workflows/qa.yaml index 0066ce089c..2bd4d37e3e 100644 --- a/.github/workflows/qa.yaml +++ b/.github/workflows/qa.yaml @@ -79,6 +79,7 @@ jobs: runs-on: ubuntu-latest needs: [ performance-test ] strategy: + max-parallel: 1 fail-fast: true matrix: # currently we have a duplication, add your benchmark names here and in `performance-test` From 24c8aae78443ca73ca8f14f0c0d92c5779891021 Mon Sep 17 00:00:00 2001 From: Florian Sihler Date: Tue, 19 Sep 2023 16:34:12 +0200 Subject: [PATCH 4/4] ci-fix: remove iniital pull again, as the limited parallel run of uploads should be everything --- .github/workflows/qa.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/qa.yaml b/.github/workflows/qa.yaml index 2bd4d37e3e..9ce86c7f11 100644 --- a/.github/workflows/qa.yaml +++ b/.github/workflows/qa.yaml @@ -98,7 +98,6 @@ jobs: - name: Name and Email for Git (config) and update run: | - git pull git config --local user.email "action@github.com" git config --local user.name "GitHub Action"