Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[draft][19.03 backport] Remove "docker engine" subcommands #2829

Closed

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Nov 5, 2020

Backport of #2207
alternative to / closes #2828

Opening this for discussion; These subcommands were created to allow upgrading a Docker Community engine to Docker Enterprise, but with the move of the enterprise products to Mirantis, these commands were no longer working; the images used to distribute the "enterprise" engine are no longer updated, and the latest versions available on docker hub are outdated (and may have unpatched vulnerabilities, so should not be recommended to use).

Reference documentation for these subcommands were already removed from docs.docker.com, and removing these commands (even as part of a patch release) may be the best option.

addresses #2213

These subcommands were created to allow upgrading a Docker Community
engine to Docker Enterprise, but never really took off.

This patch removes the `docker engine` subcommands, as they added
quite some complexity / additional code.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 43b2f52)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 7e54513)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

hm. CircleCI for some reason isn't triggered on this PR (Jenkins is though)

@thaJeztah thaJeztah closed this Jan 7, 2023
@thaJeztah thaJeztah deleted the 19.03_backport_remove_engine_activate branch January 7, 2023 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