chenyulin0719 commented on PR #849: URL: https://github.com/apache/yunikorn-k8shim/pull/849#issuecomment-2128715339
Hi @pbacsko, The temporary release context lock method makes sense to me. However, I guess the root cause of the deadlock is other tests. When another test called 'dispatcher.UnregisterAllEventHandlers()', - https://github.com/apache/yunikorn-k8shim/blob/08a9b93dcc308f1e70bbf9a6596664877977e1bd/pkg/cache/scheduler_callback_test.go#L452-L455 the EventHandler will be cleaned and the wait group will never meet the count. - https://github.com/apache/yunikorn-k8shim/blob/08a9b93dcc308f1e70bbf9a6596664877977e1bd/pkg/cache/context.go#L1574 So the context lock is not released. -- 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]
