Skip to content

feat: skip persistent schemas that don't exist on source during migration#8

Merged
passcod merged 4 commits intomainfrom
feat/skip-missing-source-schemas
Feb 28, 2026
Merged

feat: skip persistent schemas that don't exist on source during migration#8
passcod merged 4 commits intomainfrom
feat/skip-missing-source-schemas

Conversation

@passcod
Copy link
Member

@passcod passcod commented Feb 28, 2026

When the persistent_schemas list includes a schema that was never created on the source restore, the operator now queries the source database to filter the list before launching the migration job. Missing schemas are logged at warn level and silently skipped.

If none of the listed schemas exist on the source, migration is skipped entirely and the switchover proceeds immediately.

🦸 Review Hero

  • Run Review Hero
  • Auto-fix review suggestions
  • Auto-fix CI failures

…tion

When the persistent_schemas list includes a schema that was never created
on the source restore, the operator now queries the source database to
filter the list before launching the migration job. Missing schemas are
logged at warn level and silently skipped.

If none of the listed schemas exist on the source, migration is skipped
entirely and the switchover proceeds immediately.
@beyondessential beyondessential deleted a comment from review-hero bot Feb 28, 2026
@review-hero
Copy link

review-hero bot commented Feb 28, 2026

🦸 Review Hero Summary
3 agents reviewed this PR | 1 critical | 4 suggestions | 1 nitpick

@review-hero
Copy link

review-hero bot commented Feb 28, 2026

🦸 Review Hero Auto-Fix
Applied fixes for changes.

Skipped 4 comments (replied on each thread).

@beyondessential beyondessential deleted a comment from review-hero bot Feb 28, 2026
@beyondessential beyondessential deleted a comment from review-hero bot Feb 28, 2026
@beyondessential beyondessential deleted a comment from review-hero bot Feb 28, 2026
@beyondessential beyondessential deleted a comment from review-hero bot Feb 28, 2026
@review-hero
Copy link

review-hero bot commented Feb 28, 2026

🦸 Review Hero Auto-Fix
Applied fixes for 3 CI failures.

@passcod passcod force-pushed the feat/skip-missing-source-schemas branch from 099185c to 1918d73 Compare February 28, 2026 12:18
The in-progress restore check at line 898 was running before the
consecutive failures check, setting the phase back to Restoring and
returning early. This prevented the suspension block from correcting
the phase, causing it to stay stuck at Restoring even while suspended.
@passcod passcod force-pushed the feat/skip-missing-source-schemas branch from 1918d73 to 918f1d8 Compare February 28, 2026 12:19
@passcod passcod merged commit bd0fb42 into main Feb 28, 2026
14 checks passed
@passcod passcod deleted the feat/skip-missing-source-schemas branch February 28, 2026 12:43
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