[LU-5017] "lfs mv" loses file attributes during migration

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.6.0
• Fix Version/s: Lustre 2.6.0, Lustre 2.8.0
• Type: Bug
• Priority: Major
• Reporter: Andreas Dilger
• Assignee: Di Wang
• Resolution: Fixed
• Labels: dne2
• Severity: 3

Description

Migrating a directory tree using "lfs mv" drops the file attributes stored on both files and directories.

For example, setting the "noatime" attribute on a directory and a file therein, and then migrating the directory to another MDT clears this flag:

[root@sookie-gig tests]# lsattr -a /mnt/testfs/d0.new
---------------- /mnt/testfs/d0.new/.
---------------- /mnt/testfs/d0.new/..
---------------- /mnt/testfs/d0.new/f0.new
[root@sookie-gig tests]# chattr +SAD /mnt/testfs/d0.new /mnt/testfs/d0.new/f0.new
[root@sookie-gig tests]# lsattr -a /mnt/testfs/d0.new
--SD---A-------- /mnt/testfs/d0.new/.
---------------- /mnt/testfs/d0.new/..
--S----A-------- /mnt/testfs/d0.new/f0.new
[root@sookie-gig tests]# lfs mv -M 1 /mnt/testfs/d0.new
[root@sookie-gig tests]# lsattr -a /mnt/testfs/d0.new
---------------- /mnt/testfs/d0.new/f0.new
---------------- /mnt/testfs/d0.new/.
---------------- /mnt/testfs/d0.new/..

Fortunately, the IMMUTABLE and APPEND flags cannot be lost in this manner because the MDS blocks files/directories with these attributes from being migrated, so this only affects secondary attributes (noatime, sync write, dirsync).

Issue Links

• is related to: LU-5016 clean up use of la_attr for non-attri...

Comments

[Andreas Dilger]

I'm not sure if this is the same as LU-5016, or if the "lfs mv" code just isn't trying to copy the file attributes at all.

[Di Wang]

http://review.whamcloud.com/10240

[Di Wang]

The patch has been landed to master

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/10274/
Subject: LU-5017 idl: clean up file attribute flags
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 26fae8692eed9998aa0286749a068b67279e7163