Skip to content

[25.0 backport] builder/dockerfile: ADD with best-effort xattrs#47347

Closed
thaJeztah wants to merge 1 commit intomoby:25.0from
thaJeztah:25.0_backport_best-effort-xattrs-classic-builder
Closed

[25.0 backport] builder/dockerfile: ADD with best-effort xattrs#47347
thaJeztah wants to merge 1 commit intomoby:25.0from
thaJeztah:25.0_backport_best-effort-xattrs-classic-builder

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Archives being unpacked by Dockerfiles may have been created on other OSes with different conventions and semantics for xattrs, making them impossible to apply when extracting. Restore the old best-effort xattr behaviour users have come to depend on in the classic builder.

(cherry picked from commit ab6d760)

- 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)

Archives being unpacked by Dockerfiles may have been created on other
OSes with different conventions and semantics for xattrs, making them
impossible to apply when extracting. Restore the old best-effort xattr
behaviour users have come to depend on in the classic builder.

Signed-off-by: Cory Snider <[email protected]>
(cherry picked from commit ab6d760)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah marked this pull request as ready for review February 6, 2024 19:27
@thaJeztah thaJeztah requested a review from tonistiigi as a code owner February 6, 2024 19:27
@thaJeztah thaJeztah marked this pull request as draft February 6, 2024 20:01
@thaJeztah thaJeztah modified the milestones: 25.0.3, 25.0.4 Feb 6, 2024
@thaJeztah
Copy link
Copy Markdown
Member Author

@thaJeztah thaJeztah closed this Mar 1, 2024
@thaJeztah thaJeztah removed this from the 25.0.4 milestone Mar 1, 2024
@thaJeztah thaJeztah deleted the 25.0_backport_best-effort-xattrs-classic-builder branch March 1, 2024 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants