Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set ingress proxy-body-size when upload enabled in appserver #286

Merged
merged 1 commit into from
Oct 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/radar-appserver/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
appVersion: "2.4.3"
description: A Helm chart for the backend application of RADAR-base Appserver
name: radar-appserver
version: 0.7.0
version: 0.7.1
icon: "http://radar-base.org/wp-content/uploads/2022/09/Logo_RADAR-Base-RGB.png"
sources:
- https://github.com/RADAR-base/radar-helm-charts/tree/main/charts/radar-appserver
Expand Down
4 changes: 2 additions & 2 deletions charts/radar-appserver/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# radar-appserver
[![Artifact HUB](https://img.shields.io/endpoint?url=https://artifacthub.io/badge/repository/radar-appserver)](https://artifacthub.io/packages/helm/radar-base/radar-appserver)

![Version: 0.7.0](https://img.shields.io/badge/Version-0.7.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 2.4.3](https://img.shields.io/badge/AppVersion-2.4.3-informational?style=flat-square)
![Version: 0.7.1](https://img.shields.io/badge/Version-0.7.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 2.4.3](https://img.shields.io/badge/AppVersion-2.4.3-informational?style=flat-square)

A Helm chart for the backend application of RADAR-base Appserver

Expand Down Expand Up @@ -95,7 +95,7 @@ A Helm chart for the backend application of RADAR-base Appserver
| smtp.starttls | bool | `false` | set to true,if TTLS should be enabled |
| smtp.auth | bool | `true` | set to true, if the account should be authenticated before sending emails |
| upload.enabled | bool | `false` | if set to true, file upload endpoint will be enabled |
| upload.max_file_size | string | `"20MB"` | Maximum file size for upload |
| upload.max_file_size | string | `"20MB"` | Maximum file size for upload. Can be any number followed by MB or GB. |
| upload.storage.type | string | `"s3"` | Type of storage to use for file upload (s3) |
| upload.storage.url | string | `"http://minio-headless:9000"` | Internal url to storage |
| upload.storage.bucket_name | string | `"radar-output-storage"` | Bucket name of the S3 storage |
Expand Down
1 change: 1 addition & 0 deletions charts/radar-appserver/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ metadata:
annotations:
{{- toYaml . | nindent 4 }}
{{- end }}
nginx.ingress.kubernetes.io/proxy-body-size: {{ .Values.upload.max_file_size | lower | trimAll "b" }}
spec:
ingressClassName: {{ .Values.ingress.ingressClassName | quote }}
{{- if and .Values.ingress.tls (not .Values.disable_tls) }}
Expand Down
4 changes: 2 additions & 2 deletions charts/radar-appserver/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -240,8 +240,8 @@ smtp:
upload:
# -- if set to true, file upload endpoint will be enabled
enabled: false
# -- Maximum file size for upload
max_file_size: "20MB"
# -- Maximum file size for upload. Can be any number followed by MB or GB.
max_file_size: 20MB
# Settings for file storage
storage:
# -- Type of storage to use for file upload (s3)
Expand Down
Loading