From d5d958a33242cdac3b360638e9a89941d15c0893 Mon Sep 17 00:00:00 2001 From: Manuel de Brito Fontes Date: Mon, 12 Jun 2017 16:20:40 -0400 Subject: [PATCH] Avoid setting maximum number of open file descriptors lower than 1024 --- controllers/nginx/pkg/cmd/controller/nginx.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/controllers/nginx/pkg/cmd/controller/nginx.go b/controllers/nginx/pkg/cmd/controller/nginx.go index 2291a73066..8c3a5184b8 100644 --- a/controllers/nginx/pkg/cmd/controller/nginx.go +++ b/controllers/nginx/pkg/cmd/controller/nginx.go @@ -459,11 +459,13 @@ func (n *NGINXController) OnUpdate(ingressCfg ingress.Configuration) error { // the limit of open files is per worker process // and we leave some room to avoid consuming all the FDs available wp, err := strconv.Atoi(cfg.WorkerProcesses) + glog.V(3).Infof("number of worker processes: %v", wp) if err != nil { wp = 1 } maxOpenFiles := (sysctlFSFileMax() / wp) - 1024 - if maxOpenFiles < 0 { + glog.V(3).Infof("maximum number of open file descriptors : %v", sysctlFSFileMax()) + if maxOpenFiles < 1024 { // this means the value of RLIMIT_NOFILE is too low. maxOpenFiles = 1024 }