[CI] Move code-coverage upload into isolated process (#19532)

.github/workflows/ci-cd.yml

@@ -188,12 +188,14 @@ jobs:
         if: failure()
         with:
           name: rspec-artifacts
-          path: |
+          path: tmp/capybara
-            tmp/capybara
+      - name: Rename folder
-      - uses: codecov/codecov-action@v3
+        run: mv coverage/simplecov coverage/simplecov-${{ matrix.ci_node_index }}
+      - name: Upload test coverage result
+        uses: actions/upload-artifact@v3
         with:
-          flags: ruby
+          name: coverage
-          fail_ci_if_error: false
+          path: coverage/
       - name: Upload test results to BuildPulse for flaky test detection
         if: '!cancelled()' # Run this step even when the tests fail. Skip if the workflow is cancelled.
         uses: Workshop64/buildpulse-action@master
@@ -231,10 +233,11 @@ jobs:
           path: junit.xml
           key: ${{ secrets.BUILDPULSE_ACCESS_KEY_ID }}
           secret: ${{ secrets.BUILDPULSE_SECRET_ACCESS_KEY }}
-      - uses: codecov/codecov-action@v3
+      - name: Upload test coverage result
+        uses: actions/upload-artifact@v3
         with:
-          flags: jest, javascript
+          name: coverage
-          fail_ci_if_error: false
+          path: coverage/
 
   storybook:
     runs-on: ubuntu-latest
@@ -383,10 +386,15 @@ jobs:
           path: |
             tmp/cypress_screenshots
             cypress/logs
-      - uses: codecov/codecov-action@v3
+      - name: rename folder
+        run: |
+          rm -rf coverage/cypress/lcov-report
+          mv coverage/cypress coverage/cypress-${{ matrix.ci_node_index }}
+      - name: Upload test coverage result
+        uses: actions/upload-artifact@v3
         with:
-          flags: cypress, javascript
+          name: coverage
-          fail_ci_if_error: false
+          path: coverage/
       - name: Upload test results to BuildPulse for flaky test detection
         if: '!cancelled()' # Run this step even when the tests fail. Skip if the workflow is cancelled.
         uses: Workshop64/buildpulse-action@master
@@ -397,6 +405,34 @@ jobs:
           key: ${{ secrets.BUILDPULSE_ACCESS_KEY_ID }}
           secret: ${{ secrets.BUILDPULSE_SECRET_ACCESS_KEY }}
 
+  upload-coverage:
+    runs-on: ubuntu-latest
+    needs: [rspec, jest, cypress]
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/download-artifact@v3
+        with:
+          name: coverage
+      - run: |
+          mkdir coverage coverage/simplecov coverage/cypress
+          mv simplecov-* coverage/simplecov
+          mv cypress-* coverage/cypress
+          mv jest coverage/jest
+      - uses: codecov/codecov-action@v3
+        with:
+          flags: ruby
+          directory: coverage/simplecov
+          fail_ci_if_error: true
+      - uses: codecov/codecov-action@v3
+        with:
+          flags: jest, javascript
+          directory: coverage/jest
+          fail_ci_if_error: true
+      - uses: codecov/codecov-action@v3
+        with:
+          flags: cypress, javascript
+          directory: coverage/cypress
+          fail_ci_if_error: true
 
   CI-status-report:
     runs-on: ubuntu-latest

.simplecov

@@ -1,4 +1,5 @@
 unless ENV["COVERAGE"] == "false"
+  SimpleCov.coverage_dir("coverage/simplecov")
   SimpleCov.start "rails" do
     add_filter "/spec/"
     add_filter "/app/controllers/admin/"

codecov.yml

@@ -7,7 +7,6 @@ codecov:
   require_ci_to_pass: no
   notify:
     wait_for_ci: no
-    after_n_builds: 12 # Actually we have 18 total but this ensure we get a (incomplete) result even if some failed to upload
 ignore:
   - "vendor"
   - "bin"
@@ -23,4 +22,4 @@ coverage:
     project: false
 flag_management:
   default_rules:
-    carryforward: true
+    carryforward: false