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.

Sign up for GitHub

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

Jump to bottom

add a --profile flag to run synse with profiling enabled #386

Closed
edaniszewski opened this issue Apr 8, 2020 · 0 comments · Fixed by #393
Closed

add a --profile flag to run synse with profiling enabled #386

edaniszewski opened this issue Apr 8, 2020 · 0 comments · Fixed by #393
Assignees
@edaniszewski
Labels
enhancement

Comments

@edaniszewski
Copy link
Contributor

This can be useful for development and debugging, but should not be used in prod.

This will require some additional changes to make it possible to extract profiling data from the container synse-server runs in:

  • update to dockerfile to put artifacts in a well-known path
  • update to server run to respect SIGINT and SIGTERM appropriately for container termination
  • addition of --profile flag to startup options
  • cProfile is built-in to python, so there shouldn't be a need for dep updates
  • updates to docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edaniszewski edaniszewski

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add --profile flag to enable simpler debug profiling vapor-ware/synse-server
1 participant
@edaniszewski