Skip to content

Commit c4fda95

Browse files
committed
api/types/plugin: deprecate Config.DockerVersion field
Signed-off-by: Sebastiaan van Stijn <[email protected]>
1 parent 4302ae9 commit c4fda95

7 files changed

Lines changed: 35 additions & 11 deletions

File tree

api/docs/CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,9 @@ keywords: "API, Docker, rcli, REST, documentation"
7474
continues returning these fields when set for informational purposes, but
7575
they should not be depended on as they will be omitted once the legacy builder
7676
is removed.
77+
* Deprecated: the `Config.DockerVersion` field returned by the `GET /plugins`
78+
and `GET /images/{name}/json` endpoints is deprecated. The field is no
79+
longer set, and is omitted when empty.
7780

7881
## v1.50 API changes
7982

api/docs/v1.51.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3142,10 +3142,15 @@ definitions:
31423142
- Args
31433143
properties:
31443144
DockerVersion:
3145-
description: "Docker Version used to create the plugin"
3145+
description: |-
3146+
Docker Version used to create the plugin.
3147+
3148+
Depending on how the plugin was created, this field may be empty or omitted.
3149+
3150+
Deprecated: this field is no longer set, and will be removed in the next API version.
31463151
type: "string"
31473152
x-nullable: false
3148-
example: "17.06.0-ce"
3153+
x-omitempty: true
31493154
Description:
31503155
type: "string"
31513156
x-nullable: false

api/docs/v1.52.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3203,10 +3203,15 @@ definitions:
32033203
- Args
32043204
properties:
32053205
DockerVersion:
3206-
description: "Docker Version used to create the plugin"
3206+
description: |-
3207+
Docker Version used to create the plugin.
3208+
3209+
Depending on how the plugin was created, this field may be empty or omitted.
3210+
3211+
Deprecated: this field is no longer set, and will be removed in the next API version.
32073212
type: "string"
32083213
x-nullable: false
3209-
example: "17.06.0-ce"
3214+
x-omitempty: true
32103215
Description:
32113216
type: "string"
32123217
x-nullable: false

api/swagger.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3203,10 +3203,15 @@ definitions:
32033203
- Args
32043204
properties:
32053205
DockerVersion:
3206-
description: "Docker Version used to create the plugin"
3206+
description: |-
3207+
Docker Version used to create the plugin.
3208+
3209+
Depending on how the plugin was created, this field may be empty or omitted.
3210+
3211+
Deprecated: this field is no longer set, and will be removed in the next API version.
32073212
type: "string"
32083213
x-nullable: false
3209-
example: "17.06.0-ce"
3214+
x-omitempty: true
32103215
Description:
32113216
type: "string"
32123217
x-nullable: false

api/types/plugin/plugin.go

Lines changed: 5 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

daemon/pkg/plugin/backend_linux.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -707,7 +707,7 @@ func (pm *Manager) CreateFromContext(ctx context.Context, tarCtx io.ReadCloser,
707707
DiffIds: []string{rootFSBlob.Digest().String()},
708708
}
709709

710-
config.DockerVersion = dockerversion.Version
710+
config.DockerVersion = dockerversion.Version //nolint:staticcheck // ignore SA1019: field is deprecated.
711711

712712
configBlob, err := pm.blobStore.Writer(ctx, content.WithRef(name+"-config.json"))
713713
if err != nil {

vendor/github.com/moby/moby/api/types/plugin/plugin.go

Lines changed: 5 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)