Skip to content

interpreter: introduce Config#1435

Closed
florianl wants to merge 1 commit into
mainfrom
interpreter-config
Closed

interpreter: introduce Config#1435
florianl wants to merge 1 commit into
mainfrom
interpreter-config

Conversation

@florianl
Copy link
Copy Markdown
Member

Allow each interpreter to be configured individually, making it easy to tune performance for different environments.

Spin-Off from #1392, as the feedback is that we want to have per interpreter configuration. #1392 still remains active, as interpreter might not be the best name.

A local example reporting to firepit could look like this:

# OpenTelemetry Collector configuration that profiles only Python, Go and Go
# labels, reporting to a local firepit instance.
#
# Run with:
#   sudo ./otelcol-ebpf-profiler --config=cmd/otelcol-ebpf-profiler/local.firepit.python-go-labels.yaml \
#     --feature-gates=+service.profilesSupport
receivers:
  profiling:
    interpreters:
      perl:
        disabled: true
      php:
        disabled: true
      hotspot:
        disabled: true
      ruby:
        disabled: true
      v8:
        disabled: true
      dotnet:
        disabled: true
      beam:
        disabled: true


exporters:
  otlp/firepit:
    endpoint: 127.0.0.1:4317
    tls:
      insecure: true
      insecure_skip_verify: true


service:
  pipelines:
    profiles:
      receivers: [profiling]
      exporters: [otlp/firepit]

This proposal preserves the existing behaviour, that enables all unwinders by default, and allows to add dedicated per-unwinder configuration in follow up changes.

Allow each interpreter to be configured individually, making it easy to tune performance for different environments.

Spin-Off from #1392, as the feedback is that we want to have per interpreter configuration. #1392 still remains active, as `interpreter` might not be the best name.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@florianl florianl requested review from a team as code owners May 20, 2026 10:34
@github-actions github-actions Bot requested review from GregMefford and dalehamel May 20, 2026 10:34
@florianl
Copy link
Copy Markdown
Member Author

florianl commented May 20, 2026

Closing, as I pushed this PR against upstream and not my fork :| follow up with #1436.

@florianl florianl closed this May 20, 2026
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.

1 participant