Skip to content

Phase 5 — cutover plan (pre-staged; deploy held)#94

Closed
chrisbc wants to merge 1 commit into
migrate/strawberry-p4from
migrate/strawberry-p5
Closed

Phase 5 — cutover plan (pre-staged; deploy held)#94
chrisbc wants to merge 1 commit into
migrate/strawberry-p4from
migrate/strawberry-p5

Conversation

@chrisbc

@chrisbc chrisbc commented Jun 24, 2026

Copy link
Copy Markdown
Member

Stacked migration — PR 7 of N. Base = migrate/strawberry-p4 (Phase 4, #93).
Epic: GNS-Science/nshm-toshi-api#359 · Tracking: #86

Pre-stages the cutover. No deploy here — the actual test-stage deploy, cli_ab_test, and prod promote need AWS credentials + an explicit prod go-ahead.

docs/PHASE5_CUTOVER.md

  • In-place replacement — same stack/function/routes/URL; only the container image changes. Rollback = redeploy the previous image (image-only; the migration adds no new write shapes, so no data migration to reverse).
  • Validation reuses the built-in cli_ab_test (prod-vs-test) — it already covers the exact migrated surface, so no drive_live.py is needed (unlike the Model pilot). In-process parity is already locked by test_strawberry_parity.py (11 differential checks).
  • Rollback (pre-staged revert PR + triggers), promote + 30-min prod watch, comms checkpoints.
  • Post-healthy cleanup scope — delete graphene schema.py/Flask/handler.py, rename strawberry_schema.pyschema.py, drop legacy deps. Ordered after cutover (the differential tests + CompositeRuptureSections delegation still import graphene).

Status

Engineering (Phases 0–4) complete and stacked: #88#93. This PR + the deploy are the remaining steps; the deploy is yours to trigger.

- docs/PHASE5_CUTOVER.md: in-place replacement, rollback (image-only, no new
  write shapes), differential validation via the built-in cli_ab_test
  (prod-vs-test, covers the full migrated surface), promote + 30-min watch,
  post-healthy legacy-cleanup scope
- reuse cli_ab_test (no drive_live.py needed); in-process parity already locked
  by test_strawberry_parity.py
- actual deploy/promote needs AWS creds + prod go-ahead

Refs #86
@chrisbc

chrisbc commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

Landed via the combined PR #95 (one merge → one test-stage deploy, per runbook G7). Per-phase detail preserved here.

@chrisbc chrisbc closed this Jun 24, 2026
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