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

Fuse unmount #112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fuse unmount #112

wants to merge 1 commit into from

Conversation

nvictus
Copy link
Member

@nvictus nvictus commented Mar 1, 2023

Allows FuseProcess to check if a desired mountpoint is already mounted, and allows the user to forcibly unmount and restart the process locally.

Produces a warning if fuse.start(tmp_dir) is called but {tmp_dir}/schemas is already mounted.

Also adds at-exit handler to make sure a FUSE subprocess spawned by this process is cleaned up.

@manzt
Copy link
Member

manzt commented Apr 20, 2023

Kind of curious if we should move the fuse stuff under something like experimental. It would be really great to have this sorted out before a v1, but I anticipate this API is the most likely to need changes in the future. Exposing it under an experimental namespace might give us more flexibility to do so (and not prevent us from dragging our toes on an official release).

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.

None yet

2 participants