Skip to content

Commit

Permalink
chore: Modernize Performance App (use Vite) (#9707)
Browse files Browse the repository at this point in the history
* stash

* stash

* cleanup lockfile

* more updates to harness

* more work on scenarios

* add bun log

* simplify perf test build

* another pass through

* turns out we want compat

* make easier to debug

* turn on verbose logging

* try

* use string :eyeroll:

* exclude more logs

* exclude more

* add debug mode for chrome

* give longer timeout for logging

* lets inspect

* fix server

* update check, reduce logging

* fckn finly
  • Loading branch information
runspired authored Mar 5, 2025
1 parent 2c44f0b commit d272cdc
Show file tree
Hide file tree
Showing 25 changed files with 1,020 additions and 232 deletions.
17 changes: 14 additions & 3 deletions .github/workflows/perf-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,13 @@ jobs:
run: |
BROWSER_FLAGS=$(node ./scripts/perf-tracking/browser-flags.mjs)
echo "BROWSER_FLAGS=$BROWSER_FLAGS" >> $GITHUB_OUTPUT
#env:
# DEBUG: true
- uses: tracerbench/tracerbench-compare-action@6b56fb774f78e4a85cf02396412b0164870cdab3
with:
experiment-build-command: pnpm install && pnpm --filter performance-test-app exec ember build -e production --output-path dist-experiment --suppress-sizes
experiment-serve-command: pnpm --filter performance-test-app exec ember s --path dist-experiment --port 4201
control-build-command: pnpm install && pnpm --filter performance-test-app exec ember build -e production --output-path dist-control --suppress-sizes
experiment-build-command: pnpm --filter performance-test-app build --outDir dist-experiment
experiment-serve-command: pnpm --filter performance-test-app start dist-experiment -p 4201
control-build-command: pnpm --filter performance-test-app exec ember build -e production --output-path dist-control --suppress-sizes
control-serve-command: pnpm --filter performance-test-app exec ember s --path dist-control
control-sha: origin/main
sample-timeout: 60
Expand Down Expand Up @@ -114,6 +116,15 @@ jobs:
fidelity: 60
upload-traces: true
upload-results: true
# env:
# DEBUG: '*,-babel*,-vite*,-rollup*,-ember*,-broccoli*,-pnpm*,-embroider*,-tree-sync*,-fs-tree-diff*'
# - name: Upload Assets
# if: failure() || success()
# uses: actions/upload-artifact@v4
# with:
# name: built-files
# path: 'tests/performance/dist-*'
# retention-days: 1
- name: Report TracerBench Results
if: failure() || success()
env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/perf-over-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
echo "BROWSER_FLAGS=$BROWSER_FLAGS" >> $GITHUB_OUTPUT
- uses: tracerbench/tracerbench-compare-action@6b56fb774f78e4a85cf02396412b0164870cdab3
with:
experiment-build-command: pnpm install && pnpm --filter performance-test-app exec ember build -e production --output-path dist-experiment --suppress-sizes
experiment-build-command: pnpm install && pnpm --filter performance-test-app build
experiment-serve-command: pnpm --filter performance-test-app exec ember s --path dist-experiment --port 4201
control-build-command: pnpm install && pnpm --filter performance-test-app exec ember build -e production --output-path dist-control --suppress-sizes
control-serve-command: pnpm --filter performance-test-app exec ember s --path dist-control
Expand Down
Loading

0 comments on commit d272cdc

Please sign in to comment.