From 27eb5220a1e33ac8180c64414827c6923b448580 Mon Sep 17 00:00:00 2001 From: Tomasz Wrobel Date: Fri, 23 Oct 2020 08:34:41 +0200 Subject: [OOM-K8S-CERT-EXTERNAL-PROVIDER] Add logging of not supported/overridden CSR info Issue-ID: OOM-2559 Change-Id: I0199712e70d65650dd7ce0c0b95537373f94f7a4 Signed-off-by: Tomasz Wrobel --- .../src/cmpv2controller/certificate_request_controller.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'certServiceK8sExternalProvider/src/cmpv2controller/certificate_request_controller.go') diff --git a/certServiceK8sExternalProvider/src/cmpv2controller/certificate_request_controller.go b/certServiceK8sExternalProvider/src/cmpv2controller/certificate_request_controller.go index d526bbc8..81af0d36 100644 --- a/certServiceK8sExternalProvider/src/cmpv2controller/certificate_request_controller.go +++ b/certServiceK8sExternalProvider/src/cmpv2controller/certificate_request_controller.go @@ -41,6 +41,7 @@ import ( "sigs.k8s.io/controller-runtime/pkg/client" "onap.org/oom-certservice/k8s-external-provider/src/cmpv2api" + "onap.org/oom-certservice/k8s-external-provider/src/cmpv2controller/logger" provisioners "onap.org/oom-certservice/k8s-external-provider/src/cmpv2provisioner" ) @@ -122,14 +123,17 @@ func (controller *CertificateRequestController) Reconcile(k8sRequest ctrl.Reques } privateKeyBytes := privateKeySecret.Data[privateKeySecretKey] - // 8. Sign CertificateRequest + // 8. Log Certificate Request properties not supported or overridden by CertService API + logger.LogCertRequestProperties(ctrl.Log.WithName("CSR details"), certificateRequest) + + // 9. Sign CertificateRequest signedPEM, trustedCAs, err := provisioner.Sign(ctx, certificateRequest, privateKeyBytes) if err != nil { controller.handleErrorFailedToSignCertificate(ctx, log, err, certificateRequest) return ctrl.Result{}, err } - // 9. Store signed certificates in CertificateRequest + // 10. Store signed certificates in CertificateRequest certificateRequest.Status.Certificate = signedPEM certificateRequest.Status.CA = trustedCAs if err := controller.updateCertificateRequestWithSignedCerficates(ctx, certificateRequest); err != nil { -- cgit 1.2.3-korg