Skip to content

Commit

Permalink
chore: ci limits on building finsemble, and clean up
Browse files Browse the repository at this point in the history
  • Loading branch information
algreasley committed Nov 27, 2024
1 parent 4c87d94 commit d5528e9
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 84 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,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 }}
Expand All @@ -98,6 +99,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" \
Expand Down
14 changes: 0 additions & 14 deletions .github/workflows/e2e-full.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
42 changes: 0 additions & 42 deletions .github/workflows/pull-fork.yml

This file was deleted.

26 changes: 5 additions & 21 deletions .github/workflows/pull.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down Expand Up @@ -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
Expand All @@ -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
Expand Down Expand Up @@ -156,14 +140,14 @@ jobs:
uses: actions/upload-artifact@v4
if: failure()
with:
name: playwright-report
name: playwright-report-web
path: packages/client/playwright-report/
retention-days: 7

openfin-end-to-end-test-fx:
name: Openfin e2e test - FX

needs: web-end-to-end-test
needs: build

runs-on: windows-latest

Expand Down Expand Up @@ -191,14 +175,14 @@ 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

openfin-end-to-end-test-credit:
name: Openfin e2e test - Credit

needs: web-end-to-end-test
needs: build

runs-on: windows-latest

Expand Down Expand Up @@ -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
7 changes: 0 additions & 7 deletions .github/workflows/tag.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit d5528e9

Please sign in to comment.