FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(archive): handle whiteout ancestors during root path resolution (#38) by dinoallo · Pull Request #39 · labring/containerd · GitHub

fix(archive): handle whiteout ancestors during root path resolution (#38) - #39

Merged
dinoallo merged 1 commit into
labring:release/1.7+labringfrom
dinoallo:release-1.7-labring-pr38
Aug 3, 2026
Merged

fix(archive): handle whiteout ancestors during root path resolution (#38)#39
dinoallo merged 1 commit into
labring:release/1.7+labringfrom
dinoallo:release-1.7-labring-pr38

Conversation

dinoallo commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Background

Backport #38 from v1.7 to release/1.7+labring. Archive apply can fail with ENOTDIR before mkparent runs because fs.RootPath resolves parent paths first and can hit an overlay whiteout ancestor.

Changes

  • Cherry-pick 5a36943 from v1.7.
  • Retry parent path resolution after replacing an overlay whiteout ancestor.
  • Skip delayed directory mtime replay when the directory was removed or replaced.
  • Ignore lower-parent ENOTDIR during directory attribute copy-up.

Risks / Notes

Only overlay whiteout character devices are replaced. Regular files, symlinks, and non-whiteout device nodes still fail as non-directory parents. This PR does not include a release tag.

…ontainerd#38)

* fix(archive): handle whiteout ancestors during root path resolution

* fix(archive): skip mtime replay for replaced dirs

(cherry picked from commit 5a36943)
dinoallo merged commit efc3397 into labring:release/1.7+labring Aug 3, 2026
15 of 18 checks passed
dinoallo deleted the release-1.7-labring-pr38 branch August 3, 2026 10:01
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL