pinkfloyd-eminem
pinkfloyd-eminem
test: single scheduler：helm install volcano installer/helm/chart/volcano/ ok multi schedulers: helm install volcano installer/helm/chart/volcano/ --set custom.multi_scheduler_enable=true --set custom.scheduler_replicas=3 ok
/assign @hzxuzhonghu
> Hello @pinkfloyd-eminem, Thank you for implementing my feature request! 😊 > > Can you edit the PR description for me please? At the part: **Which issue(s) this PR fixes:**...
> Thank you @pinkfloyd-eminem for doing the changes! This looks really good, I have one minor request left, since there is no purpose in defining the SCHEDULER_NUM env var for...
> Ray plugin may have a flaky test, you could submit your PR after you fix the code verify and trigger CI again to check whether if the E2E sequence...
> 感谢您的贡献，顺便问一下，您测试过吗？@pinkfloyd-eminem 多调度器能正确调度 Pod 吗？我想未来我们可以为多调度器模式添加端到端测试。 @JesseStutler I tested simple vcjob in multi-scheduler mode and it was successfully scheduled and executed to completion
> Ray plugin may have a flaky test, you could submit your PR after you fix the code verify and trigger CI again to check whether if the E2E sequence...
> @pinkfloyd-eminem The CI issue shall be handled in a separate PR and I think Wonki4 handles it in #4670 So let's wait until it get's into the repository and...
> @pinkfloyd-eminem #4670 has already merged, please rebase the latest code and fix the code verify ci, thanks Hi, I rebased the latest master branch and passed CI