Bug 15716 - Failure seen with smb2.charset testsuite for ceph and glusterfs vfs modules
Summary: Failure seen with smb2.charset testsuite for ceph and glusterfs vfs modules
Status: NEW
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: VFS Modules (show other bugs)
Version: 4.21.0
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Samba QA Contact
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-09-17 18:51 UTC by Anoop C S
Modified: 2024-09-17 18:51 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anoop C S 2024-09-17 18:51:04 UTC
Following failure is seen with either vfs_ceph or vfs_glusterfs modules loaded from 'vfs objects' list.

test: samba3.smb2.charset.Testing wide-a
time: 2024-09-17 03:01:45.227688Z
time: 2024-09-17 03:01:45.344728Z
failure: samba3.smb2.charset.Testing wide-a [
Exception: ../../source4/torture/smb2/charset.c:214: status was NT_STATUS_OK, expected NT_STATUS_OBJECT_NAME_COLLISION: Failed to create wide-A

]

Additional notes:
The specific test used to pass all this while until the following commit got merged.
# git bisect good
deeca8ff176e7e9082cc0fcf48d7f981de2523c5 is the first bad commit
commit deeca8ff176e7e9082cc0fcf48d7f981de2523c5
Author: Ralph Boehme <slow@samba.org>
Date:   Thu Jun 6 15:38:16 2024 +0200

    smbd: consolidate fs capabilities code in vfswrap_fs_capabilities()
    
    This ensures the values we return via SMB_FS_ATTRIBUTE_INFORMATION is the same
    we use internally via conn->fs_capabilities.
    
    This deliberately preserves existing behaviour as much as possible and leaves
    possible improvements as a future excercize. Particularily FILE_VOLUME_QUOTAS is
    already set insided SMB_VFS_STATVFS() depending on backend filesystem flags
    which is probably the correct way to do it instead of just setting the
    capability when Samba was built with quota support.
    
    Signed-off-by: Ralph Boehme <slow@samba.org>
    Reviewed-by: David Disseldorp <ddiss@samba.org>