Revert "100 ms also leads to PATCH race conditions"

ThoSap · ThoSap · commit fc72b1d0666d · 2026-01-27T11:55:36.000+01:00
This reverts commit 0792764.
diff --git a/operator/src/main/java/it/aboutbits/postgresql/crd/database/DatabaseReconciler.java b/operator/src/main/java/it/aboutbits/postgresql/crd/database/DatabaseReconciler.java
@@ -119,7 +119,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(250, TimeUnit.MILLISECONDS);
+                    .rescheduleAfter(100, TimeUnit.MILLISECONDS);
         }
 
         // We do not actually delete the database if the reclaimPolicy is set to RETAIN, we only delete the CR instance
diff --git a/operator/src/main/java/it/aboutbits/postgresql/crd/defaultprivilege/DefaultPrivilegeReconciler.java b/operator/src/main/java/it/aboutbits/postgresql/crd/defaultprivilege/DefaultPrivilegeReconciler.java
@@ -141,7 +141,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(250, TimeUnit.MILLISECONDS);
+                    .rescheduleAfter(100, TimeUnit.MILLISECONDS);
         }
 
         var clusterRef = spec.getClusterRef();
diff --git a/operator/src/main/java/it/aboutbits/postgresql/crd/grant/GrantReconciler.java b/operator/src/main/java/it/aboutbits/postgresql/crd/grant/GrantReconciler.java
@@ -146,7 +146,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(250, TimeUnit.MILLISECONDS);
+                    .rescheduleAfter(100, TimeUnit.MILLISECONDS);
         }
 
         var clusterRef = spec.getClusterRef();
diff --git a/operator/src/main/java/it/aboutbits/postgresql/crd/role/RoleReconciler.java b/operator/src/main/java/it/aboutbits/postgresql/crd/role/RoleReconciler.java
@@ -149,7 +149,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(250, TimeUnit.MILLISECONDS);
+                    .rescheduleAfter(100, TimeUnit.MILLISECONDS);
         }
 
         var clusterRef = spec.getClusterRef();
diff --git a/operator/src/main/java/it/aboutbits/postgresql/crd/schema/SchemaReconciler.java b/operator/src/main/java/it/aboutbits/postgresql/crd/schema/SchemaReconciler.java
@@ -121,7 +121,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(250, TimeUnit.MILLISECONDS);
+                    .rescheduleAfter(100, TimeUnit.MILLISECONDS);
         }
 
         // We do not actually delete the schema if the reclaimPolicy is set to RETAIN, we only delete the CR instance

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ public DeleteControl cleanup(`
`119`	`119`	`context.getClient().resource(resource).patchStatus();`
`120`	`120`
`121`	`121`	`return DeleteControl.noFinalizerRemoval()`
`122`		`- .rescheduleAfter(250, TimeUnit.MILLISECONDS);`
	`122`	`+ .rescheduleAfter(100, TimeUnit.MILLISECONDS);`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`// We do not actually delete the database if the reclaimPolicy is set to RETAIN, we only delete the CR instance`
Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ public DeleteControl cleanup(`
`141`	`141`	`context.getClient().resource(resource).patchStatus();`
`142`	`142`
`143`	`143`	`return DeleteControl.noFinalizerRemoval()`
`144`		`- .rescheduleAfter(250, TimeUnit.MILLISECONDS);`
	`144`	`+ .rescheduleAfter(100, TimeUnit.MILLISECONDS);`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`var clusterRef = spec.getClusterRef();`
Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ public DeleteControl cleanup(`
`146`	`146`	`context.getClient().resource(resource).patchStatus();`
`147`	`147`
`148`	`148`	`return DeleteControl.noFinalizerRemoval()`
`149`		`- .rescheduleAfter(250, TimeUnit.MILLISECONDS);`
	`149`	`+ .rescheduleAfter(100, TimeUnit.MILLISECONDS);`
`150`	`150`	`}`
`151`	`151`
`152`	`152`	`var clusterRef = spec.getClusterRef();`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ public DeleteControl cleanup(`
`149`	`149`	`context.getClient().resource(resource).patchStatus();`
`150`	`150`
`151`	`151`	`return DeleteControl.noFinalizerRemoval()`
`152`		`- .rescheduleAfter(250, TimeUnit.MILLISECONDS);`
	`152`	`+ .rescheduleAfter(100, TimeUnit.MILLISECONDS);`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`var clusterRef = spec.getClusterRef();`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ public DeleteControl cleanup(`
`121`	`121`	`context.getClient().resource(resource).patchStatus();`
`122`	`122`
`123`	`123`	`return DeleteControl.noFinalizerRemoval()`
`124`		`- .rescheduleAfter(250, TimeUnit.MILLISECONDS);`
	`124`	`+ .rescheduleAfter(100, TimeUnit.MILLISECONDS);`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`// We do not actually delete the schema if the reclaimPolicy is set to RETAIN, we only delete the CR instance`