Skip to content

Commit

Permalink
Merge pull request #1765 from thaJeztah/carry_181_pids_limit
Browse files Browse the repository at this point in the history
Add `--pids-limit` flag to `docker update`
  • Loading branch information
tonistiigi authored Mar 21, 2019
2 parents cb6b33f + de3a5f0 commit dca6d2a
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 0 deletions.
7 changes: 7 additions & 0 deletions cli/command/container/update.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ type updateOptions struct {
memorySwap opts.MemSwapBytes
kernelMemory opts.MemBytes
restartPolicy string
pidsLimit int64
cpus opts.NanoCPUs

nFlag int
Expand Down Expand Up @@ -65,6 +66,8 @@ func NewUpdateCommand(dockerCli command.Cli) *cobra.Command {
flags.Var(&options.memorySwap, "memory-swap", "Swap limit equal to memory plus swap: '-1' to enable unlimited swap")
flags.Var(&options.kernelMemory, "kernel-memory", "Kernel memory limit")
flags.StringVar(&options.restartPolicy, "restart", "", "Restart policy to apply when a container exits")
flags.Int64Var(&options.pidsLimit, "pids-limit", 0, "Tune container pids limit (set -1 for unlimited)")
flags.SetAnnotation("pids-limit", "version", []string{"1.40"})

flags.Var(&options.cpus, "cpus", "Number of CPUs")
flags.SetAnnotation("cpus", "version", []string{"1.29"})
Expand Down Expand Up @@ -103,6 +106,10 @@ func runUpdate(dockerCli command.Cli, options *updateOptions) error {
NanoCPUs: options.cpus.Value(),
}

if options.pidsLimit != 0 {
resources.PidsLimit = &options.pidsLimit
}

updateConfig := containertypes.UpdateConfig{
Resources: resources,
RestartPolicy: restartPolicy,
Expand Down
1 change: 1 addition & 0 deletions contrib/completion/bash/docker
Original file line number Diff line number Diff line change
Expand Up @@ -2266,6 +2266,7 @@ _docker_container_update() {
--memory -m
--memory-reservation
--memory-swap
--pids-limit
--restart
"

Expand Down
1 change: 1 addition & 0 deletions contrib/completion/zsh/_docker
Original file line number Diff line number Diff line change
Expand Up @@ -676,6 +676,7 @@ __docker_container_subcommand() {
"($help -m --memory)"{-m=,--memory=}"[Memory limit]:Memory limit: "
"($help)--memory-reservation=[Memory soft limit]:Memory limit: "
"($help)--memory-swap=[Total memory limit with swap]:Memory limit: "
"($help)--pids-limit[Tune container pids limit (set -1 for unlimited)]"
"($help)--restart=[Restart policy]:restart policy:(no on-failure always unless-stopped)"
)
opts_help=("(: -)--help[Print usage]")
Expand Down
1 change: 1 addition & 0 deletions docs/reference/commandline/update.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ Options:
-m, --memory string Memory limit
--memory-reservation string Memory soft limit
--memory-swap string Swap limit equal to memory plus swap: '-1' to enable unlimited swap
--pids-limit int Tune container pids limit (set -1 for unlimited)
--restart string Restart policy to apply when a container exits
```

Expand Down

0 comments on commit dca6d2a

Please sign in to comment.