Project

General

Profile

Actions

Bug #65157

closed

cephfs-mirror: set layout.pool_name xattr of destination subvol correctly

Added by Milind Changire about 1 month ago. Updated 1 day ago.

Status:
Can't reproduce
Priority:
Normal
Assignee:
Category:
Correctness/Safety
Target version:
% Done:

0%

Source:
Tags:
Backport:
reef,squid
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
cephfs-mirror
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

  • mirroring doesn't handle subvols correctly
  • the layout xattr is missing on the destination subvol dir i.e. the layout.pool_name
Actions #1

Updated by Venky Shankar 16 days ago

  • Category set to Correctness/Safety
  • Assignee set to Jos Collin
  • Target version set to v20.0.0
  • Backport set to reef,squid
  • Component(FS) cephfs-mirror added
Actions #2

Updated by Jos Collin 16 days ago

  • Status changed from New to Can't reproduce

Can't reproduce this:

$ ./bin/ceph fs snapshot mirror add a /volumes/subvolgrp_a/subvol_a/
$ mkdir -p  /mnt/cephfs11/volumes/subvolgrp_a/subvol_a/.snap/snap_1
$ getfattr -n ceph.dir.layout.pool_name /mnt/cephfs11/volumes/subvolgrp_a/subvol_a/
getfattr: Removing leading '/' from absolute path names
# file: mnt/cephfs11/volumes/subvolgrp_a/subvol_a/
ceph.dir.layout.pool_name="cephfs.a.data" 

$ getfattr -n ceph.dir.layout.pool_name /mnt/cephfs12/volumes/subvolgrp_a/subvol_a/
getfattr: Removing leading '/' from absolute path names
# file: mnt/cephfs12/volumes/subvolgrp_a/subvol_a/
ceph.dir.layout.pool_name="cephfs.remotefs.data" 

@Milind, could you please help by providing your steps?

Actions #3

Updated by Milind Changire 1 day ago

I arrived at this issue only via code review.
I haven't attempted reproducing the issue.

Actions

Also available in: Atom PDF