Pablo Baeyens comments

Results 778 comments of


                                            Pablo Baeyens

[service/internal] Remove Ready from status.Reporter

@mwear can you fix the merge conflict? I can merge after that

[service] add logger provider configuration support

@mowies I don't think there is one, feel free to create one :)

Support default values and erorrs for env var expansion

> @mx-psi is Configuration WG decision on shell syntax documented / implemented somewhere? Yup, see https://opentelemetry.io/docs/specs/otel/configuration/file-configuration/#environment-variable-substitution

Support default values and erorrs for env var expansion

Note that the `:?error` syntax is not supported by the Configuration WG. I am in favor of it, but it would be good to at least check that the Configuration...

Decide what to do with `Builder`s

Fixed, we are moving them to service in #10782, #10783, #10785, #10784 and #10781. Next step is removing the deprecated API

Collector's Internal Metrics Not Disabled When Specifying service:telemetry:metrics:level:none

@RaginiDhatric Can you try with the latest version (v0.106.0)?

Collector's Internal Metrics Not Disabled When Specifying service:telemetry:metrics:level:none

@RaginiDhatric @atib92 It would be helpful if you can test which of the components is generating the metrics. To do this you can replace the different components and check if...

Add `TrustedKeys` to TLS Config

Hey @namco1992 , feel free to open a PR. I still would want to explore if other applications do this and what's their UX like to make sure we are...

confmap.Retrieved option to avoid further expansion by service config provider

What's the use case for this?

[component] Move component status features to its own module

> Part of this means having a functioning health check extension The healthcheck extension is [not in our GA roadmap](https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/ga-roadmap.md#proposal). I think it's important that we keep working on it,...