-
Notifications
You must be signed in to change notification settings - Fork 273
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
List of skipped E2E cases #224
Comments
@ritazh Could you help to check whether some of them could be added into aks-engine, so that we could re-enable those tests? |
/assign @ritazh |
/assign @chewong |
@ritazh: GitHub didn't allow me to assign the following users: chewong. Note that only kubernetes members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@ritazh @feiskyer I ran all the tests mentioned above and here is the result: Job: ci-cloud-provider-azure-serial
Job: ci-cloud-provider-azure-multiple-zones
|
@chewong Most of them are because of configuration issues. Could you help to enable those in aks-engine? |
/assign |
Here are the update test results: Job: cloud-provider-azure-serial
Job: cloud-provider-azure-multiple-zones
|
This is a great start! These should help with |
I can start enabling these tests after Azure/aks-engine#2166 is included in the latest aks-engine release. |
With Azure Disk CSI Driver v0.5.0 released, I will open a PR in aks-engine to upgrade the CSI driver version so we can start enabling most of the skipped tests in ci-cloud-provider-azure-multiple-zones. |
@chewong thanks a lot for the work |
kubernetes/test-infra#15981 should re-enable most of the tests described above with aks-engine v0.46.0. On a separate note, I have compiled a list of skipped tests for all Azure-related presubmit & periodics jobs and noticed that we are skipping a lot of unnecessary test cases because either 1) they started passing again, 2) they are already being skipped with pull-cloud-provider-azure-e2e
cloud-provider-azure-conformance
cloud-provider-azure-slow
cloud-provider-azure-serial
cloud-provider-azure-conformance-vmss
cloud-provider-azure-slow-vmss
cloud-provider-azure-multiple-zones
pull-kubernetes-e2e-aks-engine-azure
|
Most of the skipped tests are now enabled (see #224 (comment)). Let's close this issue for now. |
@chewong: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
List of skipped cases:
The text was updated successfully, but these errors were encountered: