-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
nfs-ganesha: Build with Ceph and DBus support #432492
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ceph is currently broken on master (i.e. arrow-cpp is actually broken).
Yeah, I noticed the ceph build breakage. I tested this by using the module from master on 25.05. Sorry, should have mentioned that. It turns out my project won't end up using nfs-ganesha after all, so I don't have a need for this functionality anymore. I'll still try to update the PR to make the ceph dependency optional when I have some free time. Thanks for taking the time to look at it. |
7c1b6d1
to
cdf3e9b
Compare
Turns out I do need nfs-ganesha after all :) I updated the Ceph changes to make them optional and also updated to build other Ceph features like the rados_ng recovery backend. Since I was working on this, I also made two unrelated changes - one is to bump the nfs-ganesha version to the new 7.0 release, and the other is to enable DBus (can be used for runtime management and to collect service metrics). I figured it would be easier to have all three in the same PR, but if you prefer I can open separate PRs for those. They are all self-contained in their own commits. The build patch for DBus is a bit ugly, but I just could not get cmake to pick up the DBus headers and libs through pkg-config. |
CephFS and RGW FSAL, storage of recovery info in RADOS objects, and RADOS URL support.
cdf3e9b
to
14a187d
Compare
I pulled the v7.0 patch, I'll submit that as a separate PR. Sorry for the churn. |
Okay, the v7.0 patch is ready. Needed to upgrade ntirpc as well. I added it to this PR, but let me know if you prefer me to split them up. I also tested this on nixpkgs master, since the build failures seem to be fixed now. NFS serving works, no problems observed. |
b8b88b6
to
b5cfd98
Compare
This enables using the Ceph FSAL to serve files from CephFS. Also enable DBus.
Testing was done against nixpkgs-25.05 using callPackage due to breakage on unstable, but I don't think it matters here.
Things done
passthru.tests
.nixpkgs-review
on this PR. See nixpkgs-review usage../result/bin/
.Add a 👍 reaction to pull requests you find important.