fix: helm install values checks (#19296)

## Description
- Updated ngnix as default ingress class in `values.yaml`
- Upgraded mongo's deployment image to `v5.0.12` as the previous 4.4.11
is deprecated
- Added `imagePullSecrets` to enable pulling images from private docker
registries
- Added conditional to check for k8s version >=1.1.8 to handle both old
and new k8s version scenarios for setting up the Ingress controller
class name.

Co-authored-by: Goutham Pratapa <goutham@appsmith.com>
This commit is contained in:
Sumesh Pradhan 2023-01-17 13:01:28 +05:30 committed by GitHub
parent a1b07e21c7
commit c93ed4f54c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 11 additions and 4 deletions

View File

@ -21,6 +21,6 @@ dependencies:
repository: https://charts.bitnami.com/bitnami
- condition: mongodb.enabled
name: mongodb
version: 10.31.5
appVersion: 4.4.11
version: 13.6.0
appVersion: 5.0.12
repository: https://charts.bitnami.com/bitnami

View File

@ -44,7 +44,9 @@ spec:
secretName: {{ .secretName }}
{{- end }}
{{- end }}
ingressClassName: {{ .Values.ingress.className }}
{{- if semverCompare ">=1.18-0" .Capabilities.KubeVersion.GitVersion }}
ingressClassName: {{ .Values.ingress.className | default "nginx" }}
{{- end }}
rules:
{{- if not .Values.ingress.hosts }}
- host:

View File

@ -103,6 +103,10 @@ spec:
- secretRef:
name: {{ .Values.secretName }}
{{- end }}
{{- if .Values.image.pullSecrets}}
imagePullSecrets:
- name: {{ .Values.image.pullSecrets }}
{{- end }}
volumes:
{{- if not .Values.persistence.enabled }}
- name: data

View File

@ -63,6 +63,7 @@ image:
registry: index.docker.io
repository: appsmith/appsmith-ce
pullPolicy: IfNotPresent
pullSecrets: ""
# Overrides the image tag whose default is the chart appVersion.
tag: "latest"
## ServiceAccount
@ -180,7 +181,7 @@ ingress:
## e.g:
## className: "nginx"
##
className: ""
className: "nginx"
resources:
# We usually recommend not to specify default resources and to leave this as a conscious