Closed SKosier closed 2 months ago
@sboschman @leogr what's your take here? I think the conversion method genuinely offers a valuable piece of functionality useful to other projects and that perhaps we could export as part of the plugin's package. LGTM.
Comparing 9455f9022d5278e8a8a437557a6ec67b046434c8
with latest tag plugins/k8saudit-gke/v0.1.1
Major changes:
K8s Role/Clusterrole Created
has been removedK8s Role/Clusterrole Deleted
has been removedK8s Role/Clusterrolebinding Created
has been removedK8s Role/Clusterrolebinding Deleted
has been removedMinor changes:
K8s Role Created
has been addedK8s RoleBinding Deleted
has been addedK8s ClusterRoleBinding Deleted
has been addedK8s Role Deleted
has been addedK8s RoleBinding Created
has been addedK8s ClusterRole Deleted
has been addedK8s ClusterRoleBinding Created
has been addedK8s ClusterRole Created
has been addedrolebinding
has been addedI don't mind exposing the conversion method as part of the plugin sdk. Seems a bit strange to just expose a random internal function b/c it might be useful without any method signature guarantee or being documented as part of the sdk.
As sdk changes are a bit of work, the method can be made public for the time being. Just add a comment indicating that it is an internal method without any signature guarantees. So use at your own risk.
I don't mind exposing the conversion method as part of the plugin sdk. Seems a bit strange to just expose a random internal function b/c it might be useful without any method signature guarantee or being documented as part of the sdk.
As sdk changes are a bit of work, the method can be made public for the time being. Just add a comment indicating that it is an internal method without any signature guarantees. So use at your own risk.
Thank you @sboschman for the review. I applied the feedback and added the comment above the method signature.
Comparing 1781880ca70c766c0f063a84bea3b7ee865a68e7
with latest tag plugins/k8saudit-gke/v0.1.1
Major changes:
K8s Role/Clusterrolebinding Created
has been removedK8s Role/Clusterrolebinding Deleted
has been removedK8s Role/Clusterrole Created
has been removedK8s Role/Clusterrole Deleted
has been removedMinor changes:
K8s ClusterRoleBinding Deleted
has been addedK8s ClusterRole Deleted
has been addedK8s RoleBinding Deleted
has been addedK8s Role Created
has been addedK8s Role Deleted
has been addedK8s RoleBinding Created
has been addedK8s ClusterRoleBinding Created
has been addedK8s ClusterRole Created
has been addedrolebinding
has been added[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sboschman, SKosier
The full list of commands accepted by this bot can be found here.
The pull request process is described here
LGTM label has been added.
What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area plugins
What this PR does / why we need it:
For the project I have we are ingesting gke events in other way and we would like to leverage already existing OSS code for converting gke events to
k8s_audit
ones, due to maintainability reasons.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer: