fix tests

Zlaticanin · Zlaticanin · commit 6b8a49900ef1 · 2025-04-23T12:46:24.000-07:00
diff --git a/controllers/vaultdynamicsecret_controller.go b/controllers/vaultdynamicsecret_controller.go
@@ -272,8 +272,7 @@ func (r *VaultDynamicSecretReconciler) Reconcile(ctx context.Context, req ctrl.R
 		return ctrl.Result{RequeueAfter: computeHorizonWithJitter(requeueDurationOnError)}, nil
 	}
 
-	// check if lease already exists
-	if o.Status.SecretLease.ID != "" && o.Status.LastGeneration > 0 && o.Status.LastRenewalTime > 0 &&
+	if !doSync && o.Status.SecretLease.ID != "" && o.Status.LastGeneration > 0 && o.Status.LastRenewalTime > 0 &&
 		!r.SyncRegistry.Has(req.NamespacedName) && r.isRenewableLease(&o.Status.SecretLease, o, true) {
 		horizon, inWindow := computeRelativeHorizonWithJitter(o, staticCredsJitterHorizon)
 		if !inWindow {

Original file line number	Diff line number	Diff line change
`@@ -272,8 +272,7 @@ func (r *VaultDynamicSecretReconciler) Reconcile(ctx context.Context, req ctrl.R`
`272`	`272`	`return ctrl.Result{RequeueAfter: computeHorizonWithJitter(requeueDurationOnError)}, nil`
`273`	`273`	`}`
`274`	`274`
`275`		`- // check if lease already exists`
`276`		`- if o.Status.SecretLease.ID != "" && o.Status.LastGeneration > 0 && o.Status.LastRenewalTime > 0 &&`
	`275`	`+ if !doSync && o.Status.SecretLease.ID != "" && o.Status.LastGeneration > 0 && o.Status.LastRenewalTime > 0 &&`
`277`	`276`	`!r.SyncRegistry.Has(req.NamespacedName) && r.isRenewableLease(&o.Status.SecretLease, o, true) {`
`278`	`277`	`horizon, inWindow := computeRelativeHorizonWithJitter(o, staticCredsJitterHorizon)`
`279`	`278`	`if !inWindow {`