chore: ci limits on building finsemble, and clean up

AdaptiveConsulting · Nov 27, 2024 · d327729 · d327729
1 parent 4c87d94
commit d327729
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 82 deletions.
diff --git a/.github/workflows/branch.yml b/.github/workflows/branch.yml
@@ -51,9 +51,24 @@ jobs:
         with:
           project_id: ${{ secrets.GOOGLE_PROJECT_ID }}
 
+      - name: Restore cache
+        uses: actions/cache/restore@v4
+        id: cache-npm
+        with:
+          path: packages/client/node_modules
+          key: ${{ runner.OS }}-node-${{ hashFiles('packages/client/package-lock.json') }}
+
       - name: Install dependencies
+        if: steps.cache-npm.outputs.cache-hit != 'true'
         run: npm ci
 
+      - name: Cache node modules
+        if: steps.cache-npm.outputs.cache-hit != 'true'
+        uses: actions/cache/save@v4
+        with:
+          path: packages/client/node_modules
+          key: ${{ runner.OS }}-node-${{ hashFiles('packages/client/package-lock.json') }}
+
       - name: Verify Quality (type checking, linting, format checking, tests)
         run: npm run verify
 
@@ -90,6 +105,7 @@ jobs:
             gs://$BUCKET_OPENFIN/branch/${{ steps.vars.outputs.BRANCH }}
 
       - name: Build (Finsemble)
+        if: ${{ steps.vars.outputs.ENVIRONMENT == 'dev' || steps.vars.outputs.ENVIRONMENT == 'uat' }}
         env:
           DOMAIN: https://finsemble.${{ steps.vars.outputs.ENVIRONMENT }}.reactivetrader.com
           URL_PATH: ${{ steps.vars.outputs.URL_SUFFIX }}
@@ -98,6 +114,7 @@ jobs:
         run: npm run finsemble:build
 
       - name: Deploy (Finsemble)
+        if: ${{ steps.vars.outputs.ENVIRONMENT == 'dev' || steps.vars.outputs.ENVIRONMENT == 'uat' }}
         run: |
           gsutil -m \
             -h "Cache-Control: no-cache" \

diff --git a/.github/workflows/e2e-full.yml b/.github/workflows/e2e-full.yml
@@ -55,17 +55,3 @@ jobs:
           name: playwright-report-uat
           path: packages/client/playwright-report/
           retention-days: 7
-
-      - name: Test Prod
-        env:
-          E2E_RTC_WEB_ROOT_URL: https://web.prod.reactivetrader.com
-        run: npx playwright test --project=web credit.spec.ts
-        continue-on-error: true
-
-      - name: Artifact Upload
-        uses: actions/upload-artifact@v4
-        if: failure()
-        with:
-          name: playwright-report-prod
-          path: packages/client/playwright-report/
-          retention-days: 7
diff --git a/.github/workflows/pull-fork.yml b/.github/workflows/pull-fork.yml
diff --git a/.github/workflows/pull.yml b/.github/workflows/pull.yml
@@ -15,7 +15,6 @@ defaults:
 env:
   BUCKET_WEB: reactive-trader-web-builds
   BUCKET_OPENFIN: reactive-trader-openfin-builds
-  BUCKET_FINSEMBLE: reactive-trader-finsemble-builds
   ENVIRONMENT: env
 
 jobs:
@@ -80,20 +79,6 @@ jobs:
             rsync -d -r dist \
             gs://$BUCKET_OPENFIN/pull/${{ github.event.number }}
 
-      - name: Build (Finsemble)
-        env:
-          DOMAIN: https://finsemble.env.reactivetrader.com
-          URL_PATH: /pull/${{ github.event.number }}
-          VITE_BUILD_VERSION: ${{ github.sha }}
-        run: npm run finsemble:build
-
-      - name: Deploy (Finsemble)
-        run: |
-          gsutil -m \
-            -h "Cache-Control: no-cache" \
-            rsync -d -r dist \
-            gs://$BUCKET_FINSEMBLE/pull/${{ github.event.number }}
-
       - name: Find comment
         uses: peter-evans/find-comment@v3
         id: comment
@@ -120,7 +105,6 @@ jobs:
             | OpenFin - Credit    | fins://openfin.env.reactivetrader.com/pull/${{ github.event.number }}/config/rt-credit.json            |
             | OpenFin - Launcher  | fins://openfin.env.reactivetrader.com/pull/${{ github.event.number }}/config/launcher.json             |
             | OpenFin - Workspace | fins://openfin.env.reactivetrader.com/pull/${{ github.event.number }}/workspace/config/workspace.json  |
-            | Finsemble           | https://finsemble.env.reactivetrader.com/pull/${{ github.event.number }}                               |
 
             ### Performance
 
@@ -156,7 +140,7 @@ jobs:
         uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: playwright-report
+          name: playwright-report-web
           path: packages/client/playwright-report/
           retention-days: 7
 
@@ -191,7 +175,7 @@ jobs:
         uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: playwright-report
+          name: playwright-report-of-fx
           path: packages/client/playwright-report/
           retention-days: 7
 
@@ -226,6 +210,6 @@ jobs:
         uses: actions/upload-artifact@v4
         if: failure()
         with:
-          name: playwright-report
+          name: playwright-report-of-credit
           path: packages/client/playwright-report/
           retention-days: 7
diff --git a/.github/workflows/tag.yml b/.github/workflows/tag.yml
@@ -83,13 +83,6 @@ jobs:
             rsync -d -r dist \
             gs://$BUCKET_OPENFIN/tag/${{ steps.vars.outputs.TAG }}
 
-      - name: Deploy (Launcher)
-        run: |
-          gsutil -m \
-            -h "Cache-Control: no-cache" \
-            rsync -d -r dist \
-            gs://$BUCKET_LAUNCHER/tag/${{ steps.vars.outputs.TAG }}
-
       - name: Build (Finsemble)
         env:
           DOMAIN: https://finsemble.prod.reactivetrader.com