From e942ff77f1f750f1e1ebd1648ab0d353579ded84 Mon Sep 17 00:00:00 2001 From: Serhii Mamontov Date: Thu, 23 May 2024 19:34:59 +0300 Subject: [PATCH 1/2] build(runner): use large GitHub runner --- .github/workflows/commands-handler.yml | 14 ++++++++------ .github/workflows/release.yml | 19 +++++++++++-------- .github/workflows/run-tests.yml | 22 ++++++++++++++-------- .github/workflows/run-validations.yml | 12 ++++++++---- 4 files changed, 41 insertions(+), 26 deletions(-) diff --git a/.github/workflows/commands-handler.yml b/.github/workflows/commands-handler.yml index 4e34b04c..e2262dd2 100644 --- a/.github/workflows/commands-handler.yml +++ b/.github/workflows/commands-handler.yml @@ -11,11 +11,13 @@ jobs: process: name: Process command if: github.event.issue.pull_request && endsWith(github.repository, '-private') != true - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m steps: - name: Check referred user id: user-check - env: + env: CLEN_BOT: ${{ secrets.CLEN_BOT }} run: echo "expected-user=${{ startsWith(github.event.comment.body, format('@{0} ', env.CLEN_BOT)) }}" >> $GITHUB_OUTPUT - name: Regular comment @@ -23,12 +25,12 @@ jobs: run: echo -e "\033[38;2;19;181;255mThis is regular commit which should be ignored.\033[0m" - name: Checkout repository if: steps.user-check.outputs.expected-user == 'true' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: - token: ${{ secrets.GH_TOKEN }} + token: ${{ secrets.GH_TOKEN }} - name: Checkout release actions if: steps.user-check.outputs.expected-user == 'true' - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -40,4 +42,4 @@ jobs: with: token: ${{ secrets.GH_TOKEN }} listener: ${{ secrets.CLEN_BOT }} - jira-api-key: ${{ secrets.JIRA_API_KEY }} \ No newline at end of file + jira-api-key: ${{ secrets.JIRA_API_KEY }} diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 58bff490..7c4d3960 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -2,19 +2,21 @@ name: Automated product release on: pull_request: - branches: [ master ] - types: [ closed ] + branches: [master] + types: [closed] jobs: check-release: name: Check release required - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m if: github.event.pull_request.merged && endsWith(github.repository, '-private') != true outputs: release: ${{ steps.check.outputs.ready }} steps: - name: Checkout actions - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -27,17 +29,19 @@ jobs: token: ${{ secrets.GH_TOKEN }} publish: name: Publish package - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m needs: check-release if: needs.check-release.outputs.release == 'true' steps: - name: Checkout repository - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: # This should be the same as the one specified for on.pull_request.branches ref: master - name: Checkout actions - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -70,4 +74,3 @@ jobs: with: token: ${{ secrets.GH_TOKEN }} acceptance-tests-workflow: Tests - diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index de461b9c..fb782fc9 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -15,18 +15,20 @@ defaults: jobs: tests: name: Integration and Unit tests - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m strategy: fail-fast: true matrix: pkg: [pubnub] steps: - name: Checkout repository - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: token: ${{ secrets.GH_TOKEN }} - name: Checkout actions - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -62,12 +64,14 @@ jobs: uses: ./.github/.release/actions/actions/utils/fast-jobs-failure acceptance-tests: name: Acceptance tests - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m steps: - name: Checkout project - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Checkout mock-server action - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -80,7 +84,7 @@ jobs: - name: Setup Dart uses: dart-lang/setup-dart@v1.3 - name: Cache - uses: actions/cache@v3 + uses: actions/cache@v4 with: path: ~/.pub-cache key: ${{ runner.os }}-${{ matrix.pkg }}-${{ hashFiles('**/pubspec.yaml') }} @@ -102,7 +106,9 @@ jobs: uses: ./.github/.release/actions/actions/utils/fast-jobs-failure all-tests: name: Tests - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m needs: [tests, acceptance-tests] steps: - name: Tests summary diff --git a/.github/workflows/run-validations.yml b/.github/workflows/run-validations.yml index c2224b5e..a157f35f 100644 --- a/.github/workflows/run-validations.yml +++ b/.github/workflows/run-validations.yml @@ -13,12 +13,14 @@ defaults: jobs: pubnub-yml: name: "Validate .pubnub.yml" - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m steps: - name: Checkout project - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Checkout validator action - uses: actions/checkout@v3 + uses: actions/checkout@v4 with: repository: pubnub/client-engineering-deployment-tools ref: v1 @@ -33,7 +35,9 @@ jobs: uses: ./.github/.release/actions/actions/utils/fast-jobs-failure all-validations: name: Validations - runs-on: ubuntu-latest + runs-on: + group: Default Larger Runners + labels: ubuntu-latest-m needs: [pubnub-yml] steps: - name: Validations summary From d3319f4d712073efa003e6e0b98caf05c0bfc81f Mon Sep 17 00:00:00 2001 From: Serhii Mamontov Date: Wed, 29 May 2024 01:25:01 +0300 Subject: [PATCH 2/2] build(workflow): change runner groups --- .github/workflows/commands-handler.yml | 3 +-- .github/workflows/release.yml | 10 ++++------ .github/workflows/run-tests.yml | 11 ++++------- .github/workflows/run-validations.yml | 8 +++----- 4 files changed, 12 insertions(+), 20 deletions(-) diff --git a/.github/workflows/commands-handler.yml b/.github/workflows/commands-handler.yml index e2262dd2..48f71d24 100644 --- a/.github/workflows/commands-handler.yml +++ b/.github/workflows/commands-handler.yml @@ -12,8 +12,7 @@ jobs: name: Process command if: github.event.issue.pull_request && endsWith(github.repository, '-private') != true runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m + group: Default steps: - name: Check referred user id: user-check diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 7c4d3960..557a819b 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -8,10 +8,9 @@ on: jobs: check-release: name: Check release required - runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m if: github.event.pull_request.merged && endsWith(github.repository, '-private') != true + runs-on: + group: Default outputs: release: ${{ steps.check.outputs.ready }} steps: @@ -29,11 +28,10 @@ jobs: token: ${{ secrets.GH_TOKEN }} publish: name: Publish package - runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m needs: check-release if: needs.check-release.outputs.release == 'true' + runs-on: + group: Default steps: - name: Checkout repository uses: actions/checkout@v4 diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml index fb782fc9..ccd41fc8 100644 --- a/.github/workflows/run-tests.yml +++ b/.github/workflows/run-tests.yml @@ -16,8 +16,7 @@ jobs: tests: name: Integration and Unit tests runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m + group: Default strategy: fail-fast: true matrix: @@ -65,8 +64,7 @@ jobs: acceptance-tests: name: Acceptance tests runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m + group: Default steps: - name: Checkout project uses: actions/checkout@v4 @@ -106,10 +104,9 @@ jobs: uses: ./.github/.release/actions/actions/utils/fast-jobs-failure all-tests: name: Tests - runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m needs: [tests, acceptance-tests] + runs-on: + group: Default steps: - name: Tests summary run: echo -e "\033[38;2;95;215;0m\033[1mAll tests successfully passed" diff --git a/.github/workflows/run-validations.yml b/.github/workflows/run-validations.yml index a157f35f..d3e4e6f3 100644 --- a/.github/workflows/run-validations.yml +++ b/.github/workflows/run-validations.yml @@ -14,8 +14,7 @@ jobs: pubnub-yml: name: "Validate .pubnub.yml" runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m + group: Default steps: - name: Checkout project uses: actions/checkout@v4 @@ -35,10 +34,9 @@ jobs: uses: ./.github/.release/actions/actions/utils/fast-jobs-failure all-validations: name: Validations - runs-on: - group: Default Larger Runners - labels: ubuntu-latest-m needs: [pubnub-yml] + runs-on: + group: Default steps: - name: Validations summary run: echo -e "\033[38;2;95;215;0m\033[1mAll validations passed"