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

[ottl] Add support for profiles #36104

Open
dmathieu opened this issue Oct 31, 2024 · 7 comments
Open

[ottl] Add support for profiles #36104

dmathieu opened this issue Oct 31, 2024 · 7 comments
Labels
data:profiles Profiles related issues help wanted Extra attention is needed pkg/ottl priority:p2 Medium

Comments

@dmathieu
Copy link
Member

Component(s)

pkg/ottl

Describe the issue you're reporting

We should add support for profiles in the ottlp package, so components relying on it may start adding profiles support as well.

Related: #35982

@dmathieu dmathieu added the needs triage New item requiring triage label Oct 31, 2024
Copy link
Contributor

Pinging code owners:

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

@dmathieu
Copy link
Member Author

/label help-wanted data:profiles priority:p2

@github-actions github-actions bot added the help wanted Extra attention is needed label Oct 31, 2024
@thiha-min-thant
Copy link

Hi @dmathieu , could you please assign to me? Thanks!

@mx-psi mx-psi added priority:p2 Medium data:profiles Profiles related issues and removed needs triage New item requiring triage labels Oct 31, 2024
@dmathieu
Copy link
Member Author

I can't assign issues. But I'd say feel free to start working on this.

@TylerHelmuth
Copy link
Member

@dmathieu is there anything stateful in OTEL profiles?

@dmathieu
Copy link
Member Author

Not in the OTLP, nor the collector.
The agent could have some statefulness, but nothing that's kept across restarts or gets propagated over the wire.

@TylerHelmuth
Copy link
Member

Ok cool. In that case to make profiles compatible with OTTL all that is needed is a new ottlprofiles context in ottl/contexts.

Once that exists other components that use OTTL can be updated to use the new profiles context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:profiles Profiles related issues help wanted Extra attention is needed pkg/ottl priority:p2 Medium
Projects
None yet
Development

No branches or pull requests

4 participants