Skip to content

avoid early bail out in migration manager#3322

Open
cody-littley wants to merge 2 commits intomainfrom
cjl/STO-502
Open

avoid early bail out in migration manager#3322
cody-littley wants to merge 2 commits intomainfrom
cjl/STO-502

Conversation

@cody-littley
Copy link
Copy Markdown
Contributor

Describe your changes and provide context

Don't bail out early in the migration manager if the context gets cancelled.

Testing performed to validate your change

unit test coverage

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedApr 27, 2026, 7:36 PM

@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedApr 27, 2026, 7:36 PM

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.21%. Comparing base (d21cb23) to head (1b1cf8c).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3322      +/-   ##
==========================================
- Coverage   59.21%   59.21%   -0.01%     
==========================================
  Files        2094     2094              
  Lines      172070   172068       -2     
==========================================
- Hits       101893   101891       -2     
  Misses      61370    61370              
  Partials     8807     8807              
Flag Coverage Δ
sei-chain-pr 80.69% <ø> (?)
sei-db 70.41% <ø> (ø)

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

Files with missing lines Coverage Δ
sei-db/state_db/sc/migration/migration_manager.go 96.13% <ø> (-0.04%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant