starlingx/app-rook-ceph
Code Issues Proposed changes

Fix YAML parse error due to special character

Browse Source 
This change prevents an error converting YAML to JSON in job-sync-osds
helm template, where the OS_PASSWORD env variable provided contains the
special character @ at the beginning of the string.

Additionally, the quota helm subfunction was added to the others
variables expected to be a string.

Test Plan:
 PASS: Build rook-ceph app.
 PASS: Deploy the built app in a setup that cover the special character
       password scenario.

Closes-Bug: 2088078

Change-Id: I5b3b07d81c85366a796b82681238d22e8b229eaf
Signed-off-by: Hediberto C Silva <hediberto.cavalcantedasilva@windriver.com>
This commit is contained in:
Hediberto C Silva 2024-11-13 14:15:02 -03:00
parent 3fb190bfe6
commit 0fd920eeb2
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
helm-charts/custom/rook-ceph-provisioner-helm/rook-ceph-provisioner-helm/rook-ceph-provisioner/templates/job-sync-osds.yaml
View File

@ -311,15 +311,15 @@ spec:
command: [ "/bin/bash", "/tmp/mount/sync_osds.sh" ]
env:
- name: OS_USERNAME
value: {{ .Values.sync_osds.os_username }}
value: {{ .Values.sync_osds.os_username | quote }}
- name: OS_PASSWORD
value: {{ .Values.sync_osds.os_password }}
value: {{ .Values.sync_osds.os_password | quote }}
- name: OS_AUTH_URL
value: {{ .Values.sync_osds.os_auth_url }}
value: {{ .Values.sync_osds.os_auth_url | quote }}
- name: TIMEOUT
value: {{ .Values.sync_osds.timeout }}
value: {{ .Values.sync_osds.timeout | quote }}
- name: DELAY
value: {{ .Values.sync_osds.delay }}
value: {{ .Values.sync_osds.delay | quote }}
- name: OSDS
value: {{ .Values.sync_osds.storage | toJson | quote }}
volumeMounts: