Add resources to our system components. (knative#2532)

These values were determined from the graphs in the linked issue, which are from a knative/serving install running the e2e/conformance testing multiple times. Fixes: knative#2393
vagababov · Nov 30, 2018 · b0d2550 · b0d2550
1 parent c3b8de1
commit b0d2550
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 0 deletions.
diff --git a/config/activator.yaml b/config/activator.yaml
@@ -47,6 +47,15 @@ spec:
           # and seeing k8s logs in addition to ours is not useful.
         - "-logtostderr=false"
         - "-stderrthreshold=FATAL"
+        resources:
+          # Request 2x what we saw running e2e
+          requests:
+            cpu: 20m
+            memory: 60Mi
+          # Limit to 10x the request (20x the observed peak during e2e)
+          limits:
+            cpu: 200m
+            memory: 600Mi
         env:
           - name: POD_NAME
             valueFrom:

diff --git a/config/autoscaler.yaml b/config/autoscaler.yaml
@@ -35,6 +35,15 @@ spec:
         # This is the Go import path for the binary that is containerized
         # and substituted here.
         image: github.com/knative/serving/cmd/autoscaler
+        resources:
+          # Request 2x what we saw running e2e
+          requests:
+            cpu: 30m
+            memory: 40Mi
+          # Limit to 10x the request (20x the observed peak during e2e)
+          limits:
+            cpu: 300m
+            memory: 400Mi
         ports:
         - name: websocket
           containerPort: 8080

diff --git a/config/controller.yaml b/config/controller.yaml
@@ -35,6 +35,15 @@ spec:
         # This is the Go import path for the binary that is containerized
         # and substituted here.
         image: github.com/knative/serving/cmd/controller
+        resources:
+          # Request 2x what we saw running e2e
+          requests:
+            cpu: 100m
+            memory: 100Mi
+          # Limit to 10x the request (20x the observed peak during e2e)
+          limits:
+            cpu: 1000m
+            memory: 1000Mi
         ports:
         - name: metrics
           containerPort: 9090

diff --git a/config/webhook.yaml b/config/webhook.yaml
@@ -37,6 +37,15 @@ spec:
         # This is the Go import path for the binary that is containerized
         # and substituted here.
         image: github.com/knative/serving/cmd/webhook
+        resources:
+          # Request 2x what we saw running e2e
+          requests:
+            cpu: 20m
+            memory: 20Mi
+          # Limit to 10x the request (20x the observed peak during e2e)
+          limits:
+            cpu: 200m
+            memory: 200Mi
         volumeMounts:
         - name: config-logging
           mountPath: /etc/config-logging