-
Notifications
You must be signed in to change notification settings - Fork 984
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pass conformance test #137
Comments
/priority high |
/assign @asifdxtreme |
we need to get it done in v0.1 |
I have the following queries.
|
|
This part is ok to me: only test scheduler; but the binaries/images should be built from volcano repo :) |
what's current status? |
I ran it against the local kind cluster like i did before for the kube-batch, faced some crash issues with it. |
that's great, thanks :) |
These below test cases are failing.
|
These three test cases are related to the events fix, which is available in 'kube-batch' code, but it looks like its not vendored in this project.
This below test case fail even with default scheduler also, even with 'kube-batch' hence we can ignore this for now and investigate later.
|
These are new failures and are related to
|
Please help to cherry pick them :) |
what's the status of this? Can we pass conformance test now? |
I ran the tests with local kind cluster all were passing.
|
|
|
|
Even the test case which was failing in kube-batch is now passing when run manually.
|
The PR are merged, please bump to volcano. We need to get this done ASAP! |
Will close this issue as the k8s conformance test cases are passing. Method of testing:
k8s git commit with which this was tested.
Volcano commit hash:
kubectl version:
|
/assign |
/close |
@k82cn |
That's great, thanks :) |
Is this a BUG REPORT or FEATURE REQUEST?:
/kind feature
Description:
Cherry pick related PR in kube-batch to volcano-sh/kube-batch for conformance test.
/cc @asifdxtreme
The text was updated successfully, but these errors were encountered: