Paulin Todev

Results 179 comments of Paulin Todev

@thampiotr thank you very much for the review! > IIRC this is also code that we copied from loki/promtail so it creates a further diversion. Could we have one way...

> Good point. But maybe it's worth adding functionality to serialise with secrets or at least hash of the secrets so that we can avoid the hand-written code to do...

@thampiotr [the PR](https://github.com/grafana/alloy/pull/3356) looks like a step in the right direction, but I wish we didn't have to rely on users reading about such edge cases in the docs. Is...

I think so. For example, each `prometheus.exporter` component could check for the `--cluster.enabled` cli argument and it could produce a warning if there are known issues for that component in...

Hello! There has been a [Windows 2022 image](https://hub.docker.com/layers/grafana/alloy/windowsservercore-ltsc2022/images/sha256-d592f6cdee3cc34711b1e77c33bdd7a01b89468d0dfa411b65393c06d12da0dc) for a few months now. It would be nice to also add a 2025 image, but I'll close this issue since it...

Hello! Apologies for the late response. I'm not very familiar with this, but I don't quite see why restarting the tailer once every hour is a problem if the volume...

Hi, @atoulme and @yurishkuro 👋 Would you like to opine on this PR please, since you were active on the [previous one](https://github.com/open-telemetry/opentelemetry-collector/pull/10694)?

@yurishkuro unfortunately, go-jsonschema doesn't have this level of flexibility. It an interesting idea though, and it could be worth exploring if there are concrete use cases worth looking into.

It is currently possible to override the type for a field. For example, the [earlier PoC PR](https://github.com/open-telemetry/opentelemetry-collector/pull/10694/files#diff-0e371876fcb5b60ff16678a5ec169c5603359c738bd8c5ccece93a6fe38f75c2R24-R30) would use `configopaque.String` using schema like this: ```yaml cert_pem: type: string default: ""...

I added a simple example with Optional [here](https://github.com/ptodev/opentelemetry-collector/commit/c573c091f794c8b9a6e3319ca005f26103bcc995#diff-56ab1af8cbcffe01706f9128997621fca84cec47e4c5aa0b9c17ce74edbb30e5). It is true that the default values wouldn't work with Optional. I didn't include defaults in my example either - for now...