Skip to content
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

create-spec: fix for indexed images #212

Merged
merged 2 commits into from
Jan 16, 2024
Merged

create-spec: fix for indexed images #212

merged 2 commits into from
Jan 16, 2024

Conversation

ktock
Copy link
Owner

@ktock ktock commented Jan 16, 2024

#210

  • fix platform to be correctly recognized
  • convert only container manifest

Signed-off-by: Kohei Tokunaga <ktokunaga.mail@gmail.com>
Signed-off-by: Kohei Tokunaga <ktokunaga.mail@gmail.com>
@ktock ktock merged commit 125483b into main Jan 16, 2024
13 checks passed
@ktock
Copy link
Owner Author

ktock commented Jan 17, 2024

confirmed images containing attestation manifest don't cause panic error anymore.

$ docker buildx build --attest type=sbom --output type=oci,dest=/tmp/sout2/img.tar /tmp/ctxs
$ mkdir /tmp/ex
$ tar -C /tmp/ex -xf /tmp/sout2/img.tar 
$ buildx build --progress=plain --build-arg TARGETARCH=amd64 --build-arg TARGETPLATFORM=linux/amd64 --platform=linux/amd64 -f Dockerfile --build-context assets=. --output type=local,dest=/tmp/out --build-arg OUTPUT_NAME=out.wasm --build-arg LINUX_LOGLEVEL=0 --build-arg INIT_DEBUG=false /tmp/soutex/
...
#29 0.065 Trying to unpack image as an OCI image
#29 0.066 nested manifest: processing [{application/vnd.oci.image.manifest.v1+json sha256:608ad19966c303f5bb8ed31d2e1de9aa1bb56a9b763b09d9eaddace01ec253f3 482 [] map[] [] 0xc000076480 }]

@ktock ktock deleted the indeximages branch January 21, 2024 05:22
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.

None yet

1 participant