The admission webhook is missing a lot of docs and context. Aside from the code artifacts, it isn't clear what it does and why. It was also concluded with @timoreimann that the admission server should be optional.
This change:
Adds documentation in the README.md, and in the docs/admission-server.md. These documents should address the what, the why and the how.
Splits the release manifests of the digitalocean-cloud-controller-manager and digitalocean-cloud-controller-manager-admission-server in two separate directories.
The admission webhook is missing a lot of docs and context. Aside from the code artifacts, it isn't clear what it does and why. It was also concluded with @timoreimann that the admission server should be optional.
This change:
README.md
, and in thedocs/admission-server.md
. These documents should address the what, the why and the how.digitalocean-cloud-controller-manager
anddigitalocean-cloud-controller-manager-admission-server
in two separate directories.This PR should also help address some of the concerned expressed in this discussion: https://github.com/digitalocean/digitalocean-cloud-controller-manager/discussions/688