Remove templating from values.yaml

[jessebot]

Templating should only be done in files under the charts/openbao/templates directory. The reason for this is that some users use third party templating tools, as Argo CD for gitops, which will be applying their own go-templating and it can cause conflicts and errors.

Here's an example of some values that should still be in the values.yaml as an example, but commented out by default:

openbao-helm/charts/openbao/values.yaml

Lines 303 to 304 in b59b6e5

	app.kubernetes.io/name: {{ template "vault.name" . }}-agent-injector
	app.kubernetes.io/instance: "{{ .Release.Name }}"

We can still draw up some docs to explain to users how to use the affinity features of this chart and place them in the README or our official doc site, but defaults should always be template free.

[cipherboy]

I was curious where this templating comes from; is Kubernetes doing the templating or is that something within OpenBao?

[jessebot]

No problem :) This is all gotemplating from helm, separate from OpenBao itself, but it should be live in the templates dir instead of the values.yaml.