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.

Sign up for GitHub

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

Jump to bottom

[receiver/otlpjsonfile] Add support for profiles signal #35977

Closed
mx-psi opened this issue Oct 24, 2024 · 2 comments · Fixed by #35995
Closed

[receiver/otlpjsonfile] Add support for profiles signal #35977

mx-psi opened this issue Oct 24, 2024 · 2 comments · Fixed by #35995
Labels
data:profiles Profiles related issues help wanted Extra attention is needed priority:p2 Medium receiver/otlpjsonfile

Comments

@mx-psi
Copy link
Member

mx-psi commented Oct 24, 2024

Component(s)

receiver/otlpjsonfile

Describe the issue you're reporting

This receiver would be useful for testing and validation of the experimental profiling signal.

As inspiration for how to add support this PR can be used: open-telemetry/opentelemetry-collector/pull/11071

Example of configuration using profiles (needs service.profilesSupport feature gate to be enabled):

receivers:
  otlp:
    protocols:
      grpc:
exporters:
  otlp:
    endpoint: ${OTLP_ENDPOINT}
service:
  pipelines:
    profiles:
      receivers: [otlp]
      exporters: [otlp]
@mx-psi mx-psi added help wanted Extra attention is needed priority:p2 Medium receiver/otlpjsonfile data:profiles Profiles related issues labels Oct 24, 2024
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners for receiver/otlpjsonfile: @djaglowski @atoulme. See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@haoqixu haoqixu mentioned this issue Oct 25, 2024
Merged
djaglowski pushed a commit that referenced this issue Oct 25, 2024
@haoqixu
[receiver/otlpjsonfile] Add support for profiles signal (#35995)
2416e90 
#### Description
Add support for profiles signal to `otlpjsonfilereceiver`.

#### Link to tracking issue
Fixes #35977

#### Testing
- add `TestFileProfilesReceiver`
- update `TestFileMixedSignals`
@github-actions github-actions bot mentioned this issue Oct 29, 2024
Open
jpbarto pushed a commit to jpbarto/opentelemetry-collector-contrib that referenced this issue Oct 29, 2024
@haoqixu @jpbarto
[receiver/otlpjsonfile] Add support for profiles signal (open-telemet…
23c2d00 
…ry#35995)

#### Description
Add support for profiles signal to `otlpjsonfilereceiver`.

#### Link to tracking issue
Fixes open-telemetry#35977

#### Testing
- add `TestFileProfilesReceiver`
- update `TestFileMixedSignals`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
data:profiles Profiles related issues help wanted Extra attention is needed priority:p2 Medium receiver/otlpjsonfile
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[receiver/otlpjsonfile] Add support for profiles signal haoqixu/opentelemetry-collector-contrib
1 participant
@mx-psi