smartcheck-helm icon indicating copy to clipboard operation
smartcheck-helm copied to clipboard

Helm chart validation failing , possible syntax error.

Open rameshkjha opened this issue 3 years ago • 1 comments

Hi,

I need to deploy more than one container using helm chart, I am trying to modify existing helm chart to include other container as well and do the deployment. when doing the helm lint to validate the chart, it is failing.

__[ERROR] templates/: parse error at (dashboard-svc/templates/deployment.yaml:29): "-"

Error: 1 chart(s) linted, 1 chart(s) failed [ramejha@phoenix68163 helm_chart]$ q! -bash: q!: command not found__

Error is in this statement {{- $values := .Values - }} // ERROR is in line no 29 here ------- values.yaml ---------- images: dashboardsvc: pullPolicy: Always port: 8080 nginx: pullPolicy: Always port: 8080

replicaCount: 1 image: repository: artifactory-master.****.com/docker-dashboardsvc pullPolicy: IfNotPresent tag: dashboardsvc_2110.0.5_1@sha256:345bd653a687c78414d475fff8b87ded03008b848cb10aa96897c2fdfgrg767 repository: artifactory-master.**com/docker-nginx pullPolicy: IfNotPresent tag: nginx-dashboard_2110.0.5_3@sha256:345bd653a687c78414d475fff8b87ded03008b848cb10aa96897c2fddc676767

------------ deployment.yaml --------------- spec: imagePullSecrets: - name: {{ .Values.imagePullSecretName }} serviceAccountName: {{ include "dashboardsvc.serviceAccountName" . }} securityContext: {{- toYaml .Values.podSecurityContext | nindent 8 }} containers: {{- $values := .Values - }} // ERROR is in line no 29 here {{- $appVersion := .Chart.AppVersion - }} {{- range $imageName,$image := .Values.images }} - name: {{ $imageName }} image: "{{ $image.repository }}:{{ $image.tag | default $appVersion }}" imagePullPolicy: {{ $image.pullPolicy }}

It looks like to me some syntax error, Appreciate any help.

rameshkjha avatar Jan 27 '22 00:01 rameshkjha

Hi @rameshkjha,

The Smart Check helm chart is not intended to be modified. Are you aware that multiple helm charts can be deployed to one cluster? Typically one helm chart represents one application/service. Recommend making your own helm chart for your extra containers, or deploying them without a helm chart.

johan1252 avatar Jan 27 '22 01:01 johan1252