Details
-
Bug
-
Resolution: Fixed
-
Major
-
7.1.1
-
Untriaged
-
1
-
Unknown
Description
As part of addressing MB-47591 in 7.1.1, the services need to respect the value of the GOMAXPROCS env var if specified. FTS does this in a simple way here, though it seems a bit unnecessary as according to https://pkg.go.dev/runtime#GOMAXPROCS, the GOMAXPROCS default is runtime.NumCPU anyway. Query does it in a slightly more complicated fashion here, but it looks like it also works.
Perhaps the simplest thing to do for Backup is to remove this line of code, but I'll leave it in your hands as to how exactly you address this issue.
Attachments
Issue Links
- blocks
-
MB-47591 Incorrect cgroup resource detection (core/memory) when running in Container / K8S
- Closed
For Gerrit Dashboard: MB-51864 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
173887,1 | MB-51864 Don't manually set GOMAXPROCS to number of CPUs | neo | cbbs | Status: ABANDONED | 0 | -1 |
173888,2 | MB-51864 Don't manually set GOMAXPROCS | neo | cbbs | Status: MERGED | +2 | +1 |
173964,3 | MB-51864 system.NumCPU should use GOMAXPROCS | neo | tools-common | Status: MERGED | +2 | +1 |
173965,9 | MB-51864 Make `TotalMemory` respect cgroup limits | neo | tools-common | Status: MERGED | +2 | +1 |
174594,2 | MB-51864 Update tools-common dependency | neo | backup | Status: MERGED | +2 | +1 |
174595,2 | MB-51864 Update tools-common dependency | neo | cbbs | Status: MERGED | +2 | +1 |
174596,1 | Merge branch 'neo' into master | master | tools-common | Status: MERGED | +2 | +1 |
174597,2 | Merge branch 'neo' into master | master | cbbs | Status: MERGED | +2 | +1 |
174598,1 | Merge branch 'neo' into master | master | backup | Status: MERGED | +2 | +1 |
174921,3 | MB-51864 Have 'GetInformation' honor CPU limits | neo | tools-common | Status: MERGED | +2 | +1 |
174941,2 | MB-51864 Update tools-common dependency | neo | backup | Status: MERGED | +2 | +1 |
174942,2 | MB-51864 Update tools-common dependency | neo | cbbs | Status: MERGED | +2 | +1 |
174949,1 | Merge branch 'neo' into master | master | tools-common | Status: MERGED | +2 | +1 |
174951,1 | Merge branch 'neo' into master | master | backup | Status: MERGED | +2 | +1 |
174952,1 | Merge branch 'neo' into master | master | cbbs | Status: MERGED | +2 | +1 |