chenyulin0719 opened a new pull request, #729: URL: https://github.com/apache/yunikorn-k8shim/pull/729
### What is this PR for? The PodCondition for pods pending due to exceeding the user/group limit was changed after YUNIKORN-1936. The e2e test should change the way to validate unscheduled pods. - Change validation function from WaitForPodUnschedulable() to WaitForPodPending(). - Sleep 5 seconds before check Pending pods becasue Pending is the initial PodPhase state after accepted.(5 sec is based on the the recent e2e test in CI and my local env.(~3 sec to unscheduled)) <img width="1432" alt="image" src="https://github.com/apache/yunikorn-k8shim/assets/26764036/5f1a703e-fd05-42dc-a4a8-b84add2c2a1b";> ### What type of PR is it? * [x] - Bug Fix * [ ] - Improvement * [ ] - Feature * [ ] - Documentation * [ ] - Hot Fix * [ ] - Refactoring ### Todos NA ### What is the Jira issue? https://issues.apache.org/jira/browse/YUNIKORN-2160 ### How should this be tested? Upgrade to the latest yunikorn-core, and run e2e tests under user_group_limit. (The issue will only occur in core version after YUNIKORN-1936. I'm not sure whether I should upgrade yunikorn-core in this PR, please let me know if it's ok to upgrade.) ### Screenshots (if appropriate) ### Questions: * [ ] - Should I upgrade yunikorn-core version in this PR? -- 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]
