From f8d3c4e4a7cf8c8fa9e856dd072f2c7ffd43914b Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Sun, 5 Oct 2025 17:07:04 +0200 Subject: [PATCH] api/types/image: InspectResponse: remove deprecated Parent, DockerVersion The InspectResponse type contains various fields that are deprecated and removed from current API versions, but that were kept for the API server to produce the fields when downgrading to older API versions. - The `Parent` field is only used for the legacy builder, and only set for images that are built locally (i.e., not persisted when pulling an image). - The `DockerVersion` field is only set when building images with the legacy builder, and empty in most cases. Both fields were implicitly deprecated with the deprecation of the legacy builder, and deprecated for the API in [moby@bd8a99b], which was backported to the 28.x release. This patch: - Removes the deprecated fields from the `InspectResposne` struct; this means that [`client.ImageInspect`] won't unmarshal those fields, but the [`docker image inspect`] CLI command defaults to printing the raw output as returned by the API, so can continue to show any field returned in the API response. As a side-note; we should change the CLI to default to show the unmarshalled response, and introduce a `--format=jsonraw` (or `--raw`) option to make printing the raw response opt-in. - Updates the API server to backfill the fields if they are set. [moby@bd8a99b]: https://github.com/moby/moby/commit/bd8a99b4004b0b766db7f5f79dfc2dad553b9426 [`client.ImageInspect`]: https://github.com/moby/moby/blob/f739c61c69a7155362993c2a2a624e838b3893bc/client/image_inspect.go#L14-L64 [`docker image inspect`]: https://github.com/docker/cli/blob/74e3520724d77e63ef75987cbd0c0cde507ab971/cli/command/image/inspect.go#L59-L81 Signed-off-by: Sebastiaan van Stijn --- api/types/image/image_inspect.go | 16 ----------- daemon/containerd/image_inspect.go | 2 +- daemon/images/image_inspect.go | 28 +++++++++---------- daemon/server/imagebackend/image.go | 16 +++++++++++ daemon/server/router/image/image_routes.go | 13 +++++++-- integration-cli/docker_api_build_test.go | 16 +++++++++-- .../moby/api/types/image/image_inspect.go | 16 ----------- 7 files changed, 56 insertions(+), 51 deletions(-) diff --git a/api/types/image/image_inspect.go b/api/types/image/image_inspect.go index 102b20263f..66a277e557 100644 --- a/api/types/image/image_inspect.go +++ b/api/types/image/image_inspect.go @@ -42,15 +42,6 @@ type InspectResponse struct { // the manifest is generated and its digest calculated. RepoDigests []string - // Parent is the ID of the parent image. - // - // Depending on how the image was created, this field may be empty and - // is only set for images that were built/created locally. This field - // is omitted if the image was pulled from an image registry. - // - // Deprecated: this field is deprecated, and will be removed in the next release. - Parent string `json:",omitempty"` - // Comment is an optional message that can be set when committing or // importing the image. This field is omitted if not set. Comment string `json:",omitempty"` @@ -62,13 +53,6 @@ type InspectResponse struct { // and omitted otherwise. Created string `json:",omitempty"` - // DockerVersion is the version of Docker that was used to build the image. - // - // Depending on how the image was created, this field may be omitted. - // - // Deprecated: this field is deprecated, and will be removed in the next release. - DockerVersion string `json:",omitempty"` - // Author is the name of the author that was specified when committing the // image, or as specified through MAINTAINER (deprecated) in the Dockerfile. // This field is omitted if not set. diff --git a/daemon/containerd/image_inspect.go b/daemon/containerd/image_inspect.go index dfde50e1a3..d1fc1bcf34 100644 --- a/daemon/containerd/image_inspect.go +++ b/daemon/containerd/image_inspect.go @@ -91,13 +91,13 @@ func (i *ImageService) ImageInspect(ctx context.Context, refOrID string, opts im RepoTags: repoTags, Descriptor: &target, RepoDigests: repoDigests, - Parent: parent, //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present. Size: size, Manifests: manifests, Metadata: imagetypes.Metadata{ LastTagTime: lastUpdated, }, }, + Parent: parent, // field is deprecated with the legacy builder, but returned by the API if present. } if multi.Best != nil { diff --git a/daemon/images/image_inspect.go b/daemon/images/image_inspect.go index 21bfa645a1..535bb594e2 100644 --- a/daemon/images/image_inspect.go +++ b/daemon/images/image_inspect.go @@ -56,20 +56,18 @@ func (i *ImageService) ImageInspect(ctx context.Context, refOrID string, opts im imgConfig := containerConfigToDockerOCIImageConfig(img.Config) return &imagebackend.InspectData{ InspectResponse: imagetypes.InspectResponse{ - ID: img.ID().String(), - RepoTags: repoTags, - RepoDigests: repoDigests, - Parent: img.Parent.String(), //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present (built with legacy builder). - Comment: comment, - Created: created, - DockerVersion: img.DockerVersion, //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present. - Author: img.Author, - Config: &imgConfig, - Architecture: img.Architecture, - Variant: img.Variant, - Os: img.OperatingSystem(), - OsVersion: img.OSVersion, - Size: size, + ID: img.ID().String(), + RepoTags: repoTags, + RepoDigests: repoDigests, + Comment: comment, + Created: created, + Author: img.Author, + Config: &imgConfig, + Architecture: img.Architecture, + Variant: img.Variant, + Os: img.OperatingSystem(), + OsVersion: img.OSVersion, + Size: size, GraphDriver: &storage.DriverData{ Name: i.layerStore.DriverName(), Data: layerMetadata, @@ -82,6 +80,8 @@ func (i *ImageService) ImageInspect(ctx context.Context, refOrID string, opts im LastTagTime: lastUpdated, }, }, + Parent: img.Parent.String(), // field is deprecated with the legacy builder, but still included in response when present (built with legacy builder). + DockerVersion: img.DockerVersion, // field is deprecated with the legacy builder, but still included in response when present. Container: img.Container, // field is deprecated, but still set on API < v1.45. ContainerConfig: &img.ContainerConfig, // field is deprecated, but still set on API < v1.45. }, nil diff --git a/daemon/server/imagebackend/image.go b/daemon/server/imagebackend/image.go index 22e46e6e76..b3992c44c2 100644 --- a/daemon/server/imagebackend/image.go +++ b/daemon/server/imagebackend/image.go @@ -61,6 +61,22 @@ type ImageInspectOpts struct { type InspectData struct { imagetypes.InspectResponse + // Parent is the ID of the parent image. + // + // Depending on how the image was created, this field may be empty and + // is only set for images that were built/created locally. This field + // is omitted if the image was pulled from an image registry. + // + // This field is deprecated with the legacy builder, but returned by the API if present. + Parent string `json:",omitempty"` + + // DockerVersion is the version of Docker that was used to build the image. + // + // Depending on how the image was created, this field may be omitted. + // + // This field is deprecated with the legacy builder, but returned by the API if present. + DockerVersion string `json:",omitempty"` + // Container is the ID of the container that was used to create the image. // // Depending on how the image was created, this field may be empty. diff --git a/daemon/server/router/image/image_routes.go b/daemon/server/router/image/image_routes.go index 18b1d76436..094f484f93 100644 --- a/daemon/server/router/image/image_routes.go +++ b/daemon/server/router/image/image_routes.go @@ -423,9 +423,9 @@ func (ir *imageRouter) getImagesByName(ctx context.Context, w http.ResponseWrite // These fields have "omitempty" on API v1.52 and higher, // but older API versions returned them unconditionally. legacyOptions = append(legacyOptions, compat.WithExtraFields(map[string]any{ - "Parent": inspectData.Parent, //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present (built with legacy builder). + "Parent": inspectData.Parent, // field is deprecated, but still included in response when present (built with legacy builder). "Comment": inspectData.Comment, - "DockerVersion": inspectData.DockerVersion, //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present. + "DockerVersion": inspectData.DockerVersion, // field is deprecated, but still included in response when present. "Author": inspectData.Author, })) @@ -440,6 +440,15 @@ func (ir *imageRouter) getImagesByName(ctx context.Context, w http.ResponseWrite "Config": legacyConfigFields["v1.50-v1.51"], })) } + } else { + if inspectData.Parent != "" { + // field is deprecated, but still included in response when present (built with legacy builder). + legacyOptions = append(legacyOptions, compat.WithExtraFields(map[string]any{"Parent": inspectData.Parent})) + } + if inspectData.DockerVersion != "" { + // field is deprecated, but still included in response when present. + legacyOptions = append(legacyOptions, compat.WithExtraFields(map[string]any{"DockerVersion": inspectData.DockerVersion})) + } } if len(legacyOptions) > 0 { diff --git a/integration-cli/docker_api_build_test.go b/integration-cli/docker_api_build_test.go index 506de2ca23..a0be189eef 100644 --- a/integration-cli/docker_api_build_test.go +++ b/integration-cli/docker_api_build_test.go @@ -319,10 +319,22 @@ func (s *DockerAPISuite) TestBuildOnBuildCache(c *testing.T) { // check parentID is correct // Parent is graphdriver-only if !testEnv.UsingSnapshotter() { - image, err := apiClient.ImageInspect(ctx, childID) + var buf bytes.Buffer + _, err := apiClient.ImageInspect(ctx, childID, client.ImageInspectWithRawResponse(&buf)) assert.NilError(c, err) - assert.Check(c, is.Equal(parentID, image.Parent)) //nolint:staticcheck // ignore SA1019: field is deprecated, but still included in response when present. + var image struct { + // Parent is the ID of the parent image. + // + // Depending on how the image was created, this field may be empty and + // is only set for images that were built/created locally. This field + // is omitted if the image was pulled from an image registry. + Parent string `json:",omitempty"` + } + rawResponse := buf.Bytes() + err = json.Unmarshal(rawResponse, &image) + assert.NilError(c, err, string(rawResponse)) + assert.Check(c, is.Equal(parentID, image.Parent), string(rawResponse)) } } diff --git a/vendor/github.com/moby/moby/api/types/image/image_inspect.go b/vendor/github.com/moby/moby/api/types/image/image_inspect.go index 102b20263f..66a277e557 100644 --- a/vendor/github.com/moby/moby/api/types/image/image_inspect.go +++ b/vendor/github.com/moby/moby/api/types/image/image_inspect.go @@ -42,15 +42,6 @@ type InspectResponse struct { // the manifest is generated and its digest calculated. RepoDigests []string - // Parent is the ID of the parent image. - // - // Depending on how the image was created, this field may be empty and - // is only set for images that were built/created locally. This field - // is omitted if the image was pulled from an image registry. - // - // Deprecated: this field is deprecated, and will be removed in the next release. - Parent string `json:",omitempty"` - // Comment is an optional message that can be set when committing or // importing the image. This field is omitted if not set. Comment string `json:",omitempty"` @@ -62,13 +53,6 @@ type InspectResponse struct { // and omitted otherwise. Created string `json:",omitempty"` - // DockerVersion is the version of Docker that was used to build the image. - // - // Depending on how the image was created, this field may be omitted. - // - // Deprecated: this field is deprecated, and will be removed in the next release. - DockerVersion string `json:",omitempty"` - // Author is the name of the author that was specified when committing the // image, or as specified through MAINTAINER (deprecated) in the Dockerfile. // This field is omitted if not set.