Skip to content

[STANDBY ROLLBACK] Revert #97 Strawberry promote — DO NOT MERGE unless prod degrades#101

Closed
chrisbc wants to merge 1 commit into
mainfrom
rollback/revert-strawberry-promote
Closed

[STANDBY ROLLBACK] Revert #97 Strawberry promote — DO NOT MERGE unless prod degrades#101
chrisbc wants to merge 1 commit into
mainfrom
rollback/revert-strawberry-promote

Conversation

@chrisbc

@chrisbc chrisbc commented Jun 26, 2026

Copy link
Copy Markdown
Member

Pre-staged rollback for the Graphene→Strawberry prod cutover (#97, merge 5ed85863), per the cutover plan's ~30-min watch.

Merging this reverts the entire migration on main and redeploys the previous Graphene image. Safe because the cutover is image-only — no new DynamoDB/S3 write shapes were introduced, so there is nothing to un-migrate in storage.

Do not merge unless prod health degrades during the watch window. If the watch is clean, close this PR (don't merge).

Prod smoke at promote: {about} 200 (v0.9.2), radii 200, get_parent_fault_names 200 (557).

This reverts commit 5ed8586, reversing
changes made to cdd9dfe.
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.63093% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.73%. Comparing base (5ed8586) to head (5d0d235).

Files with missing lines Patch % Lines
solvis_graphql_api/color_scale/color_scale.py 90.74% 5 Missing and 5 partials ⚠️
...i/composite_solution/composite_rupture_sections.py 94.82% 5 Missing and 1 partial ⚠️
solvis_graphql_api/data_store/model.py 86.11% 4 Missing and 1 partial ⚠️
solvis_graphql_api/geojson_style.py 95.83% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   96.54%   95.73%   -0.81%     
==========================================
  Files          12       19       +7     
  Lines        1099     1126      +27     
  Branches       63       53      -10     
==========================================
+ Hits         1061     1078      +17     
- Misses         25       34       +9     
- Partials       13       14       +1     
Flag Coverage Δ
unittests 95.73% <96.63%> (-0.81%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisbc

chrisbc commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

Closing unmerged — prod cutover healthy. 30-min watch: 6/6 ticks, 0 failures; 22+ POST /graphql 200 under live kororaa-prod traffic. The only log errors were a one-off post-deploy cold start + a client query with fault_system: "" (a KeyError in unchanged shared compute — graphene behaved identically; not a regression). Rollback not required.

@chrisbc chrisbc closed this Jun 26, 2026
@chrisbc chrisbc deleted the rollback/revert-strawberry-promote branch June 26, 2026 04:25
chrisbc added a commit that referenced this pull request Jun 26, 2026
Record the deploy-test→main promote (#97, 5ed8586), prod deploy success, the pre-staged+closed
rollback #101, the 30-min watch (6/6, 0 failures), the #99/#100 coverage epilogue, and the
non-regression KeyError note (empty fault_system in unchanged shared compute).
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