Arthur Silva Sens

Results 418 comments of Arthur Silva Sens

> Note that in the doc you say: removed in a future major version, but in the PR you say minor (I'd prefer major). whoops, PR description updated

Weird, I'm not seeing this build errors locally even though I have no diff with this branch 🤔

Ah ok, it's an incompatibility with https://github.com/prometheus/prometheus/pull/17427. Found the problem after rebasing the PR. Let me fix that

I'm re-opening this issue until the configuration options are fully removed. Thanks for kicking off the work @ldufr

I think we're allowed to change feature gate stability every two releases. I'll go after this information and will report if we can stabilize the feature gate that removes this...

[Here's what I could find about release cycles.](https://github.com/open-telemetry/opentelemetry-collector/blob/main/featuregate/README.md#:~:text=Features%20that%20prove,the%20stable%20stage.) I guess we're good to finally remove these options from our Config struct :)

I'm re-opening the issue here since the configuration options are still present in the Config struct. We need to remove them and update our docs

> Run into Renovate Merged. Most metrics away. _total_total instandof _total. A view days ago also in this [#7039](https://github.com/open-telemetry/opentelemetry-go/issues/7039). How we came back into a more stable situation? We're getting...

I'd also vote for deprecating and eventually remove them!