Fix dns ignore error field (#4)

* [FIX] Fixed ServiceAddPrincipal condition when ignoreError in Issuer is true --------- Co-authored-by: anghille <[email protected]>
Anghille · Jun 22, 2023 · e1206d0 · e1206d0
1 parent de8a82e
commit e1206d0
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/provisionners/freeipa.go b/provisionners/freeipa.go
@@ -163,8 +163,6 @@ func (s *FreeIPAPKI) Sign(ctx context.Context, cr *certmanager.CertificateReques
 			&freeipa.ServiceAddPrincipalArgs{Krbcanonicalname: name, Krbprincipalname: csr.DNSNames},
 			&freeipa.ServiceAddPrincipalOptionalArgs{}); err != nil && !s.spec.IgnoreError {
 			return nil, nil, fmt.Errorf("fail adding DNSNames SAN principal to the service %v : %v", name, err)
-		} else if err != nil && s.spec.IgnoreError {
-			log.Error(err, "Ignored Error> Failed to add DNSNames SAN principal to the service")
 		} else {
 			log.Info("Added DNSNames SAN principal to the service", "service", name)
 		}