Webhook removal - WIP (#205)

* partial webhook removal * remove generated files from source control * removed gitignore * fixed keyvault controller logging * reverted generated code removal
Azure · Sep 12, 2019 · a5d91b3 · a5d91b3
1 parent 27a1bae
commit a5d91b3
Show file tree

Hide file tree

Showing 11 changed files with 45 additions and 321 deletions.
diff --git a/Makefile b/Makefile
@@ -201,5 +201,4 @@ install-test-dependency:
 	&& go get github.com/axw/gocov/gocov \
 	&& go get github.com/AlekSi/gocov-xml \
 	&& go get github.com/onsi/ginkgo/ginkgo \
-	&& go get github.com/onsi/gomega/... \
 	&& go get golang.org/x/tools/cmd/cover
diff --git a/api/v1/eventhub_webhook.go b/api/v1/eventhub_webhook.go
diff --git a/api/v1/eventhubnamespace_webhook.go b/api/v1/eventhubnamespace_webhook.go
diff --git a/api/v1/zz_generated.deepcopy.go b/api/v1/zz_generated.deepcopy.go
@@ -20,7 +20,7 @@ limitations under the License.
 package v1
 
 import (
-	"k8s.io/apimachinery/pkg/runtime"
+	runtime "k8s.io/apimachinery/pkg/runtime"
 )
 
 // DeepCopyInto is an autogenerated deepcopy function, copying the receiver, writing into out. in must be non-nil.

diff --git a/config/default/kustomization.yaml b/config/default/kustomization.yaml
@@ -35,12 +35,12 @@ patchesStrategicMerge:
 #- manager_prometheus_metrics_patch.yaml
 
 # [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in crd/kustomization.yaml
-- manager_webhook_patch.yaml
+# - manager_webhook_patch.yaml
 
 # [CAINJECTION] Uncomment next line to enable the CA injection in the admission webhooks.
 # Uncomment 'CAINJECTION' in crd/kustomization.yaml to enable the CA injection in the admission webhooks.
 # 'CERTMANAGER' needs to be enabled to use ca injection
-- webhookcainjection_patch.yaml
+# - webhookcainjection_patch.yaml
 # the following config is for teaching kustomize how to do var substitution
 # the following config is for teaching kustomize how to do var substitution
 vars: