Add support for mounting of subdirectory#89
Add support for mounting of subdirectory#89DrDaveD merged 1 commit intovasi:masterfrom SeTSeR:subdir
Conversation
|
@DrDaveD I don't think I would be able to properly implement and test this with low-level API right now, I didn't work with the project for a quite a long time and I did not manage to install testing environment on my available boards quickly enough. I added description of new option to |
|
I don't think this should be only in high-level, so this isn't going to make it into the next release. Hopefully someone else who is interested in the feature will complete it. |
|
Redesigned the implementation in a way it would work with both high-level and low-level versions, tested with archive with archive with branching structure:
|
DrDaveD
left a comment
There was a problem hiding this comment.
Please rebase on the current main, because it wasn't uploading the test-suite.log to hopefully give us information about why the ll-smoke.sh test is failing.
Also please add the new option to squashfuse.1. It doesn't need to be in squashfuse_ll.1 because that refers people to squashfuse for descriptions of common options.
Added option to manual, fixed review comment and also fixed style to more consistent to codebase. As for test failure, I couldn't reproduce that locally, everything passes correctly: And also I'm planning to add some tests with subdirectories. |
|
Getting CI logs is kinda tricky but the test logs seem to be here, in after stage. Recent change to GitHub workflow seems to have no effect, as FreeBSD job is set up in Logs show there is some problem with mounting: I was unable trace the source of hang further, smoke test logs are not exposed by this CI. |
|
Thanks, in docker with ubuntu 20.04 I was able to reproduce the issue. For some reason mounting with these changes began to hang, but only with debug information enabled. Might be some issue with argument handling in older |
|
It was the github action logs that I most cared about, but it turned out that they were only preserving logs from one of the matrix tests. I have now fixed that in #96 to save logs for each of them. Meanwhile I too have been able to reproduce this interactively on an Ubuntu 20.04 VM. There's a little more information in test-suite.log: Ah, but you have fixed it in the latest force-push? |
|
Except it still failed on freebsd. Those logs are shown in the "Run after" section rather than being uploaded, except not all of them. I will update that test too. |
|
Yes, I fixed the crash on Ubuntu 22.04 and also added test for subdir mounting in |
|
Meanwhile I updated the freebsd after check to show all logs in #98 so if you rebase on master again you should get that debugging help. |
|
Well that wasn't helpful. It did indicate there might be some help in a /tmp log, so now in #99 those are saved as well. Please rebase once again. |
|
I managed to reproduce the problem on VM with FreeBSD 13, log is quite simple: Looks like some arguments are still not on their places. |
This commit adds ability to mount subdirectory of image with option -o subdir=/dir.
This commit adds ability to mount subdirectory of image with option -o subdir=/dir.
Continuation of #75, rebased and renamed branch