search

kadaster-labs / sensrnet-helm-charts

Other
0 stars 0 forks source link

Charts fail when their name contains upper case character #6

Closed kad-busses closed 3 years ago

kad-busses commented 3 years ago

For example helm install SensRNet SensRNet/central-viewer will fail, as Helm generates an hostname based on the deployment name (SensRNet). This hostname has to be lowercase. Add string function to cast name to lowercase https://helm.sh/docs/chart_template_guide/function_list/#lower

kad-busses commented 3 years ago

The hostname actually isn't set by our templates, but is Helm internal. We therefore cannot force the user to pick a lowercase name, or cast it as such. Since Helm prescribes only using lower case characters, numbers and dashes in their general conventions anyway (https://helm.sh/docs/chart_best_practices/conventions/#chart-names), so I've updated the readme to bring it to the user's attention