Skip to content

Route MPGv2 restores through Machines API - #5140

Merged
tripledoublev merged 2 commits into
masterfrom
vincent/mpg-machines-restore-flyctl
Aug 28, 2026
Merged

Route MPGv2 restores through Machines API#5140
tripledoublev merged 2 commits into
masterfrom
vincent/mpg-machines-restore-flyctl

Conversation

@tripledoublev

Copy link
Copy Markdown
Contributor

Summary

  • route fly mpg restore for MPGv2 clusters through the typed Machines API
  • retain the legacy MPGv2 restore endpoint as a 404 compatibility fallback without masking authoritative public errors
  • preserve MPGv1 restore routing and update fly-go to v0.9.11

Testing

  • GOWORK=off mise exec -- go test ./... -count=1
  • exercised real MPGv2 backup-ID and point-in-time restores through readiness
  • verified restored database state across pre-write backup, post-write backup, and PITR boundaries
  • verified requested and server-generated names, validation errors, public service errors, and MPGv1 routing
  • confirmed all E2E-created clusters were deleted

Use fly-go's typed restore endpoint for MPGv2 backup and point-in-time restores. Retain the legacy endpoint as a 404 fallback, while preserving the authoritative public error if that fallback also fails.

@tomr-a5 tomr-a5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tripledoublev
tripledoublev merged commit 2c42b5e into master Aug 28, 2026
27 checks passed
@tripledoublev
tripledoublev deleted the vincent/mpg-machines-restore-flyctl branch August 28, 2026 19:58
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