Mailinglist Archive: opensuse-bugs (4258 mails)

< Previous Next >
[Bug 1021933] New: Samba vfs_ceph build fails against latest Ceph
  • From: bugzilla_noreply@xxxxxxxxxx
  • Date: Wed, 25 Jan 2017 16:44:00 +0000
  • Message-id: <bug-1021933-21960@http.bugzilla.opensuse.org/>
http://bugzilla.opensuse.org/show_bug.cgi?id=1021933


Bug ID: 1021933
Summary: Samba vfs_ceph build fails against latest Ceph
Classification: openSUSE
Product: openSUSE Tumbleweed
Version: Current
Hardware: Other
OS: Other
Status: NEW
Severity: Normal
Priority: P5 - None
Component: Samba
Assignee: samba-maintainers@xxxxxxx
Reporter: ddiss@xxxxxxxx
QA Contact: samba-maintainers@xxxxxxx
Found By: ---
Blocker: ---

Encountered at https://build.opensuse.org/request/show/451654 - Ceph recently
removed a number of CephFS API functions via:
commit ce8c6f3c9f9f35b7458ab95d9affc9ed4712d0eb
Author: Jeff Layton <jlayton@xxxxxxxxxx>
Date: Mon Oct 24 10:02:58 2016 -0400

client: remove ceph_stat, ceph_lstat, ceph_fstat, ceph_setattr, and
ceph_fsetattr

...there are no more callers.

Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>

A corresponding commit was pushed to the Samba repository, to allow samba to
use the new CephFS API functions when available:

commit a9c5be394da4f20bcfea7f6d4f5919d5c0f90219
Author: Jeff Layton <jlayton@xxxxxxxxx>
Date: Thu Nov 17 13:13:57 2016 -0500

VFS: convert to using ceph_statx structures and functions, when available

Add a configure test for the ceph_statx function, and use that to
determine whether to compile in new functions that use it and its
variants, or whether to use a the older code that fetches birthtimes
from an xattr.

For cephwrap_lstat, we can use ceph_statx with the AT_SYMLINK_NOFOLLOW
flag to get the right lookup semantics.

For setting the times via cephwrap_ntimes, We can just use ceph_setattrx
and pass them all in at the same time.

Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx>
Reviewed-by: Jeremy Allison <jra@xxxxxxxxx>

--
You are receiving this mail because:
You are on the CC list for the bug.
< Previous Next >