-
Notifications
You must be signed in to change notification settings - Fork 612
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]: Help wanted: Bring Trivy integration settings into config #359
Comments
I would like to tackle this issue. How do I assign to myself? |
@briannd81 I'll assign you thanks! |
hey @briannd81 are you still working on this issue? If not I would like to take a look at it |
Please feel free @ShivangShandilya |
I would like to try and solve this issue if it's still open. |
hey @rohannair11 if you want we can both work on this issue together, I'm still exploring the project and would be amazing if we can do something together |
ofcourse! I would love to learn more about this project too |
Checklist
Is this feature request related to a problem?
No
Problem Description
Currently the settings for the Trivy integration as hard coded here
Solution Description
We should have an ability to control this outside of the code - potentially as part of the default config ( we could inject it on first run, but let users override )
I am not suggesting a solution but identifying an area for improvement.
Benefits
This would enable version to be incremented without requiring a rebuild of K8sGPT
Potential Drawbacks
No response
Additional Information
No response
The text was updated successfully, but these errors were encountered: