All of this looks fine. Just in case someone needs to revert to the original behavior, I think we should add the initChownData stanza to the user-values-prom-operator.yaml files in monitoring and samples/generic-base/monitoring as well. That Helm chart's user yaml file is several thousand lines long and this particular value is actually part of the Grafana sub-chart: I think making a user who wants to revert to the original behavior dig through all of that would be user-hostile. Leaving a few "breadcrumbs" in those files to help them find their way would be nicer.
I went ahead and added the parameter to the user-values-prom-operator.yaml file and the generic base sample.
I went ahead and added the parameter to the user-values-prom-operator.yaml file and the generic base sample.