Skip to content

Conversation

karanBRAVO
Copy link
Member

Describe the PR

As discussed on volcano-slack, I have implemented the real-time update feature.

  • Now, we do not have to refresh button(s) again and again to see the changes :)
  • This PR ensures that Job, Pods, Queues, Namespaces are updated in real-time.
  • Along with that, I also update the Age of the Pods.

Additional Context

You can test this PR by creating jobs, pods, queues, namespaces using kubectl and see the real-time updates on the UI.

apiVersion: v1
kind: Namespace
metadata:
  name: my-namespace
kubectl apply -f <your-file-name>.yaml

Also try deleting it

kubectl delete namespace my-namespace

Signed-off-by: Karan Yadav <[email protected]>
@volcano-sh-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign kevin-wangzefeng
You can assign the PR to them by writing /assign @kevin-wangzefeng in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@karanBRAVO
Copy link
Member Author

/assign @kevin-wangzefeng
/assign @Monokaix

@karanBRAVO
Copy link
Member Author

Hello @Monokaix @JesseStutler @de6p ,
I would love to get feedback on this PR.
Thanks
Karan Yadav

@RONAK-AI647
Copy link

RONAK-AI647 commented May 24, 2025

While the PR adds real-time updates globally, but it still offer manual refresh buttons on each tab (Jobs, Queues, Pods) for fallback, in case WebSocket or polling fails ??? @karanBRAVO

@karanBRAVO
Copy link
Member Author

Ya, that's true

Reason

  • Reliability: Real-time features can fail silently (e.g., due to network issues), so fallback refresh buttons ensure users can still get up-to-date information.

  • User control: Gives users the power to manually update if something looks stale.

  • Progressive enhancement: Adds features without removing existing functionality that users may rely on.

And by the way, I am using Server Sent Events in this case.
short-desc - it is used when we only one-way connection and in most cases from server to client.

Hope that this answer's your query :)

@RONAK-AI647
Copy link

yes , that's cool @karanBRAVO

@Monokaix
Copy link
Member

I think we can move this forward after @Shrutim1505 finish her work.

@karanBRAVO
Copy link
Member Author

I think we can move this forward after @Shrutim1505 finish her work.

ok

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

Successfully merging this pull request may close these issues.

5 participants