Move webhooks into pkg/webhooks

Moves webhooks from api to pkg/webhooks making only mechanical code
changes except for the removal of the defaulting webhooks, because they
weren't used. This results in there now being no mutating webhook
configured.

NOTE(stephenfin): There were a lot of conflicts here. These were mostly
mitigated by faking the addition of v1alpha8, which moved the webhooks
to the 'api/v1alpha8' package (commit 750b84d), followed by the
subsequent rename of this package to v1beta1 (commit e9fb53c), for
the webhook files and tests. This still resulted in some merge conflicts
due the v1alpha8 changes such as 564b6bd and 4368c4f (which we
obviously don't want to include here) but it made the backport much
simpler.

NOTE(stephenfin): We also include commit
30ba121 which was a follow-up that
fixed the CRD generation broken in this commit.

Signed-off-by: Stephen Finucane <[email protected]>
(cherry picked from commit 750b84d)

Author: stephenfin

Makefile

@@ -269,7 +269,9 @@ generate-manifests: $(CONTROLLER_GEN) ## Generate manifests e.g. CRD, RBAC etc.
 	$(CONTROLLER_GEN) \
 		paths=./api/... \
 		crd:crdVersions=v1 \
-		output:crd:dir=$(CRD_ROOT) \
+		output:crd:dir=$(CRD_ROOT)
+	$(CONTROLLER_GEN) \
+		paths=./pkg/webhooks/... \
 		output:webhook:dir=$(WEBHOOK_ROOT) \
 		webhook
 	$(CONTROLLER_GEN) \

api/v1alpha7/identity_types.go

@@ -16,8 +16,6 @@ limitations under the License.
 
 package v1alpha7
 
-const defaultIdentityRefKind = "Secret"
-
 // OpenStackIdentityReference is a reference to an infrastructure
 // provider identity to be used to provision cluster resources.
 type OpenStackIdentityReference struct {