Merge pull request #2223 from pubudu538/cc-redis

Add conversion webhook handling to the helm chart template
wso2 · Apr 2, 2024 · aa55725 · aa55725
2 parents e81084b + c017466
commit aa55725
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/helm-charts/templates/crds/dp.wso2.com_apis.yaml b/helm-charts/templates/crds/dp.wso2.com_apis.yaml
@@ -7,6 +7,7 @@ metadata:
     cert-manager.io/inject-ca-from: {{ .Release.Namespace }}/{{ template "apk-helm.resource.prefix" . }}-common-controller-server-cert
   name: apis.dp.wso2.com
 spec:
+  {{- if .Values.wso2.apk.webhooks.conversionwebhookconfigurations }}
   conversion:
       strategy: Webhook
       webhook:
@@ -17,6 +18,7 @@ spec:
             path: /convert
         conversionReviewVersions:
         - v1
+  {{- end }}
   group: dp.wso2.com
   names:
     kind: API

diff --git a/helm-charts/values.yaml b/helm-charts/values.yaml
@@ -22,6 +22,7 @@ wso2:
     webhooks:
       validatingwebhookconfigurations: true
       mutatingwebhookconfigurations: true
+      conversionwebhookconfigurations: true
     auth:
       enabled: true
       enableServiceAccountCreation: true
@@ -250,6 +251,9 @@ wso2:
 #                skipSSLVerification: false
 #                hostnameVerifier: "AllowAll"
 
+    cp:
+      enabled: false
+
     metrics:
       enabled: false
       # configDSBalHost: 0.0.0.0

diff --git a/helm-charts/values.yaml.template b/helm-charts/values.yaml.template
@@ -24,6 +24,7 @@ wso2:
     webhooks:
       validatingwebhookconfigurations : true
       mutatingwebhookconfigurations : true
+      conversionwebhookconfigurations : true
     auth:
       # -- Enable Service Account Creation
       enabled: true