Skip to content

Commit

Permalink
Merge pull request #286 from RADAR-base/feature/appserver-ingress-ann…
Browse files Browse the repository at this point in the history
…otation-upload

Set ingress proxy-body-size when upload enabled in appserver
  • Loading branch information
keyvaann authored Oct 22, 2024
2 parents f91d9e1 + 8a293b1 commit d652138
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
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

0 comments on commit d652138

Please sign in to comment.