fix: ensure SetControllerReference to certificates

Co-authored-by: Andrey Kontyakov <[email protected]>
clastix · Apr 3, 2024 · dc3d506 · dc3d506
1 parent 06a55f6
commit dc3d506
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/internal/resources/ca_certificate.go b/internal/resources/ca_certificate.go
@@ -105,7 +105,7 @@ func (r *CACertificate) mutate(ctx context.Context, tenantControlPlane *kamajiv1
 			}
 
 			if isValid {
-				return nil
+				return ctrl.SetControllerReference(tenantControlPlane, r.resource, r.Client.Scheme())
 			}
 		}
 

diff --git a/internal/resources/front_proxy_ca_certificate.go b/internal/resources/front_proxy_ca_certificate.go
@@ -91,7 +91,7 @@ func (r *FrontProxyCACertificate) mutate(ctx context.Context, tenantControlPlane
 				logger.Info(fmt.Sprintf("%s certificate-private_key pair is not valid: %s", kubeadmconstants.FrontProxyCACertAndKeyBaseName, err.Error()))
 			}
 			if isValid {
-				return nil
+				return ctrl.SetControllerReference(tenantControlPlane, r.resource, r.Client.Scheme())
 			}
 		}
 

diff --git a/internal/resources/sa_certificate.go b/internal/resources/sa_certificate.go
@@ -90,7 +90,7 @@ func (r *SACertificate) mutate(ctx context.Context, tenantControlPlane *kamajiv1
 				logger.Info(fmt.Sprintf("%s public_key-private_key pair is not valid: %s", kubeadmconstants.ServiceAccountKeyBaseName, err.Error()))
 			}
 			if isValid {
-				return nil
+				return ctrl.SetControllerReference(tenantControlPlane, r.resource, r.Client.Scheme())
 			}
 		}
-Original file line number
+Diff line change
@@ Expand Up @@
     			}
     			if isValid {
-    				return nil
+    				return ctrl.SetControllerReference(tenantControlPlane, r.resource, r.Client.Scheme())
     			}
     		}
@@ Expand Down @@