Skip to content

Commit

Permalink
Merge pull request #25 from risingwavelabs/shunjie/fix-empty-strings
Browse files Browse the repository at this point in the history
fix: quote all raw strings to avoid '<nil>' object or number strings
  • Loading branch information
arkbriar authored Feb 29, 2024
2 parents 80a8e1d + fcb4ff6 commit 9dfdbbb
Show file tree
Hide file tree
Showing 8 changed files with 22 additions and 22 deletions.
2 changes: 1 addition & 1 deletion charts/risingwave/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.39
version: 0.1.40

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand Down
6 changes: 3 additions & 3 deletions charts/risingwave/templates/azblob-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
AZBLOB_ACCOUNT_NAME: {{ .Values.stateStore.azblob.authentication.accountName }}
AZBLOB_ACCOUNT_KEY: {{ .Values.stateStore.azblob.authentication.accountKey }}
AZBLOB_ACCOUNT_NAME: {{ .Values.stateStore.azblob.authentication.accountName | quote }}
AZBLOB_ACCOUNT_KEY: {{ .Values.stateStore.azblob.authentication.accountKey | quote }}
{{- end }}
{{- end }}
{{- end }}
6 changes: 3 additions & 3 deletions charts/risingwave/templates/etcd-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
RW_ETCD_USERNAME: {{ .Values.metaStore.etcd.authentication.username }}
RW_ETCD_PASSWORD: {{ .Values.metaStore.etcd.authentication.password }}
{{- end }}
RW_ETCD_USERNAME: {{ .Values.metaStore.etcd.authentication.username | quote }}
RW_ETCD_PASSWORD: {{ .Values.metaStore.etcd.authentication.password | quote }}
{{- end }}
4 changes: 2 additions & 2 deletions charts/risingwave/templates/gcs-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
GOOGLE_APPLICATION_CREDENTIALS: {{ .Values.stateStore.gcs.authentication.credentials }}
GOOGLE_APPLICATION_CREDENTIALS: {{ .Values.stateStore.gcs.authentication.credentials | quote }}
{{- end }}
{{- end }}
{{- end }}
6 changes: 3 additions & 3 deletions charts/risingwave/templates/minio-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
MINIO_USERNAME: {{ .Values.stateStore.minio.authentication.username }}
MINIO_PASSWORD: {{ .Values.stateStore.minio.authentication.password }}
MINIO_USERNAME: {{ .Values.stateStore.minio.authentication.username | quote }}
MINIO_PASSWORD: {{ .Values.stateStore.minio.authentication.password | quote }}
{{- end }}
{{- end }}
{{- end }}
6 changes: 3 additions & 3 deletions charts/risingwave/templates/obs-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
OBS_ACCESS_KEY_ID: {{ .Values.stateStore.obs.authentication.accessKey }}
OBS_SECRET_ACCESS_KEY: {{ .Values.stateStore.obs.authentication.secretAccessKey }}
OBS_ACCESS_KEY_ID: {{ .Values.stateStore.obs.authentication.accessKey | quote }}
OBS_SECRET_ACCESS_KEY: {{ .Values.stateStore.obs.authentication.secretAccessKey | quote }}
{{- end }}
{{- end }}
{{- end }}
6 changes: 3 additions & 3 deletions charts/risingwave/templates/s3-secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
AWS_ACCESS_KEY_ID: {{ .Values.stateStore.s3.authentication.accessKey }}
AWS_SECRET_ACCESS_KEY: {{ .Values.stateStore.s3.authentication.secretAccessKey }}
AWS_ACCESS_KEY_ID: {{ .Values.stateStore.s3.authentication.accessKey | quote }}
AWS_SECRET_ACCESS_KEY: {{ .Values.stateStore.s3.authentication.secretAccessKey | quote }}
{{- end }}
{{- end }}
{{- end }}
8 changes: 4 additions & 4 deletions charts/risingwave/templates/secret.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ metadata:
{{ nindent 4 $annotations }}
{{- end }}
stringData:
root-user: {{ .Values.auth.rootUser }}
root-password: {{ .Values.auth.rootPassword }}
root-user: {{ .Values.auth.rootUser | quote }}
root-password: {{ .Values.auth.rootPassword | quote }}
{{- if ne .Values.auth.rootUser "root" }}
real-root-password: {{ randAlphaNum 8 }}
{{- end }}
real-root-password: {{ randAlphaNum 8 | quote }}
{{- end }}

0 comments on commit 9dfdbbb

Please sign in to comment.