Skip to content

Commit

Permalink
Bump PID limit to 512 by default
Browse files Browse the repository at this point in the history
  • Loading branch information
DaneEveritt committed Jun 29, 2021
1 parent 08a7ccd commit b618ec8
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## v1.4.5
### Changed
* Upped the process limit for a container from `256` to `512` in order to address edge-cases for some games that spawn a lot of processes.

## v1.4.4
### Added
* **[security]** Adds support for limiting the total number of pids any one container can have active at once to prevent malicious users from impacting other instances on the same node.
Expand Down
2 changes: 1 addition & 1 deletion config/config_docker.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ type DockerConfiguration struct {
// at any given moment. This is a security concern in shared-hosting environments where a
// malicious process could create enough processes to cause the host node to run out of
// available pids and crash.
ContainerPidLimit int64 `default:"256" json:"container_pid_limit" yaml:"container_pid_limit"`
ContainerPidLimit int64 `default:"512" json:"container_pid_limit" yaml:"container_pid_limit"`

// InstallLimits defines the limits on the installer containers that prevents a server's
// installation process from unintentionally consuming more resources than expected. This
Expand Down

0 comments on commit b618ec8

Please sign in to comment.