Skip to content

Remove Kubernetes commands from experimental - #1068

Merged
vdemeester merged 1 commit into
docker:masterfrom
silvin-lubecki:remove-experimental-for-orchestrator
May 28, 2018
Merged

Remove Kubernetes commands from experimental#1068
vdemeester merged 1 commit into
docker:masterfrom
silvin-lubecki:remove-experimental-for-orchestrator

Conversation

@silvin-lubecki

Copy link
Copy Markdown
Contributor

- What I did
I removed the experimentalCli annotations and all the experimental code linked to the Kubernetes features.

- How to verify it

# Make sure the CLI is not experimental
$ docker version
Client:
...
Experimental: false
# Check the help of stack commands with kubernetes orchestrator, it should list some kubernetes flags $ docker --orchestrator=kubernetes stack --help
...
--kubeconfig string Kubernetes config file

- Description for the changelog

  • Remove Kubernetes commands from experimental

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

⚠️ Needs #1023#1035#1059 to be merged before that.

@vdemeestervdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🐯
Let's wait for the required PR to get merge before merging this one though 👼

@silvin-lubeckisilvin-lubecki changed the title [WIP] Remove Kubernetes commands from experimentalRemove Kubernetes commands from experimentalMay 28, 2018
@silvin-lubecki
silvin-lubeckiforce-pushed the remove-experimental-for-orchestrator branch from 271acef to 72fdc36CompareMay 28, 2018 10:25
@silvin-lubecki

Copy link
Copy Markdown
ContributorAuthor

Rebased

},
{
doc: "kubernetesConfigFile",
configfile: `{

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like there's a couple more "experimental": "enabled" below; think those can be removed as well?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

PTAL

Signed-off-by: Silvin Lubecki <silvin.lubecki@docker.com>
@silvin-lubecki
silvin-lubeckiforce-pushed the remove-experimental-for-orchestrator branch from 72fdc36 to f2b75a8CompareMay 28, 2018 13:06

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@vdemeester
vdemeester merged commit 824a0b4 into docker:masterMay 28, 2018
@GordonTheTurtleGordonTheTurtle added this to the 18.06.0 milestone May 28, 2018
@silvin-lubecki
silvin-lubecki deleted the remove-experimental-for-orchestrator branch May 30, 2018 14:27
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@silvin-lubecki@vdemeester@thaJeztah@mat007@GordonTheTurtle