wilfred-s commented on PR #1068: URL: https://github.com/apache/yunikorn-k8shim/pull/1068#issuecomment-5312503680
This change by itself without proper handling of the bind failures would leave the system in a similar broken state. The cal to `Assume()` triggers an update of the task state which also triggers the async bind of the pod to the node. Without the proper handling of the volume and node binding errors that should happen in this case the task will be left in an allocated state in the k8shim and in a similar state on the core side. This change cleans up the cache on the k8shim side it does not handle any of the changes on the core side that should be triggered. For that we need YUNIKORN-2884 as that reverts the core side and moves it back to scheduling via the call to `rescheduleOnBindFailure()` -- 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]
