-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[WIP] Add --pids-limit
flag to docker update
#181
Conversation
Codecov Report
@@ Coverage Diff @@
## master #181 +/- ##
==========================================
- Coverage 45.67% 45.66% -0.02%
==========================================
Files 171 171
Lines 11513 11516 +3
==========================================
Hits 5259 5259
- Misses 5947 5950 +3
Partials 307 307 |
We should add this to the Compose file as well.
|
Adding PidsLimit to compose file requires swarm |
Oops, my mistake. I thought this was being added to |
@darkowlzz sorry for all the hassle; it looks like you forgot to move the documentation changes to this repository. We should wait for the changes in moby/moby upstream to be merged first |
Signed-off-by: Sunny Gogoi <indiasuny000@gmail.com>
Removed the commit related to Compose changes and updated commandline/update docs. |
Signed-off-by: Sunny Gogoi <indiasuny000@gmail.com>
Looks like the moby/moby support hasn't merged yet |
--pids-limit
flag to docker update
--pids-limit
flag to docker update
carrying in #1765 so that it may still make the cut-off for 19.03 🤗 |
- What I did
Added
pids-limit
flag to dockerupdate
command. Also added completion for bash and zsh.- Description for the changelog
Add
--pids-limit
flag todocker update
Refer: moby/moby#32519