review: unnest if

Signed-off-by: Dr. Jan-Philip Gehrcke <[email protected]>

Author: jgehrcke

cmd/compute-domain-kubelet-plugin/cleanup.go

@@ -112,20 +112,21 @@ func (m *CheckpointCleanupManager) unprepareIfStale(ctx context.Context, cpuid s
 	}
 
 	claim, err := m.getClaimByName(ctx, cpclaim.Name, cpclaim.Namespace)
+	if err != nil && errors.IsNotFound(err) {
+
+		klog.V(4).Infof(
+			"Checkpointed RC cleanup: partially prepared claim '%s/%s:%s' is stale: not found in API server",
+			cpclaim.Namespace,
+			cpclaim.Name,
+			cpuid)
+		m.unprepare(ctx, cpuid, cpclaim)
+		return
+	}
+
+	// A transient error during API server lookup. No explicit retry required.
+	// The next periodic cleanup invocation will implicitly retry.
 	if err != nil {
-		if errors.IsNotFound(err) {
-			klog.V(4).Infof(
-				"Checkpointed RC cleanup: partially prepared claim '%s/%s:%s' is stale: not found in API server",
-				cpclaim.Namespace,
-				cpclaim.Name,
-				cpuid)
-			m.unprepare(ctx, cpuid, cpclaim)
-		} else {
-			// A transient error during API server lookup. No explicit retry
-			// required. The next periodic cleanup invocation will
-			// implicitly retry.
-			klog.Infof("Checkpointed RC cleanup: skip for checkpointed claim %s: getClaimByName failed (retry later): %s", cpuid, err)
-		}
+		klog.Infof("Checkpointed RC cleanup: skip for checkpointed claim %s: getClaimByName failed (retry later): %s", cpuid, err)
 		return
 	}