Fix: Only add finalizer if not marked for deletion

This commit is contained in:
Daniel Hougaard
2024-06-13 22:53:54 +02:00
parent 68818beb38
commit 3472be480a
@@ -94,10 +94,12 @@ func (r *InfisicalSecretReconciler) Reconcile(ctx context.Context, req ctrl.Requ
fmt.Printf("\nRe-sync interval set. Interval: %v\n", requeueTime) fmt.Printf("\nRe-sync interval set. Interval: %v\n", requeueTime)
} }
// Add the finalizer if it does not exist // Add the finalizer if it does not exist, and only add it if the resource is not marked for deletion
if infisicalSecretCR.GetDeletionTimestamp() == nil || infisicalSecretCR.GetDeletionTimestamp().IsZero() {
if err := r.addFinalizer(ctx, &infisicalSecretCR); err != nil { if err := r.addFinalizer(ctx, &infisicalSecretCR); err != nil {
return ctrl.Result{}, err return ctrl.Result{}, err
} }
}
// Check if the resource is already marked for deletion // Check if the resource is already marked for deletion
if infisicalSecretCR.GetDeletionTimestamp() != nil { if infisicalSecretCR.GetDeletionTimestamp() != nil {