Skip to content

Commit b49af9b

Browse files
Add back some resource limits for operator
Fixes operator deployments: ``` Deployment.apps "pulumi-kubernetes-operator-controller-manager" is invalid: spec.template.spec.containers[0].resources.requests: Invalid value: "1G": must be less than or equal to memory limit of 128Mi ``` Fixes DACH-NY/cn-test-failures#4535 [static] Signed-off-by: Martin Florian <martin.florian@digitalasset.com>
1 parent f43715e commit b49af9b

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

cluster/expected/operator/expected.json

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -599,6 +599,10 @@
599599
"name": "docker-reg-cred"
600600
}
601601
],
602+
"limits": {
603+
"cpu": 1,
604+
"memory": "2G"
605+
},
602606
"maxHistory": 10,
603607
"resources": {
604608
"requests": {

cluster/pulumi/operator/src/operator.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@ export const operator = new k8s.helm.v3.Release(
2727
version: '2.1.0',
2828
namespace: namespace.ns.metadata.name,
2929
values: {
30+
limits: {
31+
cpu: 1,
32+
memory: config.optionalEnv('OPERATOR_MEMORY_LIMIT') || '2G',
33+
},
3034
resources: {
3135
requests: {
3236
cpu: 0.2,

0 commit comments

Comments
 (0)