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 recover in ipfs mount command. #124

Closed
whyrusleeping opened this issue Sep 25, 2014 · 5 comments
Closed

Add recover in ipfs mount command. #124

whyrusleeping opened this issue Sep 25, 2014 · 5 comments

Comments

@whyrusleeping
Copy link
Member

Ive found that if ipfs panics for any reason (as rare as that might be) the fuse stuff gets into a funky state since it wasnt properly dismounted, i wanted to discuss the pros and cons of having a recover statement in the ipfs mount command to catch those and safely unmount any mounted fuse systems.

@jbenet
Copy link
Member

jbenet commented Sep 25, 2014

Yep, SGTM. We should have proper teardown.—
Sent from Mailbox

On Wed, Sep 24, 2014 at 9:33 PM, Jeromy Johnson notifications@github.com
wrote:

Ive found that if ipfs panics for any reason (as rare as that might be) the fuse stuff gets into a funky state since it wasnt properly dismounted, i wanted to discuss the pros and cons of having a recover statement in the ipfs mount command to catch those and safely unmount any mounted fuse systems.

Reply to this email directly or view it on GitHub:
#124

@cryptix
Copy link
Contributor

cryptix commented Mar 23, 2015

I think we tackled most of these in #945 and can close this for now?

cc @whyrusleeping reopen if I missed something.

@cryptix cryptix closed this as completed Mar 23, 2015
@whyrusleeping
Copy link
Member Author

Ah, im fine with closing this. but this wasnt about race conditions, it was about fuse not being unmounted properly in the case of a panic

@cryptix
Copy link
Contributor

cryptix commented Mar 23, 2015

Oh... I switched cause and outcome here than.

I wonder where a recover() could be placed. It would have to be somewhere blocking for ipfs to exit, right? The mount core command wouldn't work for that since they return to early.

@whyrusleeping
Copy link
Member Author

Yeah, i wasnt sure where to put it, which is why i never got around to doing it, lol

@Stebalien Stebalien mentioned this issue May 26, 2020
77 tasks
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

No branches or pull requests

3 participants