-
Notifications
You must be signed in to change notification settings - Fork 6
chore: remove invalid field #51
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
base: main
Are you sure you want to change the base?
Conversation
It is an alloy field, not a grafana agent one
version: 1.14.1 | ||
version: 1.14.2 |
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.
This got bumped
@@ -118,8 +118,6 @@ global: | |||
scrape_interval: 60s | |||
# global.telemetry.profiling.scrape_timeout -- how long a request will be allowed to wait before being canceled | |||
scrape_timeout: 70s | |||
# global.telemetry.profiling.delta_profiling_duration -- duration of each pprof profiling capture, must be less than scrape_interval | |||
delta_profiling_duration: 30s |
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.
Shouldn't we be setting delta=true
?
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.
I don't think so. That argument is for profile.custom
blocks.
https://grafana.com/docs/agent/latest/flow/reference/components/pyroscope.scrape/#delta-argument
Here are the arguments for pyroscope.scrape
: https://grafana.com/docs/agent/latest/flow/reference/components/pyroscope.scrape/#arguments
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.
At least, I am not sure how to set up delta
and where to put the custom profile block 🤔.
I assume we are using some standard profiling configuration with the /debug/pprof/profile
endpoint. I'd rather not manually configure it if we can avoid it.
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.
OK cool, let's see how this fares.
It is an alloy field, not a grafana agent one