Browse Source

tests: Fix setting max openfds as memory limit (#20029)

The code was advertenly passing max openfds to debug.SetMemoryLimit(),
fixing this accelerate go test in my machine.

This is only a testing bug, since the server context has always a valid
MaxMem, so the buggy code was never called in users environments.
pull/20031/head
Anis Eleuch 1 year ago
committed by GitHub
parent
commit
ca0ce4c6ef
No known key found for this signature in database GPG Key ID: B5690EEEBB952194
  1. 6
      cmd/server-rlimit.go

6
cmd/server-rlimit.go

@ -82,11 +82,7 @@ func setMaxResources(ctx serverCtxt) (err error) {
}
if ctx.MemLimit > 0 {
maxLimit = ctx.MemLimit
}
if maxLimit > 0 {
debug.SetMemoryLimit(int64(maxLimit))
debug.SetMemoryLimit(int64(ctx.MemLimit))
}
// Do not use RLIMIT_AS as that is not useful and at times on systems < 4Gi

Loading…
Cancel
Save