Describe the bug
Deployments through helm templates do not work.
To Reproduce
Steps to reproduce the behavior.
Deploy CrAPI with Helm. The script that I used was:
- helm upgrade --install crapi ./deploy/helm --namespace $KUBE_NAMESPACE --create-namespace -f ./deploy/helm/values.yaml
Expected behavior
Deployments should work with usual changes in values.yml like: apiGatewayServiceUrl and ports.
Runtime Environment
Ubuntu 22.04, Docker 27.1.1
Reason
In the individual helm templates within /deploy/helm/templates; some of the attributes in the config.yaml of each template, some attributes are missing quotes. For instance:
deploy/helm/templates/chatbot/config.yaml
data:
DB_USER: {{ .Values.postgresdb.config.postgresUser }}
DB_PASSWORD: {{ .Values.postgresdb.config.postgresPassword }}
Here username and password are missing quotes. This should be:
data:
DB_USER: {{ .Values.postgresdb.config.postgresUser | quote}}
DB_PASSWORD: {{ .Values.postgresdb.config.postgresPassword | quote }}
I will create a pull request with working changes. Please let me know if this would be ok
Describe the bug
Deployments through helm templates do not work.
To Reproduce
Steps to reproduce the behavior.
Deploy CrAPI with Helm. The script that I used was:
- helm upgrade --install crapi ./deploy/helm --namespace $KUBE_NAMESPACE --create-namespace -f ./deploy/helm/values.yaml
Expected behavior
Deployments should work with usual changes in values.yml like: apiGatewayServiceUrl and ports.
Runtime Environment
Ubuntu 22.04, Docker 27.1.1
Reason
In the individual helm templates within /deploy/helm/templates; some of the attributes in the config.yaml of each template, some attributes are missing quotes. For instance:
deploy/helm/templates/chatbot/config.yaml
data:
DB_USER: {{ .Values.postgresdb.config.postgresUser }}
DB_PASSWORD: {{ .Values.postgresdb.config.postgresPassword }}
Here username and password are missing quotes. This should be:
data:
DB_USER: {{ .Values.postgresdb.config.postgresUser | quote}}
DB_PASSWORD: {{ .Values.postgresdb.config.postgresPassword | quote }}
I will create a pull request with working changes. Please let me know if this would be ok