https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2022-09-26T17:08:27ZCeph CephFS - Bug #57641: Ceph FS fscrypt clones missing fscrypt metadatahttps://tracker.ceph.com/issues/57641?journal_id=2262812022-09-26T17:08:27ZVenky Shankarvshankar@redhat.com
<ul></ul><p>Hi Marcel,</p>
<blockquote>
<p>Copying the ceph.fscrypt.auth xattr in the mgr/volumes async clone seems to work:<br /><a class="external" href="https://github.com/irq0/ceph/commit/001f5e88c3486ea370abc82546f3a8d98725090d">https://github.com/irq0/ceph/commit/001f5e88c3486ea370abc82546f3a8d98725090d</a></p>
</blockquote>
<p>Mind submitting this change as a PR?</p> CephFS - Bug #57641: Ceph FS fscrypt clones missing fscrypt metadatahttps://tracker.ceph.com/issues/57641?journal_id=2262822022-09-26T17:08:41ZVenky Shankarvshankar@redhat.com
<ul><li><strong>Target version</strong> set to <i>v18.0.0</i></li><li><strong>Backport</strong> set to <i>pacific,quincy</i></li></ul> CephFS - Bug #57641: Ceph FS fscrypt clones missing fscrypt metadatahttps://tracker.ceph.com/issues/57641?journal_id=2271972022-10-10T08:03:29ZVenky Shankarvshankar@redhat.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Fix Under Review</i></li><li><strong>Pull request ID</strong> set to <i>48410</i></li></ul><p>Hi Marcel,</p>
<p>I pushed a PR with your commit. We should probably take this opportunity to copy user xattrs during clone.</p> CephFS - Bug #57641: Ceph FS fscrypt clones missing fscrypt metadatahttps://tracker.ceph.com/issues/57641?journal_id=2272052022-10-10T10:56:48Zmarcel lauhoff
<ul></ul><p>Thanks Venky, I did not submit this right away as I'm unsure the xattr copy is the right way to do this.</p>
<p>I wonder how much of a performance hit the getxattr(source_path, "ceph.fscrypt.auth") is. This probably costs another round trip to the MDS, not?</p> CephFS - Bug #57641: Ceph FS fscrypt clones missing fscrypt metadatahttps://tracker.ceph.com/issues/57641?journal_id=2466192023-09-20T14:44:51ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Target version</strong> changed from <i>v18.0.0</i> to <i>v19.0.0</i></li><li><strong>Backport</strong> changed from <i>pacific,quincy</i> to <i>reef,quincy</i></li></ul>