Add ulimits support to docker service and docker stack deploy (carry 2660)#2712
Add ulimits support to docker service and docker stack deploy (carry 2660)#2712tiborvass merged 3 commits intodocker:masterfrom
Conversation
Signed-off-by: Sebastiaan van Stijn <[email protected]>
This is related to moby/moby 40639. Signed-off-by: Albin Kerouanton <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]>
| --ulimit-add | ||
| --ulimit-rm |
There was a problem hiding this comment.
Need to update the zsh script as well, but might do so in a follow-up
| --ulimit-add ulimit Add or update a ulimit option (default []) | ||
| --ulimit-rm list Remove a ulimit option |
There was a problem hiding this comment.
Because one uses a regular "list" and the other uses a ulimit option, the output for these flags is inconsistent (ulimit vs list); probably something we should fix somehow; not sure what's best
Codecov Report
|
This is related to moby/moby 40639. Signed-off-by: Albin Kerouanton <[email protected]> Signed-off-by: Sebastiaan van Stijn <[email protected]>
8d4ec82 to
9409079
Compare
albers
left a comment
There was a problem hiding this comment.
Bash completion LGM.
I'll create a follow-up PR that adds completion for the limits.
|
Hi team, |
|
@vindhya2g You need to upgrade docker to 20.10. |
|
Thank you |
carries #2660
closes #2660
- What I did
Bump docker/docker to my own fork to have API changes regarding ulimits support on service endpoints ;--ulimittodocker service create;--ulimit-addand--ulimit-rmoptions todocker service update;Ulimitstodocker service inspect --pretty;docker stack deploy;This is related to moby/moby#40639.
- How I did it
- How to verify it
Given the following
docker-compose.yaml:docker service createdocker service updatedocker service inspectdocker stack deploy- Description for the changelog
Add
ulimitssupport todocker service create|update|inspectanddocker stack deploy- A picture of a cute animal (not mandatory but encouraged)