James Hughes (Splunk)

Results 28 comments of James Hughes (Splunk)

Perhaps we can make the type of DependsOn a custom type/interface (pseudo-type union) we could then implement it as-is and extend in the future? FWIW docker-compose [just uses a reference...

Sorry. So, currently, `DependsOn` is of type `[]Container`. I was trying to say that, if we wish to support some other non-Container type in the future, we could make it...

I agree that it's a bit awkward to use and encountered similar, [this](https://github.com/open-telemetry/opentelemetry-collector/pull/9670) is my "triage" attempt. Feel free to rip it apart or add to it if useful Of...

So, we've change the underlying datatype from String to Struct in [9472](https://github.com/open-telemetry/opentelemetry-collector/pull/9472/files), but there's still a few points of "awkwardness" remaining. From my perspective: - We don't allow "empty" `Type`s...

Hey @disfluxly , I'm personally prototyping something for my own company and will be bringing up our learnings come [April 26th @8AM PDT](https://docs.google.com/document/d/19bnXziPn2MZ9wO6684UoI4D-LCjGL5bTJkGhux29bx8/edit?usp=sharing) in the next prometheus working group meeting...

Can we *add* instead of *replace* the codeowners? [example from docs](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners#example-of-a-codeowners-file)

Looks like we already have `@open-telemetry/specs-semconv-approvers` set for such. Feel free to kill/merge/modify [this PR](https://github.com/open-telemetry/build-tools/pull/287) depending if we still want this

Hey, this seems like a dupe of [31599](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/31599), could I mark it as such? TL;dr is that there are extensions/observers that can, in conjunction with `receivercreator`, dynamically add jmx, but...