[LU-4293] lfs_migrate is failing with a volatile file Operation not permitted error

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.6.0, Lustre 2.5.1
• Affects Version/s: Lustre 2.4.0, Lustre 2.4.1
• Environment: Lustre 2.4.1 RHEL6 2.6.32-358.18.1.el6_lustre.x86_64
• Severity: 4

Epic

lfs_migrate

• Assignee: Bruno Faccini (Inactive)
• Reporter: Wolfgang Baudler

Description

"lfs_migrate -y" aborts with this error

cannot swap layouts between <filename> and a volatile file (Operation not permitted)

This seems to happen for all files. The lfs_migrate operation aborts on the first file.

Issue Links

• is related to: LU-3696 sanity test_17m, test_17n: e2fsck unattached inodes failure
• is related to: LU-2445 add "lfs migrate" support
• is related to: LU-3826 copytool restore should set owner of volatile file before copy

Comments

[Peter Jones]

Landed for 2.5.1 and 2.6

[Bob Glossman (Inactive)]

backport to b2_5:
http://review.whamcloud.com/9278

[Anthony Alba]

I have also observed this on a filesystem created with 2.1.x and migrated to 2.4.2.
Exactly the same error message but this happens only with some directories.
Some directories lfs_migrate'd perfectly.

[Bruno Faccini (Inactive)]

My patch at http://review.whamcloud.com/8737 seems to address the original need (allow legitimate IGIFs, handle root special-case, ...) for this ticket now, only have to answer reviewers comments.

[Jian Yu]

Patch http://review.whamcloud.com/8616 was cherry-picked to Lustre b2_5 branch.

[Andreas Dilger]

I wonder if the -EPERM error seen on non-IGIF files is due to some file ownership problem like LU-3826 or similar?

        if ((attr1->la_uid != attr2->la_uid) ||
            (attr1->la_gid != attr2->la_gid))
                RETURN(-EPERM);

This will fail if lfs_migrate is not creating the file with the same ownership as the original file? If lfs_migrate is running as root, then it should be able to fchown() the file after it is created. The actual permissions don't matter, because the temporary file will be deleted, but this is proof that the caller of the migrate has permission to do this swap.

[Bruno Faccini (Inactive)]

Patch to allow layout swap for IGIF file is at http://review.whamcloud.com/8737.