Bug 2248639

Summary: cephfs-mirror: ceph_getxattr call always return -61 (ENODATA) in PeerReplayer::synchronize
Product: [Red Hat Storage] Red Hat Ceph Storage Reporter: Jos Collin <jcollin>
Component: CephFSAssignee: Jos Collin <jcollin>
Status: CLOSED ERRATA QA Contact: Hemanth Kumar <hyelloji>
Severity: medium Docs Contact: Akash Raj <akraj>
Priority: unspecified    
Version: 7.0CC: akraj, ceph-eng-bugs, cephqe-warriors, hyelloji, ngangadh, tserlin, vshankar
Target Milestone: ---   
Target Release: 7.1   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: ceph-18.2.1-5.el9cp Doc Type: Bug Fix
Doc Text:
.cephfs-mirroring overall performance is improved With this fix, the incremental snapshot sync is corrected, which improves the overall performance of cephfs-mirroring.
 Story Points: ---
Clone Of: 2248637 Environment:
Last Closed: 2024-06-13 14:22:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2248637    
Bug Blocks: 2267614, 2298578, 2298579    

Description Jos Collin 2023-11-08 05:26:17 UTC 
+++ This bug was initially created as a clone of Bug #2248637 +++

Description of problem:
ceph_getxattr() call always return -61 (ENODATA) in PeerReplayer::synchronize, which results in 'prev' snapshot always coming in as boost::none in PeerReplayer::do_synchronize function. Even if the ceph_fsetxattr seems successfully setting (returns 0) the 'ceph.mirror.dirty_snap_id' in the first pass, the ceph_getxattr() call always returns ENODATA in the subsequent passes (and in further snapshot creations) for the same dir_root. It never retrieves the xattr value from the dir_root. This prevents incremental snapshot synchronisation, as the `prev` snapshot is always coming in as none. 

Version-Release number of selected component (if applicable):
7.0
How reproducible:
Always

Steps to Reproduce:
1. Create a snapshot.
2. Make some changes to the dirs/files.
3. Create a second snapshot. Then it won't sync incrementally, as 'prev' is always none.

Actual results:
'prev' snapshot is none, even if it exist.

Expected results:
'prev' snapshot should have it's address if it exist.

Additional info:
As the prev snapshot is always none, this issue would have a performance impact on snapshot synchronisation.
Comment 10 errata-xmlrpc 2024-06-13 14:22:59 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Critical: Red Hat Ceph Storage 7.1 security, enhancements, and bug fix update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2024:3925