Bogdan Drutu
Bogdan Drutu
Remove `GetExtensions` from `component.Host`. This should probably be an "optional" interface in the Extension package, so that it can return `GetExtensions() map[component.ID]extension.Extension`.
We should change the NewNopSettings to accept a `component.Type` to ensure in tests and everywhere we pass to factories only settings with IDs that match their Type. **Alternative** (which may...
In https://github.com/open-telemetry/opentelemetry-collector/pull/9637, we created a unique name per settings: 1. This is inconsistent with other nop settings. 2. The change is no longer needed because we no longer use prometheus...
There are lots of `// nolint:gosec` annotations to move forward with https://github.com/open-telemetry/opentelemetry-collector/pull/10935 since the problems are not new. We should fix the wrong conversions where possible.
This package is very confusing, because it has a very generic name and applies only to mdatagen generated internal telemetry. Also, the config with include/exclude is something that we try...
One of the reason the mess with default providers (see https://github.com/open-telemetry/opentelemetry-collector/issues/11403 or https://github.com/open-telemetry/opentelemetry-collector/releases/tag/v0.109.0 or the need to disable strict check, see https://github.com/open-telemetry/opentelemetry-collector/pull/11168) is the support of building older version of...
Now that we have every component in the core as a separate module do we still need `otelcol_version` in the builder?
These 2 are exactly the same. We should keep only one. cc @open-telemetry/collector-approvers