Skip to content

Conversation

@rockdaboot
Copy link
Contributor

@rockdaboot rockdaboot commented Mar 28, 2025

Initial PR for profiles support in the transform processor.

Fixes #39009

Review

The first commit "[processor/transform] Add support for profiles]" contains the actual changes to the transformprocessor.
The other commits contain the code changes from other PRs that haven't been merged yet but that are required or wanted. Here is the list of PRs and issues.

@fksvs
Copy link

fksvs commented Apr 23, 2025

Any chance this PR will be merged soon?

@rockdaboot
Copy link
Contributor Author

rockdaboot commented Apr 23, 2025

Any chance this PR will be merged soon?

What we need to merge beforehand to get the tests working:

@rockdaboot
Copy link
Contributor Author

@fksvs The updated list of issues / PRs that should be merged first is here.

@rockdaboot rockdaboot force-pushed the transformprocessor-profiles branch 3 times, most recently from 574ec99 to 3ea23e4 Compare April 29, 2025 11:52
@fksvs
Copy link

fksvs commented Apr 29, 2025

Thanks for the update @rockdaboot, I'm currently waiting for the PR.

@rockdaboot rockdaboot force-pushed the transformprocessor-profiles branch 2 times, most recently from 1f86f28 to e30691d Compare April 30, 2025 14:58
@rockdaboot
Copy link
Contributor Author

In case anyone cares, I split out the changes for the transformprocessor into #39974, with amended tests to succeed testing without #39681 and #39416.

@rockdaboot rockdaboot force-pushed the transformprocessor-profiles branch 2 times, most recently from 9b75909 to 6ef1b2c Compare May 13, 2025 08:41
@rockdaboot rockdaboot force-pushed the transformprocessor-profiles branch from 6ef1b2c to e71e554 Compare May 21, 2025 10:36
@github-actions
Copy link
Contributor

github-actions bot commented Jun 5, 2025

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions
Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Jun 20, 2025
@rockdaboot
Copy link
Contributor Author

This should be reopened and rebase once #39416, #39681 and #39974 have been merged.

mx-psi pushed a commit that referenced this pull request Jul 14, 2025
#### Description
PR for profiles support in the transform processor.
It's a split-out from #39036 and contains only basic functionality, that
makes this PR independent from #39681, #39416 and
open-telemetry/opentelemetry-collector#12798.
For this, several tests were commented out.

The reason for this PR is the hope to get it merged so that users and
developers can start experimenting with profiles functionality in the
transformprocessor.

Attributes can't currently can't be used with profiles (due to the above
mentioned unmerged PRs).

#### Link to tracking issue
Fixes #39009

---------

Co-authored-by: Edmo Vamerlatti Costa <[email protected]>
Co-authored-by: Tyler Helmuth <[email protected]>
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
#### Description
PR for profiles support in the transform processor.
It's a split-out from open-telemetry#39036 and contains only basic functionality, that
makes this PR independent from open-telemetry#39681, open-telemetry#39416 and
open-telemetry/opentelemetry-collector#12798.
For this, several tests were commented out.

The reason for this PR is the hope to get it merged so that users and
developers can start experimenting with profiles functionality in the
transformprocessor.

Attributes can't currently can't be used with profiles (due to the above
mentioned unmerged PRs).

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

---------

Co-authored-by: Edmo Vamerlatti Costa <[email protected]>
Co-authored-by: Tyler Helmuth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[processor/transform] Add initial support for profiles signal

3 participants