Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enable pprof endpoints on metrics server #282

Merged
merged 1 commit into from
Feb 7, 2021
Merged

Enable pprof endpoints on metrics server #282

merged 1 commit into from
Feb 7, 2021

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Feb 6, 2021

Depends on fluxcd/pkg#81, will hopefully shine more light on #192.

Ref: fluxcd/flux2#873

@hiddeco hiddeco marked this pull request as draft February 6, 2021 23:03
Using the helper from `pkg/runtime/pprof`, which follows the suggestion
from controller-runtime to use `AddMetricsExtraHandler`.

Signed-off-by: Hidde Beydals <hello@hidde.co>
@hiddeco hiddeco requested a review from stefanprodan February 7, 2021 10:44
@hiddeco hiddeco marked this pull request as ready for review February 7, 2021 10:44
@hiddeco hiddeco added the area/ci CI related issues and pull requests label Feb 7, 2021
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

After we enable pprof in all controllers, we should create a troubleshooting doc and explain to users how to port forward and dump the cpu/memory profiles.

@hiddeco
Copy link
Member Author

hiddeco commented Feb 7, 2021

After we enable pprof in all controllers, we should create a troubleshooting doc and explain to users how to port forward and dump the cpu/memory profiles.

Agreed, there won't be coming more from me today however as we had our first 10-15cm of snow in years in the Netherlands, and I am planning on heading outside :-)

@hiddeco hiddeco merged commit 0465b12 into main Feb 7, 2021
@hiddeco hiddeco deleted the pprof branch February 7, 2021 11:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/ci CI related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants