Merge "[DCAEGEN2] Update CMPv2 certs usage in dcaegen2-services"
authorSylvain Desbureaux <sylvain.desbureaux@orange.com>
Fri, 28 May 2021 19:52:04 +0000 (19:52 +0000)
committerGerrit Code Review <gerrit@onap.org>
Fri, 28 May 2021 19:52:04 +0000 (19:52 +0000)
1  2 
kubernetes/dcaegen2-services/common/dcaegen2-services-common/templates/_deployment.tpl
kubernetes/dcaegen2-services/components/dcae-ves-collector/values.yaml

@@@ -274,10 -274,6 +274,10 @@@ spec
          imagePullPolicy: {{ .Values.global.pullPolicy | default .Values.pullPolicy }}
          name: {{ include "common.name" . }}
          env:
 +        {{- range $cred := .Values.credentials }}
 +        - name: {{ $cred.name }}
 +          {{- include "common.secret.envFromSecretFast" (dict "global" $ "uid" $cred.uid "key" $cred.key) | indent 10 }}
 +        {{- end }}
          {{- if $certDir }}
          - name: DCAE_CA_CERTPATH
            value: {{ $certDir }}/cacert.pem
          volumeMounts:
          - mountPath: /app-config
            name: app-config
 +        - mountPath: /app-config-input
 +          name: app-config-input
          {{- if $logDir }}
          - mountPath: {{ $logDir}}
            name: component-log
          {{- if $certDir }}
          - mountPath: {{ $certDir }}
            name: tls-info
-           {{- if and .Values.certificates .Values.global.cmpv2Enabled .Values.global.CMPv2CertManagerIntegration -}}
+           {{- if (include "dcaegen2-services-common.shouldUseCmpv2Certificates" .) -}}
            {{- include "common.certManager.volumeMountsReadOnly" . | nindent 8 -}}
            {{- end -}}
          {{- end }}
        {{- if $certDir }}
        - emptyDir: {}
          name: tls-info
-         {{ if and .Values.certificates .Values.global.cmpv2Enabled .Values.global.CMPv2CertManagerIntegration -}}
+         {{ if (include "dcaegen2-services-common.shouldUseCmpv2Certificates" .) -}}
          {{ include "common.certManager.volumesReadOnly" . | nindent 6 }}
          {{- end }}
        {{- end }}
  */}}
  {{- define "dcaegen2-services-common._certPostProcessor" -}}
    {{- $certDir := default "" .Values.certDirectory . -}}
-   {{- if and $certDir .Values.certificates .Values.global.cmpv2Enabled .Values.global.CMPv2CertManagerIntegration -}}
+   {{- if (include "dcaegen2-services-common.shouldUseCmpv2Certificates" .) -}}
      {{- $cmpv2Certificate := (index .Values.certificates 0) -}}
      {{- $cmpv2CertificateDir := $cmpv2Certificate.mountPath -}}
      {{- $certType := "pem" -}}
        value: {{ $keystoreDestinationPaths | quote }}
    {{- end }}
  {{- end -}}
+ {{/*
+   Template returns string "true" if CMPv2 certificates should be used and nothing (so it can be used in with statements)
+   when they shouldn't. Example use:
+     {{- if (include "dcaegen2-services-common.shouldUseCmpv2Certificates" .) -}}
+ */}}
+ {{- define "dcaegen2-services-common.shouldUseCmpv2Certificates" -}}
+   {{- $certDir := default "" .Values.certDirectory . -}}
+   {{- if (and $certDir .Values.certificates .Values.global.cmpv2Enabled .Values.global.CMPv2CertManagerIntegration .Values.useCmpv2Certificates) -}}
+   true
+   {{- end -}}
+ {{- end -}}
@@@ -41,7 -41,7 +41,7 @@@ certPostProcessorImage: onap/org.onap.o
  # Application configuration defaults.
  #################################################################
  # application image
 -image: onap/org.onap.dcaegen2.collectors.ves.vescollector:1.8.0
 +image: onap/org.onap.dcaegen2.collectors.ves.vescollector:1.9.2
  pullPolicy: Always
  
  # log directory where logging sidecar should look for log files
@@@ -58,8 -58,13 +58,13 @@@ certDirectory: /opt/app/dcae-certificat
  tlsServer: true
  
  # CMPv2 certificate
- # It is used only when global parameter cmpv2Enabled is true
+ # It is used only when:
+ # - certDirectory is set
+ # - global cmpv2Enabled flag is set to true
+ # - global CertManagerIntegration flag is set to true
+ # - flag useCmpv2Certificates is set to true
  # Disabled by default
+ useCmpv2Certificates: false
  certificates:
    - mountPath: /opt/app/dcae-certificate/external
      commonName: dcae-ves-collector