Skip to content

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented May 20, 2025

(This depends on #587 being merged.)

Also fixes #546
Signed-off-by: Aleksa Sarai [email protected]

@cyphar cyphar added this to the 0.6.0 milestone May 20, 2025
@codecov-commenter
Copy link

codecov-commenter commented May 20, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 40.90909% with 26 lines in your changes missing coverage. Please review.

Project coverage is 73.68%. Comparing base (36a1d10) to head (486a6a5).
Report is 16 commits behind head on main.

Files with missing lines Patch % Lines
oci/layer/tar_extract.go 40.90% 19 Missing and 7 partials ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #588      +/-   ##
==========================================
+ Coverage   72.58%   73.68%   +1.09%     
==========================================
  Files          68       69       +1     
  Lines        5424     5536     +112     
==========================================
+ Hits         3937     4079     +142     
+ Misses       1104     1072      -32     
- Partials      383      385       +2     
Files with missing lines Coverage Δ
oci/layer/tar_extract.go 57.04% <40.90%> (+0.73%) ⬆️

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cyphar
Copy link
Member Author

cyphar commented May 20, 2025

/cc @hallyn @tych0 @rchincha

This is my minimal implementation of project-stacker/umoci@84c2296 but without the checking of existing lowerdirs (which I don't think is necessary and isn't really doable until we do #574 -- so as far as I can see, project-stacker/umoci@0c6a585 is not necessary either).

@cyphar cyphar force-pushed the overlayfs-opaque-convert branch from 2f6dcea to a2c4f62 Compare May 21, 2025 01:46
tych0
tych0 previously approved these changes May 21, 2025
cyphar added 2 commits May 21, 2025 12:29
If a layer contains a regular whiteout followed by entries underneath
said path, the overlayfs-friendly behaviour is to convert the regular
whiteout to an opaque whiteout so that the whiteout semantics are
correct if the lowerdir has a non-empty directory at that path.

Signed-off-by: Aleksa Sarai <[email protected]>
@cyphar cyphar dismissed tych0’s stale review May 21, 2025 02:29

The merge-base changed after approval.

@cyphar cyphar force-pushed the overlayfs-opaque-convert branch from a2c4f62 to 486a6a5 Compare May 21, 2025 02:29
@cyphar cyphar marked this pull request as ready for review May 21, 2025 02:29
@cyphar cyphar merged commit 5a2921b into opencontainers:main May 21, 2025
18 checks passed
@cyphar cyphar deleted the overlayfs-opaque-convert branch May 21, 2025 03:35
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.

bug: extraction of path containing non-directory fails
3 participants