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

Pprof #305

Merged
merged 2 commits into from
Apr 22, 2022
Merged

Pprof #305

merged 2 commits into from
Apr 22, 2022

Conversation

squat
Copy link
Owner

@squat squat commented Apr 22, 2022

This PR enhances the Kilo agent internal HTTP server to include
pprof endpoints. For simplicity, this commit migrates the internal
server creation to https://github.com/metalmatze/signal/internalserver,
which allows for easy registration of common internal server
observability endpoints.

Navigating to the root path of the HTTP server now renders a helpful HTML page like so:
image

cc @faraonc
xref: #304

Signed-off-by: Lucas Servén Marín lserven@gmail.com

This commit enhances the Kilo agent internal HTTP server to include
pprof endpoints. For simplicity, this commit migrates the internal
server creation to https://github.com/metalmatze/signal/internalserver,
which allows for easy registration of common internal server
observability endpoints.

Signed-off-by: Lucas Servén Marín <lserven@gmail.com>
Signed-off-by: Lucas Servén Marín <lserven@gmail.com>
@squat squat merged commit 0ddeea3 into main Apr 22, 2022
@squat squat deleted the pprof branch April 22, 2022 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants