]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
Revert "quincy: ceph_fs.h: add separate owner_{u,g}id fields" 54108/head
authorVenky Shankar <vshankar@redhat.com>
Thu, 19 Oct 2023 17:08:42 +0000 (22:38 +0530)
committerVenky Shankar <vshankar@redhat.com>
Thu, 19 Oct 2023 17:13:38 +0000 (22:43 +0530)
commit71a526c01a76d29b3289f2c3601a8643a5c18ba5
tree087efa26c7790494c2fa8323721e8abbd30176fd
parent33809aa16833daf9cbc6787158e533522f8e6233
Revert "quincy: ceph_fs.h: add separate owner_{u,g}id fields"

This seems to be causing test failures:

        https://pulpito.ceph.com/yuriw-2023-10-19_16:09:31-smoke-quincy-release-distro-default-smithi/7432399

Where the MDS crashes with

In function 'CInode* Server::prepare_new_inode(MDRequestRef&, CDir*, inodeno_t, unsigned int, const file_layout_t*)'
3449: FAILED ceph_assert(_inode->gid != (unsigned)-1)

1: (ceph::__ceph_assert_fail(char const*, char const*, int, char const*)+0x14f) [0x7f991631c89e]
2: /usr/lib/ceph/libceph-common.so.2(+0x27eab0) [0x7f991631cab0]
3: (Server::prepare_new_inode(boost::intrusive_ptr<MDRequestImpl>&, CDir*, inodeno_t, unsigned int, file_layout_t const*)+0x2791) [0x55563bec4831]
4: (Server::handle_client_mkdir(boost::intrusive_ptr<MDRequestImpl>&)+0x1ec) [0x55563bede88c]

Revert this change till the failure is fixed.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
src/client/Client.cc
src/client/MetaRequest.cc
src/client/MetaRequest.h
src/include/ceph_fs.h
src/mds/Server.cc
src/mds/cephfs_features.cc
src/mds/cephfs_features.h
src/messages/MClientRequest.h