comment

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/ReconciliationDispatcher.java

@@ -381,7 +381,12 @@ public P conflictRetryingPatch(
       } catch (KubernetesClientException e) {
         log.trace("Exception during patch for resource: {}", resource);
         retryIndex++;
-        // only retry on conflict (HTTP 409), otherwise fail
+        // only retry on conflict (409) and unprocessable content (422) what
+        // can happen if JSON Patch is not a valid request since there was
+        // a concurrent request already removed another finalizer:
+        // List element removal from a list is by index in JSON Patch
+        // so if addressing a second finalizer but first is meanwhile removed
+        // it is a wrong request.
         if (e.getCode() != 409 && e.getCode() != 422) {
           throw e;
         }