Skip to content

Add systctl support for services#1754

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:add_sysctls_for_services
Mar 19, 2019
Merged

Add systctl support for services#1754
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:add_sysctls_for_services

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

cli changes for moby/moby#37701

@thaJeztah

Copy link
Copy Markdown
Member Author

ping @dperny @silvin-lubecki @vdemeester ptal

@thaJeztah thaJeztah added this to the 18.09.4 milestone Mar 19, 2019
@thaJeztah thaJeztah force-pushed the add_sysctls_for_services branch 2 times, most recently from 6327aa3 to bb72d64 Compare March 19, 2019 01:46
@codecov-io

codecov-io commented Mar 19, 2019

Copy link
Copy Markdown

Codecov Report

Merging #1754 into master will increase coverage by 0.05%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           master    #1754      +/-   ##
==========================================
+ Coverage   56.27%   56.32%   +0.05%     
==========================================
  Files         307      307              
  Lines       21151    21163      +12     
==========================================
+ Hits        11902    11920      +18     
+ Misses       8382     8378       -4     
+ Partials      867      865       -2

@silvin-lubecki silvin-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 👍

@albers albers 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.

Completion LGTM. Flag annotation looks odd.

Comment thread cli/command/service/create.go
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the add_sysctls_for_services branch from bb72d64 to f620349 Compare March 19, 2019 12:33

@vdemeester vdemeester 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 🐯

@thaJeztah thaJeztah merged commit 9a39a10 into docker:master Mar 19, 2019
@thaJeztah thaJeztah deleted the add_sysctls_for_services branch March 19, 2019 23:17
@andrewhsu andrewhsu modified the milestones: 18.09.4, 19.03.0 Mar 27, 2019
Sign up for free to 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.

7 participants