viirya commented on code in PR #828:
URL: 
https://github.com/apache/spark-kubernetes-operator/pull/828#discussion_r4021376209


##########
spark-operator/src/main/java/org/apache/spark/k8s/operator/reconciler/reconcilesteps/AppInitStep.java:
##########
@@ -67,6 +67,10 @@ public ReconcileProgress reconcile(
       return proceed();
     }
     SparkApplication app = context.getResource();
+    if (app.getSpec().isSuspend()) {

Review Comment:
   Confirmed resolved in `18636ac`. The live GET prevents the stale 
pre-deletion snapshot from authorizing initialization, and lookup failures keep 
the hold. `staleInformerSnapshotDoesNotBypassSuspend` now covers the specific 
sequence discussed here and verifies that no driver is created and the 
application stays in `ScheduledToRestart`. Thanks for adding this coverage.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to