👻 fix a name changing issue and a missing scaffolding

kubernetes-sigs · May 16, 2019 · ade3794 · ade3794
1 parent 0e13ffa
commit ade3794
Show file tree

Hide file tree

Showing 9 changed files with 22 additions and 2 deletions.
diff --git a/pkg/scaffold/project.go b/pkg/scaffold/project.go
@@ -197,6 +197,7 @@ func (p *V2Project) Scaffold() error {
 		&managerv2.Kustomization{},
 		&webhook.Kustomization{},
 		&webhook.KustomizeConfigWebhook{},
+		&webhook.Service{},
 		&webhook.InjectCAPatch{},
 		&certmanager.CertManager{},
 		&certmanager.Kustomization{},

diff --git a/pkg/scaffold/v2/webhook/kustomization.go b/pkg/scaffold/v2/webhook/kustomization.go
@@ -40,7 +40,7 @@ func (c *Kustomization) GetInput() (input.Input, error) {
 }
 
 var KustomizeWebhookTemplate = `resources:
-- webhookmanifests.yaml
+- manifests.yaml
 - service.yaml
 
 configurations:

diff --git a/pkg/scaffold/v2/webhook/service.go b/pkg/scaffold/v2/webhook/service.go
@@ -49,4 +49,7 @@ spec:
   ports:
     - port: 443
       targetPort: 443
+  selector:
+    control-plane: controller-manager
+    controller-tools.k8s.io: "1.0"
 `
diff --git a/testdata/project-v2/config/crd/bases/crew.testproject.org_captains.yaml b/testdata/project-v2/config/crd/bases/crew.testproject.org_captains.yaml
@@ -1,3 +1,4 @@
+
 ---
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition

diff --git a/testdata/project-v2/config/crd/bases/crew.testproject.org_firstmates.yaml b/testdata/project-v2/config/crd/bases/crew.testproject.org_firstmates.yaml
@@ -1,3 +1,4 @@
+
 ---
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition

diff --git a/testdata/project-v2/config/rbac/role.yaml b/testdata/project-v2/config/rbac/role.yaml
@@ -1,3 +1,4 @@
+
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole

diff --git a/testdata/project-v2/config/webhook/kustomization.yaml b/testdata/project-v2/config/webhook/kustomization.yaml
@@ -1,5 +1,5 @@
 resources:
-- webhookmanifests.yaml
+- manifests.yaml
 - service.yaml
 
 configurations:

diff --git a/...t-v2/config/webhook/webhookmanifests.yaml → .../project-v2/config/webhook/manifests.yaml b/...t-v2/config/webhook/webhookmanifests.yaml → .../project-v2/config/webhook/manifests.yaml
diff --git a/testdata/project-v2/config/webhook/service.yaml b/testdata/project-v2/config/webhook/service.yaml
@@ -0,0 +1,13 @@
+
+apiVersion: v1
+kind: Service
+metadata:
+  name: webhook-service
+  namespace: system
+spec:
+  ports:
+    - port: 443
+      targetPort: 443
+  selector:
+    control-plane: controller-manager
+    controller-tools.k8s.io: "1.0"