-
Notifications
You must be signed in to change notification settings - Fork 254
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
pyroscope: Add relabel component for modifying and filtering profiles #2574
Conversation
💻 Deploy preview deleted. |
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
aea9afe
to
6d37eb8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for working on this. A few suggestions in the inline view, but overall pretty much ready for 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for your work on this 🙂
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
## Usage | ||
|
||
```alloy | ||
pyroscope.relabel "process" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this label be "process"
or more generic "<LABEL>"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's better. Updated in cf27661.
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/pyroscope/pyroscope.relabel.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Clayton Cornell <[email protected]>
PR Description
This PR introduces a new
pyroscope.relabel
component that allows users to modify or filter profiles using Prometheus relabeling rules. This component is particularly useful for:Standardizing labels across different profile sources
Filtering out unwanted profiles based on their labels
Adding or removing labels before forwarding profiles
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist