Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: verify there are no missing migrations before migrating #89

Merged
merged 1 commit into from
Oct 13, 2020
Merged

fix: verify there are no missing migrations before migrating #89

merged 1 commit into from
Oct 13, 2020

Conversation

iand
Copy link
Contributor

@iand iand commented Oct 12, 2020

fixes #85

@iand iand requested a review from placer14 October 12, 2020 12:57
@iand iand merged commit 2625d0d into master Oct 13, 2020
@iand iand deleted the fix/migration-miscount branch October 13, 2020 13:45
@frrist frrist mentioned this pull request Oct 14, 2020
Merged
placer14 added a commit that referenced this pull request Oct 14, 2020
@placer14
Merge remote-tracking branch 'origin/master' into feat/minerposts
facef9d 
* origin/master:
  fix: remove errgrp from UnindexedBlockData persist
  fix: verify there are no missing migrations before migrating (#89)
  chore: add tests for reward and power actor state extracters (#83)
  feat: support v2 actor codes (#84)
  feat(debug): Process actor by head without persistance (#86)
  fix: use debugf logging method in message processor (#82)
  feat(task): Create chainvis views and refresher (#77)
  perf: minor optimization of market actor diffing (#78)
  chore: fail database tests if VISOR_TEST_DB not set (#79)
  test(storage): add test to check for duplicate schema migrations (#80)
  in-process lotus "repo" lens (#75)

 Conflicts:
        tasks/actorstate/miner.go
iand pushed a commit that referenced this pull request Oct 14, 2020
@frrist @iand
fix(migrations): migrations require version 0
6ae8a53 
I suspect this relates to #89 but am not certain.
frrist added a commit that referenced this pull request Oct 14, 2020
@frrist
fix(migrations): migrations require version 0
e6b228a 
I suspect this relates to #89 but am not certain.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@willscott willscott willscott approved these changes

@frrist frrist frrist approved these changes

@placer14 placer14 Awaiting requested review from placer14

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migration miscount when migration numbers are missing
3 participants
@iand @willscott @frrist