Skip to content

perf: add perf.tempo.xyz custom domain routing#960

Open
decofe wants to merge 3 commits into
mainfrom
perf-custom-domain
Open

perf: add perf.tempo.xyz custom domain routing#960
decofe wants to merge 3 commits into
mainfrom
perf-custom-domain

Conversation

@decofe
Copy link
Copy Markdown
Member

@decofe decofe commented May 15, 2026

Adds a Cloudflare custom domain route for perf.tempo.xyz to the perf Worker wrangler config, following the same pattern used by other tempo-apps Workers (snapshots, keys, explorer, etc.).

Once merged and deployed, the perf site will be accessible at https://perf.tempo.xyz in addition to the existing *.porto.workers.dev URL.

Adds a Cloudflare custom domain route for perf.tempo.xyz to the perf
Worker, following the same pattern used by other tempo-apps Workers
(snapshots, keys, explorer, etc.).

Amp-Thread-ID: https://ampcode.com/threads/T-019e2c14-4614-73b3-a1f0-2e8954ab24b1
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

Cloudflare Deployments

App Environment Status Preview
explorer devnet [>>] Skipped No changes
explorer mainnet [>>] Skipped No changes
explorer testnet [>>] Skipped No changes
fee-payer devnet [>>] Skipped No changes
fee-payer moderato [>>] Skipped No changes
fee-payer privy [>>] Skipped No changes
og - [>>] Skipped No changes
perf - [OK] Deployed View Preview
reth-snapshots-viewer - [>>] Skipped No changes
tempo-snapshots-viewer - [>>] Skipped No changes
tokenlist - [>>] Skipped No changes

Copy link
Copy Markdown
Member

@emmajam emmajam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shekhirin should merge once prod site is ready

Lower CHART_POINT_TARGET from 750 → 300 and add a MAX_METRIC_ROWS
(15,000) hard cap on the metric samples query. Mix workload benchmark
runs produce many more metric series (due to diverse label
combinations across quantiles and node types), and the cartesian
product of series × points was exceeding the Cloudflare Worker's
128 MB memory limit, causing error 1101 crashes.

300 points per series is still sufficient for smooth chart rendering,
and the 15k row cap ensures the total response stays well within
the Worker's memory budget even for the heaviest runs.

Amp-Thread-ID: https://ampcode.com/threads/T-019e2c14-4614-73b3-a1f0-2e8954ab24b1
Comment thread apps/perf/src/lib/server/bench.ts Outdated
Comment thread apps/perf/src/lib/server/bench.ts Outdated
Co-authored-by: Emma Jamieson-Hoare <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants