Skip to content

feat: pyroscope.enrich component#4797

Merged
dehaansa merged 3 commits intografana:mainfrom
AndreZiviani:pyroscope-enrich
Nov 20, 2025
Merged

feat: pyroscope.enrich component#4797
dehaansa merged 3 commits intografana:mainfrom
AndreZiviani:pyroscope-enrich

Conversation

@AndreZiviani
Copy link
Contributor

PR Description

Adds pyroscope.enrich component based on #2882 and other pyroscope components, it allows you to add labels retrieved from discovery.* to the resulting profiles

Which issue(s) this PR fixes

Fixes #4767

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@AndreZiviani AndreZiviani requested review from a team and clayton-cornell as code owners November 6, 2025 19:50
@CLAassistant
Copy link

CLAassistant commented Nov 6, 2025

CLA assistant check
All committers have signed the CLA.

@AndreZiviani
Copy link
Contributor Author

hey @clayton-cornell , any other changes required? can you allow the ci/tests to run?

Copy link
Contributor

@dehaansa dehaansa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, one nit, linter CI & merge in main to fix the failing test.

@AndreZiviani
Copy link
Contributor Author

AndreZiviani commented Nov 19, 2025

@dehaansa ci errors seems unrelated to my changes, let me know when you can review it again so that I can rebase and fix merge conflicts

Copy link
Contributor

@dehaansa dehaansa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, merge/rebase for changelog and we can get this merged.

AndreZiviani and others added 2 commits November 20, 2025 11:58
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
@AndreZiviani
Copy link
Contributor Author

done @dehaansa , thanks!

@dehaansa dehaansa enabled auto-merge (squash) November 20, 2025 16:46
@dehaansa dehaansa merged commit cc7448f into grafana:main Nov 20, 2025
38 of 39 checks passed
@dehaansa
Copy link
Contributor

done @dehaansa , thanks!

Thanks for the contribution!

@clayton-cornell clayton-cornell added the type/docs Docs Squad label across all Grafana Labs repos label Nov 20, 2025
dehaansa added a commit to madhub/alloy that referenced this pull request Dec 10, 2025
* feat: pyroscope.enrich component

* chore: Improve docs

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Sam DeHaan <sam.dehaan@grafana.com>
@AndreZiviani
Copy link
Contributor Author

hey @dehaansa my contribution was removed from changelog but the code itself is still on the repo, is this expected? there have been multiple releases since this PR, when can I expect to have a release with this feature?

@dehaansa
Copy link
Contributor

@AndreZiviani Sorry for the confusion! We've recently updated to using release-please to generate the changelog so unreleased content is no longer listed in the changelog before release.

Sadly 1.12 release started in mid november before this PR was merged, it had some release process issues so the actual release was delayed, but we didn't add non-bugfixes afterwards.

The pyroscope.enrich component will land in 1.13, early February 2026 is the current target.

@AndreZiviani
Copy link
Contributor Author

I see, thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age type/docs Docs Squad label across all Grafana Labs repos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pyroscope.receive_http: enrich received profiles

4 participants