search

cloudfoundry-incubator / kubo-deployment

Contains manifests used to deploy Cloud Foundry Container Runtime
https://www.cloudfoundry.org/container-runtime/
Apache License 2.0
275 stars 114 forks source link

Bump ulimit to be the same as in GKE #390

Closed alex-slynko closed 5 years ago

alex-slynko commented 5 years ago

What this PR does / why we need it: I would like to have better ulimit by default.

How can this PR be verified? https://github.com/cloudfoundry-incubator/kubo-ci/blob/master/src/tests/integration-tests/generic/ulimit_test.go#L25

Is there any change in kubo-release? Nope

Is there any change in kubo-ci? Maybe change test to have bigger default, but not required

Does this affect upgrade, or is there any migration required? NA

Which issue(s) this PR fixes: I can't start Eirini on default CFCR cluster.

Release note:

Bump ulimit to 1048576
cfdreddbot commented 5 years ago

:white_check_mark: Hey alex-slynko! The commit authors and yourself have already signed the CLA.

neosab commented 5 years ago

This LGTM @nehagjain15 can you merge it?