Closed ariel-miculas closed 10 months ago
Is there any reason why this can't be done with squashfuse as well as squashfuse_ll?
No, I'll add this to squashfuse also.
The code looks pretty good to me now (although the CI checks are running at the moment). Please also add the new option to the squashfuse.1
man page and add an automated test or tests.
Added some comments, but overall looks great!
Thanks, @vasi. I've addressed your comments, updated the squashfuse.1
manpage and modified some tests in ll-smoke.sh.in
to use this new notification mechanism.
The new new smoke test unfortunately failed on the freebsd check. Look for FAIL:
. It says
Image mounted successfully when it should have failed
Thanks, I was looking for this log but couldn't find it. So it's probably fine to overmount squashfuse on top of an existing squashfuse mountpoint on freebsd, I need to change the test to make it fail the mount in other ways.
I'm still awaiting approval for one workflow.
Thanks for the reminder, I'm sorry I didn't notice the push. The good news was that freebsd passed. I now approved the workflow.
This isn't fully working, see #112
Add the notify_pipe option which allows the user to specify the path to a named pipe. When the mountpoint is ready, a message will be written to this pipe: 's' for success and 'f' for failure. To avoid blocking the fuse process until the pipe is read, a new process is spawned which performs a blocking operation on the pipe. In case of mount failure, no additional process is created and the main process blocks until the pipe is read.
An example of operation is provided below:
Fixes #49