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

Split clusters and jobs/nodes retrieval to minimize rendering time of clusters list #428

Open
rezib opened this issue Jan 2, 2025 · 0 comments · May be fixed by #435
Open

Split clusters and jobs/nodes retrieval to minimize rendering time of clusters list #428

rezib opened this issue Jan 2, 2025 · 0 comments · May be fixed by #435
Assignees
@rezib
Labels
refactor Code refactoring to reduce maintenance cost and technical debt
Milestone
v4.1.0

Comments

@rezib
Copy link
Contributor

rezib commented Jan 2, 2025

Clusters list can take time to render with many jobs are in schedulers' queue. We should split retrieval of clusters list and nodes/jobs stats in multiple requests so users can quickly get the list of clusters before the jobs/nodes stats are retrieved.
@rezib rezib added the bug This was supposed to work in this case and clearly it is not. label Jan 2, 2025
@rezib rezib added this to the v4.1.0 milestone Jan 2, 2025
@rezib rezib self-assigned this Jan 2, 2025
@rezib rezib added refactor Code refactoring to reduce maintenance cost and technical debt and removed bug This was supposed to work in this case and clearly it is not. labels Jan 10, 2025
rezib added a commit that referenced this issue Jan 10, 2025
@rezib
refactor(front): async clusters stats retrieval
b7ff5e4 
Instead of expecting the clusters stats in response along the clusters
list, send asynchronous requests to retrieve the stats on all clusters
after the list of clusters is rendered to users.

fix #428
@rezib rezib linked a pull request Jan 10, 2025 that will close this issue
Open
3 tasks
rezib added a commit that referenced this issue Jan 10, 2025
@rezib
refactor(front): async clusters stats retrieval
750a94e 
Instead of expecting the clusters stats in response along the clusters
list, send asynchronous requests to retrieve the stats on all clusters
after the list of clusters is rendered to users.

fix #428
rezib added a commit that referenced this issue Jan 10, 2025
@rezib
refactor(front): async clusters stats retrieval
9007217 
Instead of expecting the clusters stats in response along the clusters
list, send asynchronous requests to retrieve the stats on all clusters
after the list of clusters is rendered to users.

fix #428
rezib added a commit that referenced this issue Jan 17, 2025
@rezib
refactor(front): async clusters stats retrieval
1855be9 
Instead of expecting the clusters stats in response along the clusters
list, send asynchronous requests to retrieve the stats on all clusters
after the list of clusters is rendered to users.

fix #428
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rezib rezib

Labels
refactor Code refactoring to reduce maintenance cost and technical debt
Projects
None yet
Milestone
v4.1.0
Development

Successfully merging a pull request may close this issue.

Asynchronous requests to retrieve stats in clusters list rackslab/Slurm-web
1 participant
@rezib