Repository navigation
DAOS-19915 build: update fused to 1.2.0 (libfuse 3.18.3) - #19417
Conversation
Point fused at the daos-stack/fused mschaara/fuse-3.18.3 branch for testing: upstream libfuse 3.18.3 renamed to fused, versioned 1.2.0, with the fusermount3 launch kept as fork+exec instead of posix_spawn, which does not work under valgrind. Bump the fused-devel package version so CI builds fused from source instead of installing 1.0.0. Features: dfuse pil4dfs Signed-off-by: Mohamad Chaarawi <mohamad.chaarawi@hpe.com>
|
Ticket title is 'update fused to latest libfuse release' |
libfuse 3.18 sets cpp_std=c++20, which gcc 7 on Leap 15 rejects when meson enables C++ for the examples. DAOS only needs the library. Features: dfuse pil4dfs Signed-off-by: Mohamad Chaarawi <mohamad.chaarawi@hpe.com>
|
Test stage Unit Test completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-19417/2/testReport/ |
|
Hi @mchaarawi, thanks for picking up the libfuse 3.18 update. A heads-up on an overlap with #19405. On 3.18.3 the fuse_kernel.h sync (d393ffa8) is already included, so the only missing piece is a backport of libfuse commit 1f4c0608. That commit is not in any upstream release yet. Would you be open to adding that one patch, either to the mschaara/fuse-3.18.3 fused branch or to deps/patches/fused in this PR? The capability is only sent to the kernel when a filesystem requests it, so behaviour without --allow-idmap is unchanged. If you agree, I'll close #19405 in favour of this PR. One packaging detail: dfuse guards the option with #ifdef, so a fused-devel 1.2.0 build without the patch would silently drop --allow-idmap. The BuildRequires should point at a fused-devel release that carries the patch. Test results for the 3.17.4 variant are in #19405: the NLT posix suite with --allow-idmap on every dfuse, and ftest with and without the option. I'm happy to rerun them against 1.2.0. |
thank you for you PRs and the heads up. i would like to keep this upgrade independent of your work and your feature request. |
|
Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-19417/3/execution/node/1309/log |
Features: dfuse pil4dfs Signed-off-by: Mohamad Chaarawi <mohamad.chaarawi@hpe.com>
daltonbohning
left a comment
There was a problem hiding this comment.
Where is this package actually used? It seems we only build fused-devel and at least for 2.8 we don't ship it?
https://packages.daos.io/v2.8.1/EL9/packages/x86_64/
we do not ship the fused RPM. it is statically built |
|
Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-19417/4/execution/node/1814/log |
update daos to build fused v1.2.0 (which is libfuse 3.18.3 + the relative patches to rename to fused and reversal to fork-exec instead of posix spawn). Signed-off-by: Mohamad Chaarawi <mohamad.chaarawi@hpe.com>
Point fused at the daos-stack/fused mschaara/fuse-3.18.3 branch for testing: upstream libfuse 3.18.3 renamed to fused, versioned 1.2.0, with the fusermount3 launch kept as fork+exec instead of posix_spawn, which does not work under valgrind. Bump the fused-devel package version so CI builds fused from source instead of installing 1.0.0.
Features: dfuse pil4dfs
Steps for the author:
After all prior steps are complete: