Migrating the Reports API from Kubernetes working group #2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is the first step in migrating the API definition from the Kubernetes Policy Working Group repository to this organization.
After copying the original files, I renamed
ClusterPolicyReport
toClusterReport
andPolicyReport
toReport
. I made these changes in theapis/reports.x-k8s.io/v1beta2
version. However, I’m wondering if we should instead implement this renaming in a new API version and take this opportunity to also change the name toapis/reports.openreports.io/v1beta2
. What are your thoughts on this?In this PR, I have not copied some directories, such as
falco-adapter
,kube-bench-adapter
,kubearmor-adapter
,oscal-transformer
, andtrivy-adapter
. I decided to exclude them because the PR is already quite large, and I would like to review the documentation more carefully to ensure they are right (I have to familiarize with the projects use of the reports 😉 ) . For this reason, I’ll add new subtasks in issue #1 to address these components.Important
I'll be on vacations for several days. Therefore, I'll not be able to respond in that period. Therefore, feel free to take over the PR. Maintainers should be able to change it if necessary.