You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As of today controllers are split into serveral deployments like statefulset-runner, kpack-builder, job-task-runner. This is not necessary and causes some pain when it comes to maintaining the Dockerfiles and generating webhook certificates.
The separation was done so that controllers can be scaled independently and so that rbac permissions are more fine grain. However we haven't really taken advantage of this so far.
Action to take
Merge all controllers back into a single deployment.
Dev Notes
We should keep the current helm value as much as possible structure to avoid breaking changes
We should keep the ability to opt out of some controllers (e.g. statefulset runner). This will be a conditional branch in the common controllers main
We should squash controllers Dockerfiles back into one
Background
As of today controllers are split into serveral deployments like
statefulset-runner,kpack-builder,job-task-runner. This is not necessary and causes some pain when it comes to maintaining the Dockerfiles and generating webhook certificates.The separation was done so that controllers can be scaled independently and so that rbac permissions are more fine grain. However we haven't really taken advantage of this so far.
Action to take
Merge all controllers back into a single deployment.
Dev Notes