-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tarGenerator.AddFile: ignore llistxattr eopnotsupp #506
Conversation
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #506 +/- ##
==========================================
- Coverage 73.48% 73.45% -0.04%
==========================================
Files 60 60
Lines 4884 4886 +2
==========================================
Hits 3589 3589
- Misses 935 937 +2
Partials 360 360
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reference, we're hitting this when trying to create a new layer from a directory that contains a linux-kernel mounted squashfs image.
@cyphar not sure why I had it in draft mode until monday - it is ready for review. |
@tych0 ping |
If it's not supported, just assume there are no xattrs, don't fail the addfile. Changelog: 10/18: use errors.Cause as suggested by Tycho Signed-off-by: Serge Hallyn <[email protected]>
38739d3
to
685e414
Compare
FWIW, LGTM. 😅 |
If it's not supported, just assume there are no xattrs, don't fail the addfile.