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

use websockets to speed up ESI UI #626

Open
tzumainn opened this issue Sep 30, 2024 · 5 comments · May be fixed by CCI-MOC/esi-ui#26
Open

use websockets to speed up ESI UI #626

tzumainn opened this issue Sep 30, 2024 · 5 comments · May be fixed by CCI-MOC/esi-ui#26
Assignees

Comments

@tzumainn
Copy link
Contributor

tzumainn commented Sep 30, 2024

The Django backend will have a websocket port listening for connections. As soon as the user logs in to the dashboard regardless if it's an esi-ui panel or not, the frontend will establish a websocket connection in esi.service.js. On connection establish, django will poll node, lease, and offer data and send it to the client at a configurable rate. esi.service.js will implement a handler function that updates the variables for lists of nodes, leases, and offers. The controllers will call a service function that returns a promise that indicates that new data has arrived, retrieve that data from the service, format the data to display with a timestamp, and then repeat.

@tzumainn
Copy link
Contributor Author

@ajamias can you populate the description of this issue? thanks!

@ajamias
Copy link
Collaborator

ajamias commented Oct 8, 2024

@tzumainn I created a draft PR that sends node data to the nodes panel using websockets here: CCI-MOC/esi-ui#26 for proof of concept. If this is satisfactory, then I'll clean it up and make it work for offers as well.

@joachimweyl
Copy link

@tzumainn What are the next steps for this?

@tzumainn
Copy link
Contributor Author

I had left some comments on the PR; I believe Austin is working on an update.

@ajamias
Copy link
Collaborator

ajamias commented Oct 23, 2024

Yes, I am still working on this. Sorry progress has been slow recently..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

3 participants