Revert "chore: add webhook toggle" (#334)

This reverts commit 4ab0d88.

Makefile

@@ -152,7 +152,6 @@ e2etest: generate local-release local-deploy chainsaw
 
 local-deploy: kind ctlptl tilt kustomize clusterctl
 	@echo -n "LINODE_TOKEN=$(LINODE_TOKEN)" > config/default/.env.linode
-	@echo -n "ENABLE_WEBHOOKS=$(ENABLE_WEBHOOKS)" > config/default/.env.manager
 	$(CTLPTL) apply -f .tilt/ctlptl-config.yaml
 	$(TILT) ci -f Tiltfile
 
@@ -205,7 +204,6 @@ endif
 .PHONY: tilt-cluster
 tilt-cluster: ctlptl tilt kind clusterctl
 	@echo -n "LINODE_TOKEN=$(LINODE_TOKEN)" > config/default/.env.linode
-	@echo -n "ENABLE_WEBHOOKS=$(ENABLE_WEBHOOKS)" > config/default/.env.manager
 	$(CTLPTL) apply -f .tilt/ctlptl-config.yaml
 	$(TILT) up --stream
 

Tiltfile

@@ -98,8 +98,6 @@ for resource in manager_yaml:
         resource["stringData"]["apiToken"] = os.getenv("LINODE_TOKEN")
     if resource["kind"] == "CustomResourceDefinition" and resource["spec"]["group"] == "infrastructure.cluster.x-k8s.io":
         resource["metadata"]["labels"]["clusterctl.cluster.x-k8s.io"] = ""
-    if resource["metadata"]["name"] == "capl-manager-config":
-        resource["data"]["ENABLE_WEBHOOKS"] = os.getenv("ENABLE_WEBHOOKS", "true")
 k8s_yaml(encode_yaml_stream(manager_yaml))
 
 if os.getenv("SKIP_DOCKER_BUILD", "false") != "true":
@@ -130,7 +128,6 @@ k8s_resource(
         "capl-manager-rolebinding:clusterrolebinding",
         "capl-proxy-rolebinding:clusterrolebinding",
         "capl-manager-credentials:secret",
-        "capl-manager-config:configmap",
         "capl-serving-cert:certificate",
         "capl-selfsigned-issuer:issuer",
         "capl-validating-webhook-configuration:validatingwebhookconfiguration",

config/default/.gitignore

@@ -1,2 +1 @@
 .env.linode
-.env.manager

config/default/kustomization.yaml

@@ -13,7 +13,6 @@ resources:
 - ../rbac
 - ../manager
 - linode-token-secret.yaml
-- manager-configmap.yaml
 # [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in
 # crd/kustomization.yaml
 - ../webhook
@@ -36,26 +35,9 @@ patches:
 # 'CERTMANAGER' needs to be enabled to use ca injection
 - path: webhookcainjection_patch.yaml
 
-replacements:
-
-# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in
-# crd/kustomization.yaml
- - source:
-     kind: ConfigMap
-     version: v1
-     name: manager-config # this name should match the one in manager-configmap.yaml
-     fieldPath: .metadata.name
-   targets:
-     - select:
-         kind: Deployment
-         group: apps
-         version: v1
-         name: controller-manager
-       fieldPaths:
-         - .spec.template.spec.containers.[name=manager].env.[name=ENABLE_WEBHOOKS].valueFrom.configMapKeyRef.name
-
 # [CERTMANAGER] To enable cert-manager, uncomment all sections with 'CERTMANAGER' prefix.
 # Uncomment the following replacements to add the cert-manager CA injection annotations
+replacements:
  - source: # Add cert-manager annotation to ValidatingWebhookConfiguration, MutatingWebhookConfiguration and CRDs
      kind: Certificate
      group: cert-manager.io

config/default/manager_webhook_patch.yaml

@@ -8,12 +8,6 @@ spec:
     spec:
       containers:
       - name: manager
-        env:
-        - name: ENABLE_WEBHOOKS
-          valueFrom:
-            configMapKeyRef:
-              name: manager-config
-              key: ENABLE_WEBHOOKS
         ports:
         - containerPort: 9443
           name: webhook-server