Skip to content

ci(relay): add a forced manual relay deploy - #13550

Merged
juliusmarminge merged 1 commit into
mainfrom
t3code/relay-force-redeploy
Sep 25, 2026
Merged

juliusmarminge merged 1 commit into
mainfrom
t3code/relay-force-redeploy

Conversation

@juliusmarminge

@juliusmarminge juliusmarminge commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Changing a relay variable in the production environment never reaches the Worker. Alchemy does not redeploy the Worker when only a Config value read in its Init changes (alchemy-run/alchemy#1831), so both a rerun and a push to main without relay code changes plan "no changes". That's why RELAY_TUNNEL_CLEANUP_MODE=dry-run is set but production still runs off. The same would happen to the incident switch back to off, and to a rotated secret such as CLERK_SECRET_KEY.

This adds a workflow_dispatch trigger to Deploy T3 Connect relay with a force input (checked by default) that runs alchemy deploy --force. The job only runs on refs/heads/main, so a manual run from another branch can't deploy that branch to production. Pushes to main behave as before.

--force makes every resource in the stack reconcile. I checked the ones that hold credentials: CloudMintKeyPair, ApnsDeliveryJobSigningSecret, the Axiom ingest tokens, and RelayPostgresRuntimeRole all keep their stored value when one already exists. A forced deploy rotates no keys, tokens or passwords.

docs/operations/release.md now says a changed variable needs a forced deploy, and uses one in the cleanup rollout, the incident switch, the rollback, and the canary.

Verification: vp run forwards --force to alchemy deploy, confirmed with a probe script. actionlint reports only the existing custom Blacksmith runner label. After merging, run the workflow from main once to apply dry-run, then check relay.managed_endpoint_reaper.mode on the next sweep span in Axiom.

🤖 Generated with Claude Code


Devin Review

Summary by CodeRabbit

  • Improvements
    • Production relay deployments can now be manually run with or without a forced redeployment, and are limited to the main branch.
  • Documentation
    • Updated relay release and managed tunnel cleanup guidance, including deployment, monitoring, and rollback steps.

Alchemy does not redeploy the relay Worker when only a Config value read in
its Init changes (alchemy-run/alchemy#1831). Changing a production variable
such as RELAY_TUNNEL_CLEANUP_MODE therefore never reached the Worker, and
neither would turning tunnel cleanup off during an incident. Add a manual
trigger that runs alchemy deploy --force from main, and update the release
steps to use it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Sep 25, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — The workflow adds a manual --force reconciliation path for the production relay, changing how production infrastructure and the Worker are deployed. Although the normal push path and documentation changes are low risk, the production deployment behavior warrants human review.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.5 KiB 13.5 KiB −14 B (−0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −3 B (−0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.5 KiB 6.5 KiB −11 B (−0.2%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 56.3 KiB 56.3 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 10 10 0 (0.0%) 21 ✅
Claude Total thread wire 13.5 KiB 13.5 KiB +11 B (+0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +8 B (+0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.5 KiB 6.5 KiB +3 B (+0.0%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 57.0 KiB 57.0 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 9 9 0 (0.0%) 21 ✅

Baseline: 568c9bc · PR result: 2f71548 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 114.0 KiB
  • Claude decoded thread snapshot: 114.7 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@juliusmarminge
juliusmarminge merged commit fc46b8c into main Sep 25, 2026
21 of 23 checks passed
@juliusmarminge
juliusmarminge deleted the t3code/relay-force-redeploy branch September 25, 2026 03:38
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 25, 2026
## What's Changed
* fix(web): sidebar Back always returns to the main app by @t3dotgg in pingdotgg/t3code#13516
* fix(desktop): desktop updates reconnect in seconds, not minutes by @t3dotgg in pingdotgg/t3code#12006
* fix(connect): remove tunnels after hosts go offline by @t3dotgg in pingdotgg/t3code#9386
* fix(mobile): capture a lit 6.9-inch lock screen in the agent-activity showcase by @juliusmarminge in pingdotgg/t3code#13522
* feat(grok): offer one-click updates through `grok update` by @juliusmarminge in pingdotgg/t3code#13523
* fix(mobile): make Android subscription usage widgets scrollable by @tris203 in pingdotgg/t3code#13474
* fix(web): keep sidebar terminal pulses in sync by @t3dotgg in pingdotgg/t3code#12962
* feat(web): add iPhone Duo 3D controls by @juliusmarminge in pingdotgg/t3code#12813
* fix(relay): export tunnel cleanup counters to Axiom by @juliusmarminge in pingdotgg/t3code#13528
* fix(server): Grok accounts with no usage yet no longer vanish from Limits by @jakeleventhal in pingdotgg/t3code#12799
* fix(server): report the Grok account email so usage limits merge across environments by @jakeleventhal in pingdotgg/t3code#12588
* feat(web): add usage page keybinding by @jakeleventhal in pingdotgg/t3code#9434
* chore: clear Effect language service suggestions by @juliusmarminge in pingdotgg/t3code#13536
* ci(relay): add a forced manual relay deploy by @juliusmarminge in pingdotgg/t3code#13550
* fix(web): selected text stays visible on a revealed file line by @t3dotgg in pingdotgg/t3code#13548
* fix(web): collapsed composer bar stops flipping its labels while you scroll by @t3dotgg in pingdotgg/t3code#13555
* fix(clients): sync status no longer flickers when opening running threads by @t3dotgg in pingdotgg/t3code#13551
* fix(server): newer Codex models get T3 Code's instructions again by @t3dotgg in pingdotgg/t3code#13547


**Full Changelog**: pingdotgg/t3code@v0.0.43-nightly.20260924.2223...v0.0.43-nightly.20260925.2237

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.43-nightly.20260925.2237
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant