Skip to content

Update the ngix configuration to support resource requests and service monitoring

Vincent Sellier requested to merge mr/nginx-config into production
  • also reduce the deployment history to 3

Staging changes:

--- staging-before.yaml 2023-05-24 18:26:14.105935089 +0200
+++ staging-after.yaml  2023-05-24 18:26:22.245960559 +0200
@@ -155,6 +155,7 @@
     helm:
       releaseName: ingress-nginx
       values: |
+        revisionHistoryLimit: 3
         controller:
           watchIngressWithoutClass: false
           service:
@@ -164,6 +165,14 @@
           ingressClassResource:
             name: "nginx"
             default: true
+          metrics:
+            enabled: true
+            serviceMonitor:
+              enabled: true
+          resources:
+            requests:
+              cpu: 50m
+              memory: 200Mi
   destination:
     server: https://rancher.euwest.azure.internal.softwareheritage.org/k8s/clusters/c-m-9n5h9nrf
     namespace: ingress-nginx```

Production changes:

--- production-before.yaml      2023-05-23 19:42:54.412459578 +0200
+++ production-after.yaml       2023-05-24 18:25:21.049769069 +0200
@@ -155,6 +155,7 @@
     helm:
       releaseName: ingress-nginx
       values: |
+        revisionHistoryLimit: 3
         controller:
           watchIngressWithoutClass: false
           service:
@@ -164,6 +165,10 @@
           ingressClassResource:
             name: "nginx"
             default: true
+          resources:
+            requests:
+              cpu: 100m
+              memory: 90Mi
   destination:
     server: https://rancher.euwest.azure.internal.softwareheritage.org/k8s/clusters/c-m-75xcg59s
     namespace: ingress-nginx
Edited by Vincent Sellier

Merge request reports