cyclops
cyclops copied to clipboard
feat/cyclops-ui: Add support to fetch Deployment/StatefulSet logs
closes #257
📑 Description
Test video of Deployment/StatefulSet logs feature:
https://github.com/user-attachments/assets/643a6066-6c37-4b77-81fa-0e2ceb54e7ed
✅ Checks
- [x] I have tested my code (provide screenshots or screen recordings of a working solution)
- [x] I have performed a self-review of my code
ℹ Additional context
Reinstated the Handler functions for Deployment/StatefulSet logs i.e:
h.router.GET("/resources/deployments/:namespace/:deployment/:container/logs", modulesController.GetDeploymentLogs)
h.router.GET("/resources/statefulsets/:namespace/:name/:container/logs", modulesController.GetStatefulSetsLogs)
Tested the feature with both traditional API call using axios and the existing logstream function successfully.
Hi @petar-cvit please check this out!
@quest-bot loot #257
Quest PR submitted!
@Sheikh-Abubaker, you are attempting to solve the issue and loot this Quest. Will you be successful?
Questions? Check out the docs.
Hey @petar-cvit, just updated the branch and it is now in sync with the main.
@petar-cvit if you'd like to test the statefulset logs feature, you could utilise Grafana tempo chart: https://github.com/grafana/helm-charts/tree/main/charts/tempo/templates, and any template from the official Cyclops template repository: https://github.com/cyclops-ui/templates could be utilised to test deployment logs.
Hey @petar-cvit tuned in the feature with the refactored code!
Hey @petar-cvit looking forward to your review!
@petar-cvit I'll address the changes in the coming weeks and let you know!
Hey @petar-cvit I have refactored the code to create and reuse a single component which would fetch logs for K8s objects like Deployment or StatefulSets.
@petar-cvit I have updated the branch and it is now in sync with main!
Hey @petar-cvit just a gentle heads up, I haven't abandoned this PR, it's just that I'm having a limited bandwidth currently and will revert to working on it in the coming days, please don't close this PR 😀.
noted 👍