-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update values.yaml #530
Update values.yaml #530
Conversation
I think that a volume name prefix is required for persistent storage. When I set the name the PVCs were created. Maybe I am wrong, maybe if a storageClassName is set and the prefix is left empty then PVCs will also get created? Signed-off-by: DanRoscigno <[email protected]>
Since the kube-starrocks chart is composed of two sub-charts (the operator chart and the starrocks chart), the values.yaml file for the kube-starrocks chart is automatically derived by merging their respective values.yaml files. Therefore, the modification should be made to the values.yaml of the starrocks chart. Then, in the scripts directory, run the bash script create-parent-chart-values.sh to update the values.yaml file of the parent chart (the kube-starrocks chart). |
Signed-off-by: DanRoscigno <[email protected]>
Signed-off-by: DanRoscigno <[email protected]>
Thanks @yandongxiao updated and ran the script. |
@@ -335,7 +335,7 @@ starrocks: | |||
# Note: Once set, the following fields will not be allowed to be modified. | |||
storageSpec: | |||
# Specifies the name prefix of the volumes to mount. If left unspecified, `emptyDir` volumes will be used by default, which | |||
# are ephemeral and data will be lost on pod restart. For persistent storage, it is recommended to specify a volume name prefix. | |||
# are ephemeral and data will be lost on pod restart. For persistent storage, specify a volume name prefix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes you are right, name is required. The values.yaml of starrocks chart also needs to be updated
I think that a volume name prefix is required for persistent storage. When I set the name the PVCs were created.
Maybe I am wrong, maybe if a storageClassName is set and the prefix is left empty then PVCs will also get created?
Description
Please provide a detailed description of the changes you have made. Include the motivation for these changes, and any
additional context that may be important.
Related Issue(s)
Please list any related issues and link them here.
Checklist
For operator, please complete the following checklist:
make generate
to generate the code.golangci-lint run
to check the code style.make test
to run UT.make manifests
to update the yaml files of CRD.For helm chart, please complete the following checklist:
file of starrocks chart.
scripts
directory, runbash create-parent-chart-values.sh
to update the values.yaml file of the parentchart( kube-starrocks chart).