Skip to content

Increase the max client body size limit to match the former static environment configuration

Vincent Sellier requested to merge deposit-limits into staging

The deposits are currently rejected if they are larger then 1m.

helm diff 
[swh] Comparing changes between branches production and deposit-limits (per environment)...
Your branch is ahead of 'origin/production' by 10 commits.
  (use "git push" to publish your local commits)
[swh] Generate config in production branch for environment staging, namespace swh...
[swh] Generate config in production branch for environment staging, namespace swh-cassandra...
[swh] Generate config in production branch for environment staging, namespace swh-cassandra-next-version...
Your branch is up to date with 'origin/deposit-limits'.
[swh] Generate config in deposit-limits branch for environment staging...
[swh] Generate config in deposit-limits branch for environment staging...
[swh] Generate config in deposit-limits branch for environment staging...
Your branch is ahead of 'origin/production' by 10 commits.
  (use "git push" to publish your local commits)
[swh] Generate config in production branch for environment production, namespace swh...
[swh] Generate config in production branch for environment production, namespace swh-cassandra...
[swh] Generate config in production branch for environment production, namespace swh-cassandra-next-version...
Your branch is up to date with 'origin/deposit-limits'.
[swh] Generate config in deposit-limits branch for environment production...
[swh] Generate config in deposit-limits branch for environment production...
[swh] Generate config in deposit-limits branch for environment production...


------------- diff for environment staging namespace swh -------------

No differences


------------- diff for environment staging namespace swh-cassandra -------------

--- /tmp/swh-chart.swh.dnNZoQCo/staging-swh-cassandra.before    2024-02-20 09:40:08.735081856 +0100
+++ /tmp/swh-chart.swh.dnNZoQCo/staging-swh-cassandra.after     2024-02-20 09:40:09.443081749 +0100
@@ -33460,20 +33460,23 @@
 kind: Ingress
 metadata:
   namespace: swh-cassandra
   name: deposit-ingress-authenticated
   labels:
     app: deposit
     endpoint-definition: authenticated
   annotations:
     nginx.ingress.kubernetes.io/service-upstream: "true"
     kubernetes.io/ingress.class: nginx
+    nginx.ingress.kubernetes.io/client-body-buffer-size: 128K
+    nginx.ingress.kubernetes.io/proxy-body-size: 4G
+    nginx.ingress.kubernetes.io/proxy-buffering: "on"
     # type of authentication
     nginx.ingress.kubernetes.io/auth-type: basic
     # an htpasswd file in the key auth within the secret
     nginx.ingress.kubernetes.io/auth-secret-type: auth-file
     # name of the secret that contains the user/password definitions
     nginx.ingress.kubernetes.io/auth-secret: swh-cassandra/deposit-auth-secrets
     # message to display with an appropriate context why the authentication is required
     nginx.ingress.kubernetes.io/auth-realm: 'Authentication Required'

 spec:
@@ -33516,20 +33519,23 @@
 kind: Ingress
 metadata:
   namespace: swh-cassandra
   name: deposit-ingress-default
   labels:
     app: deposit
     endpoint-definition: default
   annotations:
     nginx.ingress.kubernetes.io/service-upstream: "true"
     kubernetes.io/ingress.class: nginx
+    nginx.ingress.kubernetes.io/client-body-buffer-size: 128K
+    nginx.ingress.kubernetes.io/proxy-body-size: 4G
+    nginx.ingress.kubernetes.io/proxy-buffering: "on"

 spec:
   rules:
   - host: deposit-rpc-ingress
     http:
       paths:
       - path: /
         pathType: Prefix
         backend:
           service:


------------- diff for environment staging namespace swh-cassandra-next-version -------------

No differences


------------- diff for environment production namespace swh -------------

--- /tmp/swh-chart.swh.dnNZoQCo/production-swh.before   2024-02-20 09:40:10.003081665 +0100
+++ /tmp/swh-chart.swh.dnNZoQCo/production-swh.after    2024-02-20 09:40:10.515081587 +0100
@@ -35924,20 +35924,23 @@
 kind: Ingress
 metadata:
   namespace: swh
   name: deposit-ingress-authenticated
   labels:
     app: deposit
     endpoint-definition: authenticated
   annotations:
     nginx.ingress.kubernetes.io/service-upstream: "true"
     kubernetes.io/ingress.class: nginx
+    nginx.ingress.kubernetes.io/client-body-buffer-size: 128K
+    nginx.ingress.kubernetes.io/proxy-body-size: 4G
+    nginx.ingress.kubernetes.io/proxy-buffering: "on"
     # type of authentication
     nginx.ingress.kubernetes.io/auth-type: basic
     # an htpasswd file in the key auth within the secret
     nginx.ingress.kubernetes.io/auth-secret-type: auth-file
     # name of the secret that contains the user/password definitions
     nginx.ingress.kubernetes.io/auth-secret: swh/deposit-auth-secrets
     # message to display with an appropriate context why the authentication is required
     nginx.ingress.kubernetes.io/auth-realm: 'Authentication Required'

 spec:
@@ -35980,20 +35983,23 @@
 kind: Ingress
 metadata:
   namespace: swh
   name: deposit-ingress-default
   labels:
     app: deposit
     endpoint-definition: default
   annotations:
     nginx.ingress.kubernetes.io/service-upstream: "true"
     kubernetes.io/ingress.class: nginx
+    nginx.ingress.kubernetes.io/client-body-buffer-size: 128K
+    nginx.ingress.kubernetes.io/proxy-body-size: 4G
+    nginx.ingress.kubernetes.io/proxy-buffering: "on"

 spec:
   rules:
   - host: deposit-rpc-ingress
     http:
       paths:
       - path: /
         pathType: Prefix
         backend:
           service:


------------- diff for environment production namespace swh-cassandra -------------

No differences
Edited by Antoine R. Dumont

Merge request reports