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

[EPIC] Introduce CPU usage profiling #2364

Open
3 tasks
mayaCostantini opened this issue Jul 19, 2022 · 1 comment
Open
3 tasks

[EPIC] Introduce CPU usage profiling #2364

mayaCostantini opened this issue Jul 19, 2022 · 1 comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/stack-guidance Categorizes an issue or PR as relevant to SIG Stack Guidance.

Comments

@mayaCostantini
Copy link
Contributor

mayaCostantini commented Jul 19, 2022 •
edited
Loading

Problem statement

It would be useful to implement CPU usage profiling of the adviser to identify bottlenecks in the resolution process and work on optimizing the most computationally expensive parts.

Profiling already exists for memory (see https://github.com/thoth-station/adviser/blob/master/thoth/adviser/boots/_debug/memtrace.py)

Acceptance Criteria

  • Research the most appropriate tools to perform profiling
  • Implement a module to monitor CPU usage for different parts of the adviser during an advise run
  • Make adviser runs more observable when performing CPU usage profiling, computing metrics about performance which could eventually be synced in Thoth's database for further analysis and comparisons between versions of adviser
@mayaCostantini mayaCostantini added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 19, 2022
@mayaCostantini mayaCostantini changed the title [SpikeIntroduce CPU usage profiling [Spike]Introduce CPU usage profiling Jul 19, 2022
@mayaCostantini
Copy link
Contributor Author

/priority important-longterm
/sig stack-guidance

@sesheta sesheta added priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/stack-guidance Categorizes an issue or PR as relevant to SIG Stack Guidance. labels Jul 19, 2022
@mayaCostantini mayaCostantini changed the title [Spike]Introduce CPU usage profiling [EPIC] Introduce CPU usage profiling Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. sig/stack-guidance Categorizes an issue or PR as relevant to SIG Stack Guidance.
Projects
Planning Board
Status: 🆕 New
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sesheta @mayaCostantini