Skip to content

Commit b66aace

Browse files
authored
executor: increase k8s executor memory limit to avoid OOM (#894)
# Summary Increase k8s executor memory limit 1Gi->4Gi to avoid OOM. Does not increase memory request not to cause relocating of pods and greater nodes. ### Checklist - [x] Follow the [manual testing process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md) - [ ] Update [changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md) - [ ] Update [Kubernetes update doc](https://docs.sourcegraph.com/admin/updates/kubernetes) ### Test plan ```sh ### Chart [ sourcegraph-executor-k8s ] ./charts/sourcegraph-executor/k8s/ PASS executor charts/sourcegraph-executor/k8s/tests/executor_test.yaml Charts: 1 passed, 1 total Test Suites: 1 passed, 1 total Tests: 7 passed, 7 total Snapshot: 0 passed, 0 total Time: 42.867208ms ```
1 parent 8db7e22 commit b66aace

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

charts/sourcegraph-executor/k8s/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ In addition to the documented values, the `executor` and `private-docker-registr
9696
| executor.queueNames | list | `[]` | The names of multiple queues to pull jobs from to. Possible values: batches and codeintel. **Either this or queueName is required.** |
9797
| executor.replicas | int | `1` | |
9898
| executor.resources.limits.cpu | string | `"1"` | |
99-
| executor.resources.limits.memory | string | `"1Gi"` | |
99+
| executor.resources.limits.memory | string | `"4Gi"` | |
100100
| executor.resources.requests.cpu | string | `"500m"` | |
101101
| executor.resources.requests.memory | string | `"200Mi"` | |
102102
| executor.securityContext | object | `{"fsGroup":null,"privileged":false,"runAsGroup":null,"runAsUser":null}` | DEPRECATED: Use `executor.containerSecurityContext` or `executor.podSecurityContext` instead. |

charts/sourcegraph-executor/k8s/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ executor:
6666
resources:
6767
limits:
6868
cpu: "1"
69-
memory: 1Gi
69+
memory: 4Gi
7070
requests:
7171
cpu: 500m
7272
memory: 200Mi

0 commit comments

Comments
 (0)