Skip to content

e2e: load to docker store#1460

Merged
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:docker-load
Dec 15, 2022
Merged

e2e: load to docker store#1460
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:docker-load

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max
Copy link
Copy Markdown
Member Author

crazy-max commented Dec 11, 2022

https://github.com/docker/buildx/actions/runs/3670274233/jobs/6204713067#step:11:205

#12 exporting to docker image format
#12 ERROR: docker exporter does not currently support exporting manifest lists
------
 > exporting to docker image format:
------
ERROR: failed to solve: docker exporter does not currently support exporting manifest lists

@jedevc I think we need changes here to properly handle multi-platform for OCI exporter like we do for the image one.

@jedevc
Copy link
Copy Markdown
Collaborator

jedevc commented Dec 11, 2022

@crazy-max I think this should be solved by moby/buildkit#3380.

It's a result of buildx attempting to attach provenance attestations by default, but, we need to filter out the inline attestations for the docker exporter just like we do for the local and tar exporter.

@crazy-max
Copy link
Copy Markdown
Member Author

@jedevc Nice, didn't catch that PR.

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max marked this pull request as ready for review December 13, 2022 23:15
@crazy-max crazy-max merged commit a49d28e into docker:master Dec 15, 2022
@crazy-max crazy-max deleted the docker-load branch December 15, 2022 09:35
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.

2 participants