Tyler Helmuth
Tyler Helmuth
Reading through this issue and https://github.com/open-telemetry/opentelemetry-collector/issues/9823, I have an alternative proposal to consider: - Keep `error` on `Start`. If an error is returned from `Start` it means the collector should...
@mx-psi @dmitryax i think we should implement this via a feature-gate-like config. The first PR can add the option to use the Go setting instead of the extension, but disabled...
We dont have anything like feature gates in the chart, we normally do it via a configuration option (or 2).
I have started work on this
@mx-psi is this helm chart feature critical for the decision on whether or not to deprecate the extension or has the decision already been made to deprecate it?
In that case I think we need to leave it around as long as it take to make a decision. We won't move forward with turning it always on until...
We see similar stats at Honeycomb
@mx-psi I'd still like to see the extension officially deprecated in Core before we make this a default value in the helm chart.
Ya I was imagining after the initial testing that the helm chart would follow the guidance from Core. My though being that I didn't want to disrupt users until the...
@galen-reltio we have an [UPGRADING.md file](https://github.com/open-telemetry/opentelemetry-helm-charts/blob/main/charts/opentelemetry-operator/UPGRADING.md) that is the right place for this warning. Would you like to submit an entry for this situation?