-
Notifications
You must be signed in to change notification settings - Fork 95
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
image-rs | failed to handle image layers that has long name hard link file inside #689
Comments
cc @arronwy |
The issue doesn't occur if switching back to older implementation of using sync tar, ie before commit |
Good. It means that the sync tar had implemented this, thus we could have a good teacher to learn from to promote async-tar. |
Commit 921006fe7e04241057685bba16770320f5388d81 introduced async tar for performance optimisation. However the unintended consequence of it is layers with long pathnames (100 chars) failed to unpack Ref: confidential-containers/guest-components#689 Signed-off-by: Pradipta Banerjee <pradipta.banerjee@gmail.com>
I put the fix PR on original |
Hi @Xynnn007, looks like for the last weeks the PR you created has not yet been merged. I bumped it mentioning the repository owner. In the meantime, have you possibly found a way to workaround this? |
@JakubLedworowski Thanks for raising that up in the thread. Up to now I have only two ways except waiting for upstream
Both might be difficult. Do you have any ideas? @bpradipt |
@Xynnn007 let's wait till this week to see if there is any update to the PR. Otherwise we can plan to fork the code and keep it under coco org. Option-2 is not practical imho |
Apologies for jumping in on this issue but wanted to communicate somewhere with folks :) First, I didn't know our fork was being used for Confidential Containers but that's super cool given that we are have similar goals at Edera (we aren't aiming to be confidential only, our platform provides isolation in general, with confidentiality being a feature add)! I just published release v0.4.1 of krata-tokio-tar with the necessary change, and v0.4.2 which swaps to portable-atomic for atomics, broadening platform support for krata-tokio-tar. Now that I am aware of it's use, I'd be happy to make our fork a collaborative effort. Please feel free to contact me at alex AT edera.dev if there are any questions! |
The new version of krata-tokio-tar supports long link name. Fixes confidential-containers#689 Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
Thanks to @bpradipt , we found that image-rs currently cannot pull image layers that have long name hard link file (more than 100 chars) layer. Errors like the following
Two typical images triggering this
This is not supported by underlying
krata-tokio-tar
crate. Related issue intar
(tokio-tar
is based on that) is alexcrichton/tar-rs#369The text was updated successfully, but these errors were encountered: