Skip to content

Conversation

@nicolas-boussuge-fintech

Fixes #4340 🛠️

@google-cla
Copy link

google-cla bot commented Jan 9, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@nicolas-boussuge-fintech nicolas-boussuge-fintech marked this pull request as ready for review January 10, 2025 15:36
@nicolas-boussuge-fintech
Copy link
Author

@chanseokoh I allowed myself to create this PR, is it ok with you ?

@chanseokoh
Copy link
Member

I am not the repo maintainer, sorry.

@nicolas-boussuge-fintech
Copy link
Author

@mpeddada1 are you or do you know a maintainer for this project ?

@nicolas-boussuge-fintech
Copy link
Author

@ldetmer can you review this ? I am having difficulties using the latest debian images as source images.

@tlimin-glb
Copy link

Hello @ldetmer, @mpeddada1 , @suztomo, @blakeli0
I see that you have recently reviewed some PRs in this project. Could you review this one please?

@Jonathan-Diaz-Rosa
Copy link

I facing the exactly same issue that should be corrected by this PR, anyone for review this one please ?

@vvanroye-payplug
Copy link

I don't understand why this fix is not merged.

I drop a way to patch your Base Image before using JIB.

      skopeo login $REGISTRY
      skopeo copy docker://$DOCKER_IMAGE_TARGET oci-archive:image.tar
      mkdir -p image-patched
      tar -xf image.tar -C image-patched
      MANIFEST_DIGEST=$(jq -r '.manifests[0].digest' image-patched/index.json | cut -d: -f2)
      yq -i 'del(.config.data)' image-patched/blobs/sha256/$MANIFEST_DIGEST
      tar -cf image-patched.tar -C image-patched .
      skopeo copy oci-archive:image-patched.tar docker://$DOCKER_IMAGE_TARGET

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.

Content descriptor with optional embedded data field breaks manifest decoding

6 participants