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

Improve the Filestore document #5797

Closed
wants to merge 1 commit into from

Conversation

bjzhang
Copy link

@bjzhang bjzhang commented Nov 26, 2018

Fix for the #5746. Switch to Filestore need to restart the ipfs node.

License: MIT
Signed-off-by: Bamvor Zhang jian.zhang@ipfsbit.com

Fix for the ipfs#5746. Switch to Filestore need to restart the ipfs node.

License: MIT
Signed-off-by: Bamvor Zhang <jian.zhang@ipfsbit.com>
@bjzhang bjzhang requested a review from Kubuxu as a code owner November 26, 2018 17:25
@kevina kevina self-assigned this Nov 26, 2018
@kevina kevina self-requested a review November 26, 2018 17:36
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

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

Sorry for letting this fall through the cracks!

Restart ipfs node. Otherwise `ipfs add` will not complain and `ipfs filestore`
will raise error.

Then pass the `--nocopy` flag when running `ipfs add`

Copy link
Member

Choose a reason for hiding this comment

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

As written, this is a bit awkward. How about something like:

Then restart your IPFS node to reload your config.

Finally, when adding files with ipfs add, pass the --nocopy flag to use the filestore instead of copying the files into your local IPFS repo.

Copy link
Author

Choose a reason for hiding this comment

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

Hi, I push the above modification to the new pr: #5927.
I have to create a new pr because the permission of ipbit is changed. Sorry for inconvenience.

@Stebalien Stebalien closed this Jan 17, 2019
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.

3 participants