Skip to content

[19.03 backport] cli-plugins: add concept of experimental plugin, only enabled in experimental mode - #1905

Merged
tiborvass merged 2 commits into
docker:19.03from
thaJeztah:19.03_backport_plugin_experimental
May 23, 2019
Merged

[19.03 backport] cli-plugins: add concept of experimental plugin, only enabled in experimental mode#1905
tiborvass merged 2 commits into
docker:19.03from
thaJeztah:19.03_backport_plugin_experimental

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport of #1898 for 19.03
closes#1897 for 19.03

To test, add $(pwd)/build/plugins-linux-amd64 to "cliPluginsExtraDirs" config and run:

make plugins
make binary
HELLO_EXPERIMENTAL=1 docker helloworld

To show it enabled:

HELLO_EXPERIMENTAL=1 DOCKER_CLI_EXPERIMENTAL=enabled docker helloworld

Tibor Vass added 2 commits May 23, 2019 21:28
…rimental mode
To test, add $(pwd)/build/plugins-linux-amd64 to "cliPluginsExtraDirs" config and run:
make plugins
make binary
HELLO_EXPERIMENTAL=1 docker helloworld
To show it enabled:
HELLO_EXPERIMENTAL=1 DOCKER_CLI_EXPERIMENTAL=enabled docker helloworld
Signed-off-by: Tibor Vass <tibor@docker.com>
(cherry picked from commit 6ca8783)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Tibor Vass <tibor@docker.com>
(cherry picked from commit bb8e89b)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #1905 into 19.03 will decrease coverage by <.01%.
The diff coverage is 35.29%.

@@ Coverage Diff @@## 19.03 #1905 +/- ##
==========================================
- Coverage 56.73% 56.72% -0.01% 
==========================================
Files 309 309 Lines 21666 21678 +12 ==========================================
+ Hits 12292 12297 +5 - Misses 8477 8483 +6 - Partials 897 898 +1

@andrewhsuandrewhsu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tiborvass
tiborvass merged commit 649097f into docker:19.03May 23, 2019
@thaJeztah
thaJeztah deleted the 19.03_backport_plugin_experimental branch May 23, 2019 21:07
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.

6 participants

@thaJeztah@codecov-io@andrewhsu@tiborvass@silvin-lubecki@GordonTheTurtle