[COMMON] Move onap truststore to cert-wrapper 71/114071/1
authorKrzysztof Opasiak <k.opasiak@samsung.com>
Tue, 20 Oct 2020 21:17:17 +0000 (23:17 +0200)
committerKrzysztof Opasiak <k.opasiak@samsung.com>
Tue, 20 Oct 2020 21:17:17 +0000 (23:17 +0200)
certInitializer is included multiple times in number of different
projects. If it contains the truststore then under if it is not used
it increases the size of the chart itself so that it our final ONAP
chart does not fit into default 20 Mb chartmuseum limit.

Let's resolve this by moving the configmap and its content to the
cert-wrapper which is included only once per onap instance.

Issue-ID: AAF-1134
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
Change-Id: I654d9158e7b776c012653dbef2c8091a393635f0

kubernetes/common/cert-wrapper/resources/import-custom-certs.sh [moved from kubernetes/common/certInitializer/resources/import-custom-certs.sh with 100% similarity]
kubernetes/common/cert-wrapper/resources/truststoreONAP.p12.b64 [moved from kubernetes/common/certInitializer/resources/truststoreONAP.p12.b64 with 100% similarity]
kubernetes/common/cert-wrapper/resources/truststoreONAPall.jks.b64 [moved from kubernetes/common/certInitializer/resources/truststoreONAPall.jks.b64 with 100% similarity]
kubernetes/common/cert-wrapper/templates/configmap.yaml [new file with mode: 0644]
kubernetes/common/certInitializer/templates/configmap.yaml
kubernetes/common/certInitializer/values.yaml

diff --git a/kubernetes/common/cert-wrapper/templates/configmap.yaml b/kubernetes/common/cert-wrapper/templates/configmap.yaml
new file mode 100644 (file)
index 0000000..117a4ab
--- /dev/null
@@ -0,0 +1,22 @@
+{{/*
+# Copyright © 2020 Samsung Electronics
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+*/}}
+
+apiVersion: v1
+kind: ConfigMap
+{{- $suffix := "certs" }}
+metadata: {{- include "common.resourceMetadata" (dict "suffix" $suffix "dot" . ) | nindent 2 }}
+data:
+{{ tpl (.Files.Glob "resources/*").AsConfig . | indent 2 }}
index 69d74e1..7abbf9c 100644 (file)
@@ -23,12 +23,3 @@ data:
   aaf-add-config.sh: |
     {{ tpl .Values.aaf_add_config . | indent 4 }}
 {{- end }}
-{{- if .Values.createCertsCM }}
----
-apiVersion: v1
-kind: ConfigMap
-{{- $suffix := "certs" }}
-metadata: {{- include "common.resourceMetadata" (dict "suffix" $suffix "dot" . ) | nindent 2 }}
-data:
-{{ tpl (.Files.Glob "resources/*").AsConfig . | indent 2 }}
-{{- end -}}
index 00e74c4..66251fa 100644 (file)
@@ -57,5 +57,8 @@ truststoreMountpath: ""
 truststoreOutputFileName: truststore.jks
 truststorePassword: changeit
 
-createCertsCM: false
-certsCMName: '{{ include "common.release" . }}-cert-initializer-certs'
+# This introduces implicit dependency on cert-wrapper
+# if you are using cert initializer cert-wrapper has to be also deployed.
+# We had to move this CM to a separate chart to reduce the total size of our charts
+# as it exceeds the default helm limits.
+certsCMName: '{{ include "common.release" . }}-cert-wrapper-certs'