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.

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

Add 'ipfs repo migrate' command #8428

Merged
merged 2 commits into from
May 6, 2022
Merged

Add 'ipfs repo migrate' command #8428

merged 2 commits into from
May 6, 2022

Conversation

gammazero
Copy link
Contributor

@gammazero gammazero commented Sep 10, 2021

This PR replaces #7658 that was originally contributed by @zaibon, in order to move code into a branch and avoid some CI problems.

The command allows the user to run the repo migration without starting the daemon.

resolves #7471

@gammazero gammazero requested review from aschmahmann and removed request for aschmahmann September 10, 2021 19:09
@BigLep BigLep requested a review from aschmahmann March 3, 2022 13:32
@BigLep BigLep added this to the Best Effort Track milestone Mar 3, 2022
@BigLep BigLep requested review from guseggert and removed request for aschmahmann May 6, 2022 15:59
@guseggert guseggert force-pushed the feat/cmds/repo-migrate branch 2 times, most recently from e1a3b32 to e148f03 Compare May 6, 2022 20:22
This PR replaces #7658 that was originally contributed by zaibons, in order to move code into a branch and avoid some CI problem.

The command allows the user to run the repo migration without starting the daemon.

resolves #7471
@guseggert
Copy link
Contributor

I rebased w/ master and fixed a bug where this wasn't propagating non-ErrNeedMigration errors (e.g. if you try to migrate while the daemon is running, you should get a lock error, not "no migration is needed")

@guseggert guseggert merged commit 889f73e into master May 6, 2022
@guseggert guseggert deleted the feat/cmds/repo-migrate branch May 6, 2022 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

ipfs migration without starting the daemon
3 participants