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

[processor/resourcedetection] add support for Profiles signal type #36107

Merged

Conversation

odubajDT
Copy link
Contributor

@odubajDT odubajDT commented Oct 31, 2024

Description

  • added support for Profiles signal in processor
  • added tests
  • adapted README
  • performed go mod tidy on components which depend on the processor

Link to tracking issue

Fixes #35980

@github-actions github-actions bot added the processor/resourcedetection Resource detection processor label Oct 31, 2024
@odubajDT odubajDT force-pushed the resourcedetectionprocessor-profiles branch from efb40d0 to df0eb01 Compare October 31, 2024 13:06
@odubajDT odubajDT force-pushed the resourcedetectionprocessor-profiles branch 2 times, most recently from 4b46f17 to 9d790f1 Compare October 31, 2024 16:04
@odubajDT odubajDT marked this pull request as ready for review October 31, 2024 16:17
@odubajDT odubajDT requested a review from a team as a code owner October 31, 2024 16:17
@mx-psi
Copy link
Member

mx-psi commented Nov 7, 2024

@odubajDT Can you fix the merge conflicts? Happy to merge after that

@odubajDT odubajDT force-pushed the resourcedetectionprocessor-profiles branch from 22f73b2 to dd016f1 Compare November 7, 2024 12:22
Signed-off-by: odubajDT <[email protected]>
Signed-off-by: odubajDT <[email protected]>
@mx-psi mx-psi merged commit 9ae4b5b into open-telemetry:main Nov 7, 2024
157 of 158 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 7, 2024
pull bot pushed a commit to abaguas/opentelemetry-collector-contrib that referenced this pull request Nov 7, 2024
…pen-telemetry#36107)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

- added support for Profiles signal in processor
- added tests
- adapted README
- performed `go mod tidy` on components which depend on the processor

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#35980

---------

Signed-off-by: odubajDT <[email protected]>
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
…pen-telemetry#36107)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

- added support for Profiles signal in processor
- added tests
- adapted README
- performed `go mod tidy` on components which depend on the processor

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#35980

---------

Signed-off-by: odubajDT <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connector/datadog exporter/datadog Datadog components processor/resourcedetection Resource detection processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[processor/resourcedetection] Add support for profiles signal
4 participants