-
Notifications
You must be signed in to change notification settings - Fork 187
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
oci: Skip symlinks found in upstream artifacts #1246
Conversation
Signed-off-by: Stefan Prodan <[email protected]>
Do not error out when upstream artifacts contain symlinks in the content layer, instead skip all symlinks during decompression. Signed-off-by: Stefan Prodan <[email protected]>
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.
@stefanprodan thank you and appreciate you picking it up. With a great intention to contribute this fix, I got occupied with few other commitments. This change make sense and aligns with fluxcd/pkg#655 nicely. Thank you!
@rashedkvm if you have time, could you please test this with a container image that was failing before? |
Yep, I will test it and share my finding. |
Validated fix using image with sym-link. Validation passed. Thanks @stefanprodan
|
Successfully created backport PR for |
I'm currently having this same issue on v2.12. terraform-oci oci://ghcr.io/tmayweather/manifests/terraform False failed to extract layer contents from artifact: tar file entry bin/arch contained unsupported file type Lrwxrwxrwx 15m |
Do not error out when upstream artifacts contain symlinks in the contents of the tar+gzip layer, instead skip all symlinks during decompression.
This allows Flux users to make use of OCI artifacts that may have symlinks in their contents which is the case when artifacts are created by other means than
flux push artifact
.Ref: fluxcd/pkg#655
Fix: fluxcd/pkg#538
Fix: fluxcd/flux2#4209