Skip to content

feat(relay): add syd (Sydney) region to relay fleet - #5544

Merged
Kitenite merged 1 commit into
mainfrom
add-australia-relay-regio
Jul 9, 2026
Merged

Kitenite merged 1 commit into
mainfrom
add-australia-relay-regio

Conversation

@Kitenite

@Kitenite Kitenite commented Jul 9, 2026 •

Copy link
Copy Markdown
Member

What

Adds Fly's Sydney region (syd) to the relay fleet, bringing it to 7 machines, 1 per region: sjc (primary), iad, fra, nrt, sin, syd, gru.

Why

The relay fleet is driven entirely by the REGIONS array in apps/relay/scripts/deploy.sh — the relay reads FLY_REGION dynamically and fly-replay routes by region string, so there's no client-side allowlist to touch.

Australian users previously had no closer relay than sin/nrt. When their remote host lands in a Sydney-adjacent Fly region, the region mismatch surfaces as a relay 502 on WebSocket upgrades (fly-replay isn't transparent for WS upgrades). A syd machine gives AU traffic a local relay and closes that gap.

Changes

  • apps/relay/scripts/deploy.sh — add syd to REGIONS (6 → 7)
  • apps/relay/fly.toml — update topology comment

Rollout

Run apps/relay/scripts/deploy.sh: the fly scale count "app=7" --region ... --max-per-region 1 step provisions the Sydney machine, followed by the rolling deploy and smoke test (which now covers syd).


Open in Stage

Summary by cubic

Add syd (Sydney) to the relay fleet, bringing it to 7 regions with 1 machine each. This gives AU users a local relay and reduces WebSocket 502s caused by region mismatches.

  • Migration
    • Run apps/relay/scripts/deploy.sh; it scales to 7 regions (adds syd), deploys, and runs smoke tests.

Written for commit 8ab79f2. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores
    • Expanded deployment coverage to an additional region.
    • Updated scaling settings so the service now starts with one more machine, improving regional availability and consistency.

@cursor

cursor Bot commented Jul 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@ghost

ghost commented Jul 9, 2026

Copy link
Copy Markdown

Ready to review this PR? Stage has broken it down into 2 individual chapters for you:

Title
1 Add Sydney region to relay deployment script
2 Update topology documentation in fly.toml
Open in Stage

Chapters generated by Stage for commit 8ab79f2 on Jul 9, 2026 6:51pm UTC.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4ef71b40-fe79-42e7-87b0-b8c3b052243f

📥 Commits

Reviewing files that changed from the base of the PR and between 4ea555e and 8ab79f2.

📒 Files selected for processing (2)
  • apps/relay/fly.toml
  • apps/relay/scripts/deploy.sh

📝 Walkthrough

Walkthrough

The relay deployment script's REGIONS array is expanded to include the "syd" region, affecting the computed machine count and region list used for fly scale and smoke test invocations. The fly.toml comment is updated to reflect the new target of 7 machines (1 per region).

Changes

Relay Region Expansion

Layer / File(s) Summary
Add syd region to deploy fleet
apps/relay/scripts/deploy.sh, apps/relay/fly.toml
REGIONS array now includes syd, changing computed COUNT and REGION_LIST for fly scale and smoke test; fly.toml comment updated from 6 to 7 machines (1 per region).

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • superset-sh/superset#4678: Both PRs modify apps/relay/scripts/deploy.sh's REGIONS list and derived region count/list used for fly scale.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the main change: adding the Sydney relay region to the relay fleet.
Description check ✅ Passed The description covers what changed, why, rollout, and the affected files, though some template sections are left unfilled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-australia-relay-regio

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Kitenite
Kitenite merged commit 7156da7 into main Jul 9, 2026
10 of 11 checks passed
@greptile-apps

greptile-apps Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds Fly's Sydney (syd) region to the relay fleet, growing it from 6 to 7 machines (one per region). The change reduces latency for Australian users who previously had to rely on sin or nrt as their nearest relay.

  • apps/relay/scripts/deploy.sh — syd inserted into the REGIONS array; COUNT and REGION_LIST are derived automatically, so the fly scale count call and smoke test both pick it up with no further edits required.
  • apps/relay/fly.toml — topology comment updated to reflect the new 7-machine target and includes syd in the region list.

Confidence Score: 5/5

Safe to merge — single-region addition with no logic changes and automatic propagation through the deploy script.

The change is two lines: one string inserted into a bash array, one comment updated. COUNT and REGION_LIST are both derived from the array at runtime, so the scale count, rolling deploy, and smoke test all cover syd automatically. The fly.toml comment is accurate. No risk of misconfiguration or regression.

No files require special attention.

Important Files Changed

Filename Overview
apps/relay/scripts/deploy.sh Adds syd to the REGIONS array; COUNT and REGION_LIST are derived from the array, so scale count, deploy, and smoke-test arguments all update automatically.
apps/relay/fly.toml Comment-only update: topology description now reflects 7 machines with syd included — no functional config changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph deploy["deploy.sh execution"]
        A["REGIONS=(sjc iad fra nrt sin syd gru)"] --> B["fly scale count app=7\n--region sjc,iad,fra,nrt,sin,syd,gru\n--max-per-region 1"]
        B --> C["fly deploy (rolling)"]
        C --> D["fly status"]
        D --> E["smoke-test.sh over all 7 regions"]
    end

    subgraph fleet["Relay Fleet (post-deploy)"]
        sjc["sjc — primary"]
        iad["iad"]
        fra["fra"]
        nrt["nrt"]
        sin["sin"]
        syd["syd NEW"]
        gru["gru"]
    end

    E --> fleet
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    subgraph deploy["deploy.sh execution"]
        A["REGIONS=(sjc iad fra nrt sin syd gru)"] --> B["fly scale count app=7\n--region sjc,iad,fra,nrt,sin,syd,gru\n--max-per-region 1"]
        B --> C["fly deploy (rolling)"]
        C --> D["fly status"]
        D --> E["smoke-test.sh over all 7 regions"]
    end

    subgraph fleet["Relay Fleet (post-deploy)"]
        sjc["sjc — primary"]
        iad["iad"]
        fra["fra"]
        nrt["nrt"]
        sin["sin"]
        syd["syd NEW"]
        gru["gru"]
    end

    E --> fleet
Loading

Reviews (1): Last reviewed commit: "feat(relay): add syd (Sydney) region to ..." | Re-trigger Greptile

@github-actions

github-actions Bot commented Jul 9, 2026 •

Copy link
Copy Markdown
Contributor

🚀 Preview Deployment

🔗 Preview Links

Service Status Link
Neon Database (Neon) ✅ View Branch
Vercel API (Vercel) ✅ Open Preview
Vercel Web (Vercel) ✅ Open Preview
Vercel Marketing (Vercel) ✅ Open Preview
Vercel Admin (Vercel) ✅ Open Preview
Vercel Docs (Vercel) ✅ Open Preview

Preview updates automatically with new commits

This branch was previously deployed

1 inactive deployment
preview — 8ab79f2d Deployed Jul 9, 2026 by Kitenite via Deploy Marketing #6281
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.

1 participant