From a21daa2670ca191743a3c95fc24892070abd3e4c Mon Sep 17 00:00:00 2001 From: Charles Bochet Date: Sat, 20 Sep 2025 12:14:29 +0200 Subject: [PATCH] Optimize CI runner cost (#14628) --- .github/workflows/ci-breaking-changes.yaml | 5 +---- .github/workflows/ci-emails.yaml | 2 +- .github/workflows/ci-front.yaml | 6 +++--- .github/workflows/ci-server.yaml | 2 +- packages/twenty-front/src/modules/auth/hooks/useAuth.ts | 2 +- .../messaging-relaunch-failed-message-channels.command.ts | 2 +- 6 files changed, 8 insertions(+), 11 deletions(-) diff --git a/.github/workflows/ci-breaking-changes.yaml b/.github/workflows/ci-breaking-changes.yaml index 18deea454ac..fc6d4246eba 100644 --- a/.github/workflows/ci-breaking-changes.yaml +++ b/.github/workflows/ci-breaking-changes.yaml @@ -6,9 +6,6 @@ on: branches: - main -permissions: - contents: read - concurrency: group: ${{ github.workflow }}-${{ github.ref }} cancel-in-progress: true @@ -37,7 +34,7 @@ jobs: needs: changed-files-check if: needs.changed-files-check.outputs.any_changed == 'true' timeout-minutes: 45 - runs-on: depot-ubuntu-24.04-8 + runs-on: shipfox-8vcpu-ubuntu-2404 services: postgres: image: twentycrm/twenty-postgres-spilo diff --git a/.github/workflows/ci-emails.yaml b/.github/workflows/ci-emails.yaml index 13c043b7693..111e5d8cb6d 100644 --- a/.github/workflows/ci-emails.yaml +++ b/.github/workflows/ci-emails.yaml @@ -52,7 +52,7 @@ jobs: ci-emails-status-check: if: always() && !cancelled() timeout-minutes: 5 - runs-on: depot-ubuntu-24.04-8 + runs-on: ubuntu-latest needs: [changed-files-check, emails-test] steps: - name: Fail job if any needs failed diff --git a/.github/workflows/ci-front.yaml b/.github/workflows/ci-front.yaml index 18b199a4fbb..3651c7e0675 100644 --- a/.github/workflows/ci-front.yaml +++ b/.github/workflows/ci-front.yaml @@ -93,7 +93,7 @@ jobs: path: packages/twenty-front/coverage/storybook/coverage-shard-${{matrix.shard}}.json merge-reports-and-check-coverage: timeout-minutes: 30 - runs-on: depot-ubuntu-24.04-8 + runs-on: ubuntu-latest needs: front-sb-test env: PATH_TO_COVERAGE: packages/twenty-front/coverage/storybook @@ -146,7 +146,7 @@ jobs: needs: changed-files-check if: needs.changed-files-check.outputs.any_changed == 'true' timeout-minutes: 30 - runs-on: depot-ubuntu-24.04-8 + runs-on: ubuntu-latest env: TASK_CACHE_KEY: front-task-${{ matrix.task }} strategy: @@ -185,7 +185,7 @@ jobs: ci-front-status-check: if: always() && !cancelled() timeout-minutes: 5 - runs-on: depot-ubuntu-24.04-8 + runs-on: ubuntu-latest needs: [ changed-files-check, diff --git a/.github/workflows/ci-server.yaml b/.github/workflows/ci-server.yaml index 0185088f6de..061e3823972 100644 --- a/.github/workflows/ci-server.yaml +++ b/.github/workflows/ci-server.yaml @@ -249,7 +249,7 @@ jobs: ci-server-status-check: if: always() && !cancelled() timeout-minutes: 5 - runs-on: depot-ubuntu-24.04-8 + runs-on: ubuntu-latest needs: [changed-files-check, server-setup, server-test, server-integration-test] steps: - name: Fail job if any needs failed diff --git a/packages/twenty-front/src/modules/auth/hooks/useAuth.ts b/packages/twenty-front/src/modules/auth/hooks/useAuth.ts index 7ddc302a632..53631b734a6 100644 --- a/packages/twenty-front/src/modules/auth/hooks/useAuth.ts +++ b/packages/twenty-front/src/modules/auth/hooks/useAuth.ts @@ -313,7 +313,7 @@ export const useAuth = () => { handleSetAuthTokens(authTokens); // TODO: We can't parallelize this yet because when loadCurrentUSer is loaded - // then UserProvider updates its children and PrefetchDataProvider is triggered + // then UserProvider updates its children and PrefetchDataProvider is then triggered // which requires the correct metadata to be loaded (not the mocks) await loadCurrentUser(); await refreshObjectMetadataItems(); diff --git a/packages/twenty-server/src/modules/messaging/message-import-manager/commands/messaging-relaunch-failed-message-channels.command.ts b/packages/twenty-server/src/modules/messaging/message-import-manager/commands/messaging-relaunch-failed-message-channels.command.ts index 720db6048a5..b58500a5c54 100644 --- a/packages/twenty-server/src/modules/messaging/message-import-manager/commands/messaging-relaunch-failed-message-channels.command.ts +++ b/packages/twenty-server/src/modules/messaging/message-import-manager/commands/messaging-relaunch-failed-message-channels.command.ts @@ -72,7 +72,7 @@ export class MessagingRelaunchFailedMessageChannelsCommand extends ActiveOrSuspe } this.logger.log( - `${options.dryRun ? ' (DRY RUN): ' : ''}Relaunched ${failedMessageChannels.length} failed message channels`, + `${options.dryRun ? ' (DRY RUN) ' : ''}Relaunched ${failedMessageChannels.length} failed message channels`, ); } catch (error) { this.logger.error(