Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add mechanism to report go module for each factory #10598

Closed
wants to merge 1 commit into from

Conversation

djaglowski
Copy link
Member

Description

Adds each component's module to the components subcommand output.

➜  ./bin/otelcorecol_darwin_arm64 components                                                                                    ✭
...
buildinfo:
    command: otelcorecol
    description: Local OpenTelemetry Collector binary, testing only.
    version: 0.104.0-dev
receivers:
    - name: nop
      module: go.opentelemetry.io/collector/receiver/[email protected]
      stability:
        logs: Beta
        metrics: Beta
        traces: Beta
    - name: otlp
      module: go.opentelemetry.io/collector/receiver/[email protected]
      stability:
        logs: Beta
        metrics: Stable
        traces: Stable
processors:
    - name: batch
      module: go.opentelemetry.io/collector/processor/[email protected]
      stability:
        logs: Beta
        metrics: Beta
        traces: Beta
    - name: memory_limiter
      module: go.opentelemetry.io/collector/processor/[email protected]
      stability:
        logs: Beta
        metrics: Beta
        traces: Beta
exporters:
    - name: debug
      module: go.opentelemetry.io/collector/exporter/[email protected]
      stability:
        logs: Development
        metrics: Development
        traces: Development
    - name: logging
      module: go.opentelemetry.io/collector/exporter/[email protected]
      stability:
        logs: Deprecated
        metrics: Deprecated
        traces: Deprecated
    - name: nop
      module: go.opentelemetry.io/collector/exporter/[email protected]
      stability:
        logs: Beta
        metrics: Beta
        traces: Beta
    - name: otlp
      module: go.opentelemetry.io/collector/exporter/[email protected]
      stability:
        logs: Beta
        metrics: Stable
        traces: Stable
    - name: otlphttp
      module: go.opentelemetry.io/collector/exporter/[email protected]
      stability:
        logs: Beta
        metrics: Stable
        traces: Stable
connectors:
    - name: forward
      module: go.opentelemetry.io/collector/connector/[email protected]
      stability:
        logs-to-logs: Beta
        logs-to-metrics: Undefined
        logs-to-traces: Undefined
        metrics-to-logs: Undefined
        metrics-to-metrics: Beta
        metrics-to-traces: Undefined
        traces-to-logs: Undefined
        traces-to-metrics: Undefined
        traces-to-traces: Beta
extensions:
    - name: memory_ballast
      module: go.opentelemetry.io/collector/extension/[email protected]
      stability:
        extension: Deprecated
    - name: memory_limiter
      module: go.opentelemetry.io/collector/extension/[email protected]
      stability:
        extension: Development
    - name: zpages
      module: go.opentelemetry.io/collector/extension/[email protected]
      stability:
        extension: Beta

Issue

Resolves #10570

Testing

Manual only so far.

Documentation

None so far.

@djaglowski
Copy link
Member Author

I believe #10599 is a far better solution as it does not rely on any assumptions about the call stack.

@djaglowski djaglowski closed this Jul 11, 2024
mx-psi pushed a commit that referenced this pull request Jul 16, 2024
Resolves #10570

Alternative to #10598

This implementation is more reliable that #10598 because it gets
information directly from the builder manifest. It relies on additional
structure in the `component.go` file, ultimately encoded in
`otelcol.Factories`.

An alternative would be to push the enhancements deeper, into the
`<kind>.Factories` implementations, so that the module information is
available directly alongside the Factory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Components subcommand - Add module@version for each component
1 participant