[operator] manager.serviceMonitor.enabled is redundant

[pl4nty]

The operator now creates a ServiceMonitor by default, taking over the chart's ServiceMonitor if it exists. So manager.serviceMonitor.enabled is redundant, a ServiceMonitor will always be created.

I don't want a ServiceMonitor, so I used this workaround:

manager:
  extraArgs:
    - --enable-cr-metrics=false

Also tracked on the operator repo as open-telemetry/opentelemetry-operator#3474

[TylerHelmuth]

@jaronoff97 how do you want to handle this in the chart?

[jaronoff97]

i was away when this was added... but i was under the impression this was only the case in openshift environments. Looking.