-
Notifications
You must be signed in to change notification settings - Fork 560
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
[Feature]: Enhance Caching Functionality #944
Comments
Can you help us understand what the actual use case might be for this? |
Yeah, sure,
By incorporating the features mentioned above, we can achieve the same behaviour as in other auth, filter, and integration commands. This approach would lessen the need to memorize various details for cache functionality, as the same behaviour would be consistent across all commands. So updating the |
I believe I understand the enhancement you want to make, I would be supportive of this work. |
Checklist
Is this feature request related to a problem?
None
Problem Description
Consistency is key for K8sGPT, as it ensures ease of use and speed. Therefore, it would be beneficial to implement Cmd logging functionality for cache similar to what is done for auth, integration, and filter commands.
While the current implementation of adding caching functionality is commendable, there are some necessary improvements that can be made:
Issues with the current caching setup include:
Solution Description
1) List command:
Update a list command for caching that functions similarly to auth, filter, and integration list commands.
k8sgpt cache list
2) Data command:
Introduce a command that retrieves all cache data along with the name of the used cache provider.
k8sgpt cache data
3) Add & Remove command:
Prevent the addition and removal of the same cache provider multiple times.
Benefits
Improved visibility: Users will have better insight into which cache provider stores the data, enhancing transparency.
Prevents redundancy: By disallowing the addition of the same cache provider multiple times, we can avoid redundancy in caching, leading to better resource utilization.
Enhanced user experience: Implementing these improvements will streamline the caching process, resulting in a smoother user experience and improved performance.
Potential Drawbacks
No response
Additional Information
Feel free to provide any further context or insights related to this caching enhancement request. Your feedback and suggestions are highly valued as we work towards improving our project's caching functionality for the benefit of all users.
The text was updated successfully, but these errors were encountered: