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

Stream pod logs #555

Open
petar-cvit opened this issue Sep 4, 2024 · 4 comments · May be fixed by #568
Open

Stream pod logs #555

petar-cvit opened this issue Sep 4, 2024 · 4 comments · May be fixed by #568
Assignees
Labels
go Pull requests that update Go code help wanted Extra attention is needed react typescript

Comments

@petar-cvit
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Currently, Cyclops UI supports fetching logs of a pod, but logs are not streamed in real time. The scope of the issue is to implement log streaming for pods on both the backend and frontend.

Describe the solution you'd like
You can use SSE to stream updates. #544 implements streaming for resources so similar libs can be used.

@petar-cvit petar-cvit added help wanted Extra attention is needed go Pull requests that update Go code typescript react labels Sep 4, 2024
@naineel1209
Copy link
Contributor

hi @petar-cvit, I would like to pick this up, can you please assign it me?

@petar-cvit
Copy link
Collaborator Author

Sure! You can add a checkbox to the logs modal that would indicate that the logs need to be streamed

@naineel1209
Copy link
Contributor

hi @petar-cvit, had a doubt regarding this -

Should I reuse the SSE implementation from #544 (wait for it to get merged) or work on the same branch as yours or start from scratch?

@petar-cvit
Copy link
Collaborator Author

Hey @naineel1209, you can create a branch from my branch and continue on my changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code help wanted Extra attention is needed react typescript
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants