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

feat: kyverno operator support #1210

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

ronaldpetty
Copy link
Contributor

@ronaldpetty ronaldpetty commented Jul 29, 2024

📑 Description

Initial attempt to integrate Kyverno with K8sGPT Operator

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

Needs peer review and additional fixes (especially around rpcs).

* serialized "RunAnalysis" due to bug in Kyverno client
* use of ronaldpetty instead of k8sgpt-ai rpcs
* updated go.mod relating to above
* serialized "RunAnalysis" due to bug in Kyverno client
* use of ronaldpetty instead of k8sgpt-ai rpcs
* updated go.mod relating to above

Signed-off-by: Ronald Petty <[email protected]>
@ronaldpetty ronaldpetty requested review from a team as code owners July 29, 2024 03:41
ronaldpetty and others added 20 commits August 17, 2024 16:40
Signed-off-by: yanweili <[email protected]>
Co-authored-by: yanweili <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
…1197)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
…k8sgpt-ai#1145)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Alex Jones <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
* chore: initial custom analysis

Signed-off-by: Alex Jones <[email protected]>

* chore: saving the missing stuff

Signed-off-by: Alex Jones <[email protected]>

* chore: removed redundant error handling

Signed-off-by: Alex Jones <[email protected]>

* chore: missing piece of the custom analysis response

Signed-off-by: Alex Jones <[email protected]>

---------

Signed-off-by: Alex Jones <[email protected]>
Signed-off-by: AlexsJones <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
…-ai#1214)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
…k8sgpt-ai#1127)

Signed-off-by: Bo Wang <[email protected]>
Signed-off-by: Alex Jones <[email protected]>
Co-authored-by: Alex Jones <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
* feat: add custom analyzer management capability

Introduced the ability to manage custom analyzers in the K8sGPT application, enabling users to add, deploy, and configure custom analyzers from various sources. This enhancement supports extending the application's analytical capabilities by integrating external analysis tools, thus offering more flexibility and customization options to meet specific user needs.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: enhance custom analyzer management with removal functionality

Introduced the ability to remove custom analyzers, streamlining the management process and ensuring flexibility in custom analyzer configuration. This enhancement addresses the need for dynamic customization and maintenance of analyzer setups, facilitating easier updates and modifications to the analysis environment.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: add list command to customAnalyzer for displaying configured analyzers

Implemented a new list command within the customAnalyzer module to enable users to view all configured custom analyzers. This enhancement aims to improve usability by providing a straightforward method for users to inspect their custom analyzer configurations directly from the command line.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: add support for listing, adding, and removing custom analyzers

This update introduces commands to manage custom analyzers in the k8sgpt tool, enhancing flexibility and control over analyzer configurations without the need for direct installation or docker dependency.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: support private docker image authentication for custom analyzers

Added authentication support for pulling private Docker images when adding custom analyzers, enhancing security and access control.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: remove Docker custom analyzer installation

Removed the installation and deployment functionality for custom analyzers, streamlining the process of adding analyzers. This change focuses on simplifying the configuration by eliminating the need for specifying installation types, package URLs, and authentication details for Docker images. The goal is to enhance user experience by making the addition of custom analyzers more straightforward and less error-prone.

Signed-off-by: Matthis Holleville <[email protected]>

* fix: remove unused packageUrl

Signed-off-by: Matthis Holleville <[email protected]>

* feat: update add command description to reflect broader functionality

Signed-off-by: Matthis Holleville <[email protected]>

* feat: Add name validation for custom analyzer creation

To ensure the integrity and consistency of analyzer names, we introduced a validation step that checks the format of the name against a predefined regex pattern. This change aims to prevent the creation of analyzers with invalid names, enhancing the system's reliability and usability.

Signed-off-by: Matthis Holleville <[email protected]>

* feat: refactor customAnalyzer package for consistent naming

Refactored the customAnalyzer package and its references to use consistent snake_case naming for improved code readability and alignment with Go naming conventions.

Signed-off-by: Matthis Holleville <[email protected]>

---------

Signed-off-by: Matthis Holleville <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
Signed-off-by: magicsong <[email protected]>

Co-authored-by: magicsong <[email protected]>
Co-authored-by: Alex Jones <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
…ible [security] (k8sgpt-ai#1220)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
* feat: refactoring to the new schema

Signed-off-by: AlexsJones <[email protected]>

* chore: updated readme with grpc commands

Signed-off-by: AlexsJones <[email protected]>

* chore: updated deps

Signed-off-by: AlexsJones <[email protected]>

---------

Signed-off-by: AlexsJones <[email protected]>
Co-authored-by: Matthis <[email protected]>
…1224)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
…12 (k8sgpt-ai#1226)

Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Ronald Petty <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
@ronaldpetty
Copy link
Contributor Author

modified to use normal schema repo

@ronaldpetty ronaldpetty changed the title feat: Kyverno operator support feat: kyverno operator support Aug 18, 2024
Signed-off-by: Ronald Petty <[email protected]>
Signed-off-by: Ronald Petty <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Proposed
Development

Successfully merging this pull request may close these issues.

8 participants