[FFe] samba-vfs-modules misses ceph vfs module
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
samba (Debian) |
Fix Released
|
Unknown
|
|||
samba (Ubuntu) |
Fix Released
|
Medium
|
Nish Aravamudan | ||
Xenial |
Won't Fix
|
Wishlist
|
Unassigned | ||
Yakkety |
Won't Fix
|
Wishlist
|
Unassigned |
Bug Description
Feature Freeze Exception justification:
* samba currently has no build-dependency on libcephfs-dev. Thus, at build-time, no cephfs support is detected and ceph support is not built into samba. However, we ship man-pages that refer to that support, and there is nothing in debian/changelog indicating this is an intentional omission.
* libcephfs-dev is in main as is libcephfs1 (the runtime dependency that results for samba), so no component mismatches should exist.
* Adding the build-dependency has been tested locally to generate the correct .so file as mentioned in this bug.
* Users expecting to use ceph with samba are currently surprised and possibly confused by the mixed support, including a man-page that states configuration optoins that will result in errors from samba.
* This is not a feature change, but a feature introduction, so I do not believe any regressions are possible, except if ceph support were somehow buggy. But if that is the case, we would currently never find this out or fix it ... This is also an issue for Debian and I plan on submitting the same change so it should not introduce any delta.
---
Man page for vfs_ceph exists (/usr/share/
Defining a share using "vfs objects = ceph" results in
"Error loading module '/usr/lib/
Changed in samba (Ubuntu): | |
assignee: | nobody → Nish Aravamudan (nacc) |
importance: | Undecided → Medium |
summary: |
- samba-vfs-modules misses ceph vfs module + [FFe] samba-vfs-modules misses ceph vfs module |
description: | updated |
Changed in samba (Debian): | |
status: | Unknown → New |
Changed in samba (Debian): | |
status: | New → Fix Committed |
Changed in samba (Debian): | |
status: | Fix Committed → Fix Released |
Changed in samba (Ubuntu Xenial): | |
importance: | Undecided → Wishlist |
Changed in samba (Ubuntu Yakkety): | |
importance: | Undecided → Wishlist |
Changed in samba (Ubuntu Xenial): | |
status: | New → Triaged |
Changed in samba (Ubuntu Yakkety): | |
status: | New → Triaged |
Hello and thank you for this report! It does seem like what is missing is a build-dependency on libcephfs-dev (also needed on Debian):
Checking for header cephfs/libcephfs.h : 03:18:32 runner /usr/bin/gcc -g -O2 -fdebug- prefix- map=/<< BUILDDIR> >/samba- 4.5.4+dfsg= . -fPIE -fstack- protector- strong -Wformat -Werror= format- security -D_FILE_ OFFSET_ BITS=64 -MD -D_FILE_ OFFSET_ BITS=64 -Wdate-time -D_FORTIFY_SOURCE=2 -I/usr/ local/include -D_SAMBA_BUILD_=4 -DHAVE_CONFIG_H=1 -D_GNU_SOURCE=1 -D_XOPEN_ SOURCE_ EXTENDED= 1 ../test.c -c -o default/test_1.o
no
Based upon the build-log for the 17.04 version. I expect the same output/missing file on prior versions. I am building a test version for 17.04 with the fixed b-d first and then will submit the bug to Debian and upload for 17.04.
Thanks,
Nish