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

Add py-spy to hub image #1327

Merged
merged 1 commit into from
Jul 11, 2019
Merged

Add py-spy to hub image #1327

merged 1 commit into from
Jul 11, 2019

Conversation

yuvipanda
Copy link
Collaborator

https://github.com/benfred/py-spy is a sampling profiler for Python
that works really solidly with minimum fuss, and has been extremely
helpful in debugging issues. It's also very useful in doing performance
analysis. Having this be present in the hub image by default will
make it easier for everyone to create profiles of the hub process
at loads they are having trouble with, and share it with the community
to see what is up.

https://github.com/benfred/py-spy is a sampling profiler for Python
that works really solidly with minimum fuss, and has been extremely
helpful in debugging issues. It's also very useful in doing performance
analysis. Having this be present in the hub image by default will
make it easier for everyone to create profiles of the hub process
at loads they are having trouble with, and share it with the community
to see what is up.
@minrk minrk merged commit 733d771 into master Jul 11, 2019
@minrk
Copy link
Member

minrk commented Jul 11, 2019

Nice!

@consideRatio
Copy link
Member

consideRatio commented Jul 11, 2019

Cool tool! I'll find use for this now as well!

( The CI Pipeline for the PR failed, related to this being a in-repo-branch-pr, but the actual pipeline triggered on the merge to master ran as it should, so there should be a new helm chart release with this image bumped still )

@yuvipanda yuvipanda deleted the yuvipanda-patch-1 branch November 6, 2019 22:48
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

3 participants