Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 25, 2025

These were exported in f60369d to be used in docker enterprise, but this never happened, and there's no known consumers of these, so we should deprecate these. External consumers can still call the API-client directly, which should've been the correct thing to do in the first place.

This deprecates:

  • RunConfigCreate and CreateOptions
  • RunConfigInspect and InspectOptions
  • RunConfigList and ListOptions
  • RunConfigRemove and RemoveOptions

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

Go SDK: cli/command/config: deprecate `RunConfigCreate`,  `CreateOptions`, `RunConfigInspect`, `InspectOptions`, `RunConfigList`, `ListOptions`, `RunConfigRemove`, and `RemoveOptions`.

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah thaJeztah added this to the 29.0.0 milestone Aug 25, 2025
@thaJeztah thaJeztah added impact/changelog impact/deprecation status/2-code-review area/swarm kind/refactor PR's that refactor, or clean-up code area/go-sdk Changes affecting the Go SDK impact/go-sdk Noteworthy (compatibility changes) in the Go SDK process/cherry-pick/28.x labels Aug 25, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 66.12903% with 21 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/config/create.go 68.18% 7 Missing ⚠️
cli/command/config/inspect.go 61.11% 6 Missing and 1 partial ⚠️
cli/command/config/ls.go 68.75% 5 Missing ⚠️
cli/command/config/remove.go 66.66% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

These were exported in f60369d to be
used in docker enterprise, but this never happened, and there's no
known consumers of these, so we should deprecate these. External
consumers can still call the API-client directly, which should've
been the correct thing to do in the first place.

This deprecates:

- `RunConfigCreate` and  `CreateOptions`
- `RunConfigInspect` and `InspectOptions`
- `RunConfigList` and `ListOptions`
- `RunConfigRemove` and `RemoveOptions`

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Copy link
Contributor

@ctalledo ctalledo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ...

@thaJeztah thaJeztah requested review from Benehiko and vvoland August 26, 2025 09:34
@thaJeztah thaJeztah merged commit fda64eb into docker:master Aug 26, 2025
162 of 164 checks passed
@thaJeztah thaJeztah deleted the deprecate_exported_config_funcs branch August 26, 2025 11:17
@thaJeztah
Copy link
Member Author

Also rebased the follow-up (for master / v29 only) and moved it out of draft #6370

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/go-sdk Changes affecting the Go SDK area/swarm impact/changelog impact/deprecation impact/go-sdk Noteworthy (compatibility changes) in the Go SDK kind/refactor PR's that refactor, or clean-up code process/cherry-picked status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants