Skip to content

d/graphdriver/copy: support src fs w/o xattr support#45463

Merged
cpuguy83 merged 1 commit intomoby:masterfrom
corhere:copy-xattr-notsupp
May 4, 2023
Merged

d/graphdriver/copy: support src fs w/o xattr support#45463
cpuguy83 merged 1 commit intomoby:masterfrom
corhere:copy-xattr-notsupp

Conversation

@corhere
Copy link
Contributor

@corhere corhere commented May 4, 2023

Treat copying extended attributes from a source filesystem which does not support extended attributes as a no-op, same as if the file did not possess the extended attribute. Only fail copying extended attributes if the source file has the attribute and the destination filesystem does not support xattrs.

cc @ChenQi1989

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@corhere corhere added area/storage Image Storage kind/bugfix PR's that fix bugs labels May 4, 2023
@corhere corhere requested review from AkihiroSuda and thaJeztah May 4, 2023 16:29
Treat copying extended attributes from a source filesystem which does
not support extended attributes as a no-op, same as if the file did not
possess the extended attribute. Only fail copying extended attributes if
the source file has the attribute and the destination filesystem does
not support xattrs.

Signed-off-by: Cory Snider <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ChenQi1989
Copy link

@corhere Thanks for the fix 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vfs storage driver does not work on NFS

4 participants