From 6e57e1093fac8b5762295dd64ed978171cba979d Mon Sep 17 00:00:00 2001 From: joshvanl Date: Wed, 30 Jun 2021 18:13:55 +0100 Subject: [PATCH] Adds comment about what the pickup ID is in the CSR controller Signed-off-by: joshvanl --- pkg/controller/certificatesigningrequests/venafi/venafi.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/controller/certificatesigningrequests/venafi/venafi.go b/pkg/controller/certificatesigningrequests/venafi/venafi.go index 89101536a..2dcc48492 100644 --- a/pkg/controller/certificatesigningrequests/venafi/venafi.go +++ b/pkg/controller/certificatesigningrequests/venafi/venafi.go @@ -127,6 +127,9 @@ func (v *Venafi) Sign(ctx context.Context, csr *certificatesv1.CertificateSignin return err } + // The signing process with Venafi is slow. The "pickupID" allows us to track + // the progress of the certificate signing. It is set as an annotation the + // first time the Certificate is reconciled. pickupID := csr.GetAnnotations()[experimentalapi.CertificateSigningRequestVenafiPickupIDAnnotationKey] // check if the pickup ID annotation is there, if not set it up.