From b2e646064f5556261e5c68431df5991e4eb8d2cb Mon Sep 17 00:00:00 2001
From: Dario Tranchitella <dario@tranchitella.eu>
Date: Sat, 3 Dec 2022 11:10:39 +0100
Subject: [PATCH] fix(helm): switching over webhook server service

---
 charts/kamaji/README.md                 |  2 --
 charts/kamaji/templates/controller.yaml |  3 +++
 charts/kamaji/templates/service.yaml    | 16 ----------------
 charts/kamaji/values.yaml               |  4 ----
 4 files changed, 3 insertions(+), 22 deletions(-)
 delete mode 100644 charts/kamaji/templates/service.yaml

diff --git a/charts/kamaji/README.md b/charts/kamaji/README.md
index 68c9e4fd..b7a69f32 100644
--- a/charts/kamaji/README.md
+++ b/charts/kamaji/README.md
@@ -125,8 +125,6 @@ Here the values you can override:
 | resources.requests.cpu | string | `"100m"` |  |
 | resources.requests.memory | string | `"20Mi"` |  |
 | securityContext | object | `{"allowPrivilegeEscalation":false}` | The securityContext to apply to the Kamaji controller container only. It does not apply to the Kamaji RBAC proxy container. |
-| service.port | int | `8443` |  |
-| service.type | string | `"ClusterIP"` |  |
 | serviceAccount.annotations | object | `{}` |  |
 | serviceAccount.create | bool | `true` |  |
 | serviceAccount.name | string | `"kamaji-controller-manager"` |  |
diff --git a/charts/kamaji/templates/controller.yaml b/charts/kamaji/templates/controller.yaml
index 8765e12c..36bb7ee4 100644
--- a/charts/kamaji/templates/controller.yaml
+++ b/charts/kamaji/templates/controller.yaml
@@ -50,6 +50,9 @@ spec:
         {{- end }}
         name: manager
         ports:
+        - containerPort: 9443
+          name: webhook-server
+          protocol: TCP
         - containerPort: 8081
           name: healthcheck
           protocol: TCP
diff --git a/charts/kamaji/templates/service.yaml b/charts/kamaji/templates/service.yaml
deleted file mode 100644
index 2509a0d9..00000000
--- a/charts/kamaji/templates/service.yaml
+++ /dev/null
@@ -1,16 +0,0 @@
-apiVersion: v1
-kind: Service
-metadata:
-  name: {{ include "kamaji.fullname" . }}
-  labels:
-    {{- include "kamaji.labels" . | nindent 4 }}
-  namespace: {{ .Release.Namespace }}
-spec:
-  type: {{ .Values.service.type }}
-  ports:
-  - name: https
-    port: {{ .Values.service.port }}
-    protocol: TCP
-    targetPort: https
-  selector:
-    {{- include "kamaji.selectorLabels" . | nindent 4 }}
diff --git a/charts/kamaji/values.yaml b/charts/kamaji/values.yaml
index 220c1651..bbaf7508 100644
--- a/charts/kamaji/values.yaml
+++ b/charts/kamaji/values.yaml
@@ -124,10 +124,6 @@ securityContext:
   # runAsNonRoot: true
   # runAsUser: 1000
 
-service:
-  type: ClusterIP
-  port: 8443
-
 resources:
   limits:
     cpu: 200m