fix(redis): Remove fullnameOverride from redis

[mjasion]

Waits for #21

Copying @omesser comment from #123

I recommend the approach where for such things we provide a full name override but the default behavior is to calculate the name in the chart, so the fullnameOverride value is either commented out (@jesper7 will hate it) or just empty string.

We can create a helper template (in _helpers.tpl) along those lines:

{{- define "studio.rayOperator.fullname" -}}
{{- if .Values.rayOperator.fullnameOverride -}}
{{- .Values.rayOperator.fullnameOverride | trunc 63 | trimSuffix "-" -}}
{{- else -}}
{{- printf "%s-ray-operator" .Release.Name | trunc 63 | trimSuffix "-" -}}
{{- end -}}
{{- end -}}

and use in values via {{ include "studio.rayOperator.fullname" . }}

Whenever we have such if-else boilerplate to calc a name or field - we can move it to a helper value to keep the resource charts clean and we would probably end up reusing a lot of those.

don't have a strong opinion about ray vs rayOperator but since the actual ray cluster is a CRD and not the operator that's installed in the chart it might be less confusing to make the distinction obvious via naming, harming brevity

[shcheklein]

@mjasion is still relevant?

[0x2b3bfa0]

No longer relevant in the short term; opened #619 for tracking.