holdenk opened a new pull request #27905: [SPARK-31125][K8S]Terminating pods have a deletion timestamp but they are not yet dead. URL: https://github.com/apache/spark/pull/27905 ### What changes were proposed in this pull request? Change what we consider a deleted pod to not include "Terminating" ### Why are the changes needed? If we get a new snapshot while a pod is in the process of being cleaned up we shouldn't delete the executor until it is fully terminated. ### Does this PR introduce any user-facing change? No ### How was this patch tested? This should be covered by the decommissioning tests in that they currently are flaky because we sometimes delete the executor instead of allowing it to decommission all the way. I also ran this in a loop locally ~80 times with the only failures being the PV suite because of unrelated minikube mount issues.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
