Skip to content

[27.x backport] daemon/containerd: getPushDescriptor: fix formatting of platform in errors#48632

Closed
thaJeztah wants to merge 1 commit intomoby:27.xfrom
thaJeztah:27.x_backport_improve_multiarch_errors
Closed

[27.x backport] daemon/containerd: getPushDescriptor: fix formatting of platform in errors#48632
thaJeztah wants to merge 1 commit intomoby:27.xfrom
thaJeztah:27.x_backport_improve_multiarch_errors

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

The platform was printed in its raw format, which didn't produce a very readable output;

Before this change:

$ docker image save --platform=linux/amd64 -o alpine_amd64.tar alpine:latest
Error response from daemon: no suitable export target found for platform linux/amd64: no suitable image manifest found for platform {amd64 linux  [] }

After this change:

$ docker image save --platform=linux/amd64 -o alpine_amd64.tar alpine:latest
Error response from daemon: no suitable export target found for platform linux/amd64: no suitable image manifest found for platform linux/amd64

(cherry picked from commit ad24262)

…rrors

The platform was printed in its raw format, which didn't produce a very
readable output;

Before this change:

    $ docker image save --platform=linux/amd64 -o alpine_amd64.tar alpine:latest
    Error response from daemon: no suitable export target found for platform linux/amd64: no suitable image manifest found for platform {amd64 linux  [] }

After this change:

    $ docker image save --platform=linux/amd64 -o alpine_amd64.tar alpine:latest
    Error response from daemon: no suitable export target found for platform linux/amd64: no suitable image manifest found for platform linux/amd64

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit ad24262)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review kind/bugfix PR's that fix bugs containerd-integration Issues and PRs related to containerd integration area/ux labels Oct 10, 2024
@thaJeztah thaJeztah added this to the 27.4.0 milestone Oct 10, 2024
@thaJeztah thaJeztah self-assigned this Oct 10, 2024
@thaJeztah thaJeztah marked this pull request as ready for review October 11, 2024 07:35
@thaJeztah thaJeztah marked this pull request as draft October 11, 2024 15:21
@thaJeztah
Copy link
Copy Markdown
Member Author

It looks like back porting #48631 is slightly more complicated because related changes around errPlatformNotFound were added in 23d565c, as part of a larger refactor.

@thaJeztah
Copy link
Copy Markdown
Member Author

@thaJeztah thaJeztah removed status/2-code-review kind/bugfix PR's that fix bugs containerd-integration Issues and PRs related to containerd integration area/ux labels Oct 18, 2024
@thaJeztah thaJeztah removed this from the 27.4.0 milestone Oct 18, 2024
@thaJeztah thaJeztah closed this Oct 18, 2024
@thaJeztah thaJeztah deleted the 27.x_backport_improve_multiarch_errors branch October 18, 2024 15:06
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