ci: fix external pull requests (#33063)

.github/actions/build-docker/action.yml

@@ -17,13 +17,25 @@ inputs:
     required: false
     description: 'Containers to build along with Rocket.Chat'
     type: string
+  turbo-cache:
+    required: false
+    description: 'Enable turbo cache'
+    default: 'true'
+  publish-image:
+    required: false
+    description: 'Publish image'
+    default: 'true'
+  setup:
+    required: false
+    description: 'Setup node.js'
+    default: 'true'
 
 runs:
   using: composite
 
   steps:
     - name: Login to GitHub Container Registry
-      if: (github.event.pull_request.head.repo.full_name == github.repository || github.event_name == 'release' || github.ref == 'refs/heads/develop')
+      if: inputs.publish-image == 'true' &&(github.event.pull_request.head.repo.full_name == github.repository || github.event_name == 'release' || github.ref == 'refs/heads/develop')
       uses: docker/login-action@v2
       with:
         registry: ghcr.io
@@ -42,17 +54,20 @@ runs:
         cd /tmp/build
         tar xzf Rocket.Chat.tar.gz
         rm Rocket.Chat.tar.gz
-
     - uses: rharkor/caching-for-turbo@v1.5
+      # if we are testing a PR from a fork, we already called the turbo cache at this point, so it should be false
+      if: inputs.turbo-cache == 'true'
 
     - name: Setup NodeJS
       uses: ./.github/actions/setup-node
+      if: inputs.setup == 'true'
       with:
         node-version: ${{ inputs.node-version }}
         cache-modules: true
         install: true
 
     - run: yarn build
+      if: inputs.setup == 'true'
       shell: bash
 
     - name: Build Docker images
@@ -63,9 +78,14 @@ runs:
         docker compose -f docker-compose-ci.yml build "${args[@]}"
 
     - name: Publish Docker images to GitHub Container Registry
-      if: (github.event.pull_request.head.repo.full_name == github.repository || github.event_name == 'release' || github.ref == 'refs/heads/develop')
+      if: inputs.publish-image == 'true' && (github.event.pull_request.head.repo.full_name == github.repository || github.event_name == 'release' || github.ref == 'refs/heads/develop')
       shell: bash
       run: |
         args=(rocketchat ${{ inputs.build-containers }})
 
         docker compose -f docker-compose-ci.yml push "${args[@]}"
+
+    - name: Clean up temporary files
+      shell: bash
+      run: |
+        sudo rm -rf /tmp/bundle

.github/workflows/ci-test-e2e.yml

@@ -130,7 +130,9 @@ jobs:
           node-version: ${{ inputs.node-version }}
           cache-modules: true
           install: true
+      - uses: rharkor/caching-for-turbo@v1.5
 
+      - run: yarn build
       # if we are testing a PR from a fork, we need to build the docker image at this point
       - uses: ./.github/actions/build-docker
         if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository
@@ -138,8 +140,11 @@ jobs:
           CR_USER: ${{ secrets.CR_USER }}
           CR_PAT: ${{ secrets.CR_PAT }}
           node-version: ${{ inputs.node-version }}
-
-      - uses: rharkor/caching-for-turbo@v1.5
+          # we already called the turbo cache at this point, so it should be false
+          turbo-cache: false
+          # the same reason we need to rebuild the docker image at this point is the reason we dont want to publish it
+          publish-image: false
+          setup: false
 
       - name: Start httpbin container and wait for it to be ready
         if: inputs.type == 'api'
@@ -159,8 +164,6 @@ jobs:
             exit 1
           fi
 
-      - run: yarn build
-
       - name: Prepare code coverage directory
         if: inputs.release == 'ee'
         run: |
@@ -187,10 +190,6 @@ jobs:
         run: |
           docker compose -f docker-compose-ci.yml up -d
 
-      - name: Clean up temporary files
-        run: |
-          sudo rm -rf /tmp/bundle
-
       - name: Cache Playwright binaries
         if: inputs.type == 'ui'
         uses: actions/cache@v3