-
Notifications
You must be signed in to change notification settings - Fork 38.9k
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
[Federation][init-08] Refactor the tests by pulling the common utilities into a testing package. #35865
[Federation][init-08] Refactor the tests by pulling the common utilities into a testing package. #35865
Conversation
Jenkins GKE smoke e2e failed for commit 5accac6. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GKE smoke e2e failed for commit 5accac6. Full PR test history. The magic incantation to run this job again is |
Jenkins Kubemark GCE e2e failed for commit 5accac6. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE e2e failed for commit 5accac6. Full PR test history. The magic incantation to run this job again is |
I didn't review the code in detail (as that is done in a separate PR). But moving it to a library makes perfect sense. LGTM. |
…rver's service loadbalancer address before waiting. This speeds up the tests. Otherwise tests end up unnecessarily waiting for the poll interval/duration which is 5 seconds right now.
5accac6
to
72109a9
Compare
Jenkins unit/integration failed for commit 72109a9a1ab6bb4ab4af806a8e3540feb3b56e1b. Full PR test history. The magic incantation to run this job again is |
72109a9
to
87a2122
Compare
…ies into a testing package. This makes these utilities reusable by other packages.
87a2122
to
efea270
Compare
Jenkins verification failed for commit 87a21223a33626757a6b32703436f49dfb4d7df7. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE etcd3 e2e failed for commit 72109a9a1ab6bb4ab4af806a8e3540feb3b56e1b. Full PR test history. The magic incantation to run this job again is |
Just rebased the PR and fixed the failing tests/builds. Adding the LGTM label back. |
Jenkins GCE Node e2e failed for commit 72109a9a1ab6bb4ab4af806a8e3540feb3b56e1b. Full PR test history. The magic incantation to run this job again is |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
Jenkins GCI GCE e2e failed for commit efea270. Full PR test history. The magic incantation to run this job again is |
Automatic merge from submit-queue |
…akeclient Automatic merge from submit-queue Make the fake command factory return the clientset with appropriate rest clients for all the API groups. Please review only the last commit here. This is based on PR #35865 which will be reviewed independently. Design Doc: PR #34484 cc @kubernetes/sig-cluster-federation @nikhiljindal
Please review only the last commit here. This is based on PR #35864 which will be reviewed independently.
Design Doc: PR #34484
cc @kubernetes/sig-cluster-federation @nikhiljindal
This change is![Reviewable](https://camo.githubusercontent.com/2d899f4291d07d3cd2fa4aaae1e3b243f164c23fce87d30a589ace0d496a444c/68747470733a2f2f72657669657761626c652e6b756265726e657465732e696f2f7265766965775f627574746f6e2e737667)