fix: retry finalizer on http 422

csviri · csviri · commit 5cf7d41683df · 2025-04-28T17:42:40.000+02:00
this can happen if there are multiple finalizers present and multiple reconilers competent to remove their finalizer, since we remove the finalizer with JSON Patch

Signed-off-by: Attila Mészáros &lt;a_meszaros@apple.com&gt;
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/ReconciliationDispatcher.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/ReconciliationDispatcher.java
@@ -382,7 +382,7 @@ public P conflictRetryingPatch(
         log.trace("Exception during patch for resource: {}", resource);
         retryIndex++;
         // only retry on conflict (HTTP 409), otherwise fail
-        if (e.getCode() != 409) {
+        if (e.getCode() != 409 && e.getCode() != 422) {
           throw e;
         }
         if (retryIndex >= MAX_UPDATE_RETRY) {

Original file line number	Diff line number	Diff line change
`@@ -382,7 +382,7 @@ public P conflictRetryingPatch(`
`382`	`382`	`log.trace("Exception during patch for resource: {}", resource);`
`383`	`383`	`retryIndex++;`
`384`	`384`	`// only retry on conflict (HTTP 409), otherwise fail`
`385`		`- if (e.getCode() != 409) {`
	`385`	`+ if (e.getCode() != 409 && e.getCode() != 422) {`
`386`	`386`	`throw e;`
`387`	`387`	`}`
`388`	`388`	`if (retryIndex >= MAX_UPDATE_RETRY) {`