kubewarden / kubewarden-controller

Manage admission policies in your Kubernetes cluster with ease
https://kubewarden.io
Apache License 2.0
180 stars 30 forks source link

Controller restructuring #769

Open flavio opened 1 week ago

flavio commented 1 week ago

Kubewarden controller quality cleanup: Spike: Restructure the project to allow kubebuilder generators to be able to find the right folders, revisit how the code is structured. Take inspiration from https://github.com/spinkube/runtime-class-manager

This Spike will produce workable items as an output.

jvanz commented 1 week ago

I'm fine with the proposed changes. I just have a suggestion about the following:

Rename admissionregistration package in crypto and remove usage of generateCertFunc inversion of control

Can we reuse this code as starter point? As far as I can remember, I've removed those generation functions and add tests for the whole package

fabriziosestito commented 1 week ago

@jvanz good point, added to the issue